Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix namespace logos on s3 #1807

Merged
merged 6 commits into from
Sep 9, 2023
Merged

Fix namespace logos on s3 #1807

merged 6 commits into from
Sep 9, 2023

Conversation

newswangerd
Copy link
Member

Add minio CI tests

Issue: AAH-2575

@github-actions github-actions bot added backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) labels Jul 20, 2023
@newswangerd newswangerd force-pushed the fix/AAH-2575-namespace-logo-s3 branch 2 times, most recently from 23a518a to 7fcd107 Compare July 20, 2023 14:31
@newswangerd newswangerd removed backport-4.2 This PR should be backported to stable-4.2 (1.2) backport-4.4 This PR should be backported to stable-4.4 (2.1) backport-4.5 This PR should be backported to stable-4.5 (2.2) backport-4.6 This PR should be backported to stable-4.6 (2.3) backport-4.7 This PR should be backported to stable-4.7 (2.4) labels Jul 20, 2023
@newswangerd newswangerd assigned jctanner and unassigned jctanner Jul 20, 2023
@newswangerd newswangerd requested a review from jctanner July 20, 2023 14:35
@newswangerd newswangerd force-pushed the fix/AAH-2575-namespace-logo-s3 branch from 7fcd107 to 6376fb4 Compare July 31, 2023 19:58
@drodowic
Copy link
Collaborator

/retest

newswangerd and others added 3 commits August 28, 2023 17:04
@newswangerd @bmclaughlin
Fix namespace logos on S3
04bda19 
Issue: AAH-2575
@bmclaughlin
Fix deprecation warning
275325b 
AssertionError: [DEPRECATION WARNING]: Setting verbosity before the arg sub command is
deprecated, set the verbosity after the sub command. This feature will be
removed from ansible-core in version 2.13. Deprecation warnings can be disabled
by setting deprecation_warnings=False in ansible.cfg.

No-Issue
@bmclaughlin
Updated proxy for tests
84cb2b1 
Issue: AAH-2575
@bmclaughlin bmclaughlin force-pushed the fix/AAH-2575-namespace-logo-s3 branch from 6376fb4 to 84cb2b1 Compare September 7, 2023 17:38
@jctanner
Copy link
Collaborator

jctanner commented Sep 8, 2023

The new proxy scripts needs to be copied to profiles/insights/proxy and then tests should start passing.

@jctanner
Copy the new proxy script to the oci profile path.
db5f80b 
No-Issue

Signed-off-by: James Tanner <[email protected]>
@jctanner
Add workaround for keycloak issues.
5f45a72 
No-Issue

Signed-off-by: James Tanner <[email protected]>
@jctanner
Add another workaround for keycloak.
778c46d 
No-Issue

Signed-off-by: James Tanner <[email protected]>
@jctanner jctanner merged commit 08c2985 into ansible:master Sep 9, 2023
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jctanner jctanner jctanner approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@newswangerd @drodowic @jctanner @bmclaughlin