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

fix(core): Access configmap from another namespace for KEP-1755 #4849

Merged
merged 5 commits into from
Nov 3, 2023

Conversation

gansheer
Copy link
Contributor

@gansheer gansheer commented Oct 24, 2023

Ref #3615

Change kubernetes CLI usage to actually get the configmap local-registry-hosting in namespace kube-public for kind action registry configuration from KEP-1755.

Release Note

fix(core): Access configmap from another namespace for KEP-1755

@gansheer
Copy link
Contributor Author

@squakez @oscerd If you can fire the install/non-olm e2e workflow that would be nice.

@gansheer gansheer force-pushed the chore/test_kustomize_integration branch 2 times, most recently from a63ebbf to 9d12a92 Compare October 24, 2023 09:44
@gansheer gansheer changed the title chore(ci): Check kind registry configuration for kustomize fix(core): Access configmap from another namespace for KEP-1755 Oct 24, 2023
@gansheer gansheer marked this pull request as ready for review October 24, 2023 10:34
@gansheer gansheer force-pushed the chore/test_kustomize_integration branch from 9d12a92 to 028d7ac Compare October 24, 2023 10:43
@gansheer gansheer force-pushed the chore/test_kustomize_integration branch 3 times, most recently from 2f23084 to 86b2e95 Compare October 25, 2023 13:04
@squakez
Copy link
Contributor

squakez commented Oct 26, 2023

OLM checks should be fixed now. You can rebase please.

@gansheer gansheer force-pushed the chore/test_kustomize_integration branch from 86b2e95 to ed88203 Compare October 26, 2023 14:56
@gansheer
Copy link
Contributor Author

OLM checks should be fixed now. You can rebase please.

Done, but I still need to fix some configuration to really test KEP-1755.

@gansheer gansheer force-pushed the chore/test_kustomize_integration branch 2 times, most recently from 2678586 to 6674d90 Compare October 30, 2023 13:59
@gansheer gansheer marked this pull request as draft October 30, 2023 15:15
@gansheer gansheer force-pushed the chore/test_kustomize_integration branch 4 times, most recently from 1b9c713 to 59474f0 Compare November 2, 2023 09:24
@gansheer gansheer force-pushed the chore/test_kustomize_integration branch from 364be0d to c411563 Compare November 2, 2023 11:53
@gansheer gansheer marked this pull request as ready for review November 2, 2023 14:17
@gansheer
Copy link
Contributor Author

gansheer commented Nov 2, 2023

@squakez @oscerd could one of you please run again the failing tests ? 🙏

@squakez
Copy link
Contributor

squakez commented Nov 3, 2023

All green. Is it ready to merge?

@gansheer
Copy link
Contributor Author

gansheer commented Nov 3, 2023

Yes please.

@squakez squakez merged commit b5b094f into apache:main Nov 3, 2023
16 checks passed
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.

3 participants