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.

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

Adding profile support for registry #442

Merged
merged 10 commits into from
Nov 15, 2021

Conversation

deepakraj1997
Copy link
Contributor

@deepakraj1997 deepakraj1997 commented Nov 8, 2021

Resolves #435

controllers/registry.go Outdated Show resolved Hide resolved
@deepakraj1997 deepakraj1997 changed the title Adding profile support for registry Adding profile support for registry [WIP] Nov 8, 2021
@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2021

Codecov Report

Merging #442 (29460ff) into master (7763e55) will increase coverage by 0.03%.
The diff coverage is 56.86%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #442      +/-   ##
==========================================
+ Coverage   41.48%   41.51%   +0.03%     
==========================================
  Files           9        9              
  Lines        1849     1869      +20     
==========================================
+ Hits          767      776       +9     
- Misses       1033     1040       +7     
- Partials       49       53       +4     
Impacted Files Coverage Δ
controllers/registry.go 41.01% <56.86%> (+0.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update af016c1...29460ff. Read the comment docs.

@deepakraj1997 deepakraj1997 changed the title Adding profile support for registry [WIP] Adding profile support for registry Nov 9, 2021
@deepakraj1997 deepakraj1997 requested review from shubham-pampattiwar and kaovilai and removed request for jwmatthews November 9, 2021 01:40
Copy link
Member

@shubham-pampattiwar shubham-pampattiwar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@deepakraj1997 Thank you for the PR. Overall changes look great ! Just added some suggestions.

controllers/registry.go Outdated Show resolved Hide resolved
controllers/registry.go Show resolved Hide resolved
controllers/registry.go Outdated Show resolved Hide resolved
controllers/registry.go Show resolved Hide resolved
@deepakraj1997
Copy link
Contributor Author

/retest

@deepakraj1997
Copy link
Contributor Author

@shubham-pampattiwar thanks for the review. I have made the requested changes. Do review when u have time. Thanks!

Copy link
Member

@shubham-pampattiwar shubham-pampattiwar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just added a couple of nits. Apart from that LGTM ! Thank you !

@dymurray dymurray merged commit f77a356 into openshift:master Nov 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for multiple AWS credential profiles
4 participants