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 image.registry field #1003

Merged
merged 1 commit into from
May 22, 2023

Conversation

lengrongfu
Copy link
Collaborator

No description provided.

@lengrongfu lengrongfu force-pushed the feat/chart-add-registry branch 3 times, most recently from 6590e0b to 5bc00f1 Compare January 4, 2023 07:29
Copy link
Collaborator

@thcdrt thcdrt left a comment

Choose a reason for hiding this comment

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

LGTM

@lengrongfu lengrongfu force-pushed the feat/chart-add-registry branch 2 times, most recently from db32eb4 to 232b50c Compare January 12, 2023 06:03
@MarcelMue
Copy link
Collaborator

Can you resolve the merge conflicts please? All the CI dependency issues should now be resolved.

@lengrongfu lengrongfu force-pushed the feat/chart-add-registry branch from 232b50c to 5143574 Compare March 2, 2023 02:56
@lengrongfu lengrongfu force-pushed the feat/chart-add-registry branch from 5143574 to ccd49b2 Compare May 19, 2023 06:31
@lengrongfu
Copy link
Collaborator Author

@MarcelMue Sorry, I've been out of touch for a while. PTAL

@MarcelMue
Copy link
Collaborator

I think the code generation needs to be re-run. Are you aware how it works from the makefile?

Signed-off-by: lengrongfu <[email protected]>
@lengrongfu lengrongfu force-pushed the feat/chart-add-registry branch from ccd49b2 to 83cbcdc Compare May 22, 2023 12:51
@MarcelMue MarcelMue merged commit 4cabfa9 into goharbor:master May 22, 2023
CoderTH pushed a commit to CoderTH/harbor-operator that referenced this pull request May 23, 2023
Signed-off-by: lengrongfu <[email protected]>
Signed-off-by: coderth <[email protected]>
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.

4 participants