Fix image push tests permissions in CI#128
Fix image push tests permissions in CI#128Siddhant-K-code wants to merge 3 commits intoapple:mainfrom
Conversation
|
Thanks for the PR! Will take a deeper look and get back! In the meantime could you please set up commit signing and re push your commits? |
0b27707 to
825f0ad
Compare
|
Thanks, I've signed the commit |
c2b4575 to
f044c26
Compare
|
👋🏻 @adityaramani, any progress on this? |
|
Hi @Siddhant-K-code, please run |
8b94a70 to
108db95
Compare
108db95 to
4fac487
Compare
adityaramani
left a comment
There was a problem hiding this comment.
Sorry for the long delay.
I like the update to the ImageStoreTests, but Im not sure about the RegistryClientTests
Do you want to back that out from the PR and we can re-think if there is a better way to do it?
adityaramani
left a comment
There was a problem hiding this comment.
Thanks! Looks good! Just two small comments
602f4a5 to
7e45ae0
Compare
|
Looks like you need to run |
Implemented a mock-based testing approach that validates push functionality without requiring external registry permissions.
fixes #56