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

Add support for cross-namespace sourceRef in ImageUpdateAutomation #299

Merged
merged 1 commit into from
Jan 25, 2022

Conversation

aryan9600
Copy link
Member

@aryan9600 aryan9600 commented Jan 21, 2022

ImageUpdateAutomation objects can now refer to GitRepository objects in other
namespaces. Implemented by switching sourceRef from a SourceReference to a
dependency.CrossNamespaceDependencyReference.

Fix: #294
Fix: #201
Fix: #180

Signed-off-by: Sanskar Jaiswal [email protected]

@aryan9600 aryan9600 force-pushed the aryan9600/ref-gitrepo-cross-ns branch from e51acbe to a7d4c60 Compare January 21, 2022 08:59
@aryan9600 aryan9600 force-pushed the aryan9600/ref-gitrepo-cross-ns branch from 1410e16 to c5e1415 Compare January 21, 2022 09:13
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

Please add a test for a GitRepository and its SSH secret in a different namespace

@aryan9600 aryan9600 force-pushed the aryan9600/ref-gitrepo-cross-ns branch from a45b58e to 8bf5e1e Compare January 25, 2022 08:45
ImageUpdateAutomation objects can now refer to GitRepository objects in other
namespaces. Implemented by switching sourceRef from a SourceReference to a
dependency.CrossNamespaceDependencyReference.

Signed-off-by: Sanskar Jaiswal <[email protected]>
@aryan9600 aryan9600 force-pushed the aryan9600/ref-gitrepo-cross-ns branch from 8bf5e1e to 3de51e7 Compare January 25, 2022 08:51
Copy link
Member

@stefanprodan stefanprodan left a comment

Choose a reason for hiding this comment

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

LGTM

Thanks @aryan9600

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
2 participants