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

feat: multinamespace name config #1786

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

imkira
Copy link

@imkira imkira commented May 22, 2024

What issue type does this pull request address? (keep at least one, remove the others)
/kind enhancement
/kind feature

What does this pull request do? Which issues does it resolve? (use resolves #<issue_number> if possible)

Support more flexible namespace naming format in the multi name space mode via configuration.

Please provide a short message that should be published in the vcluster release notes
Added more flexible namespace format (prefix, base, suffix) in multi-namespace mode

What else do we need to know?

This is part of a collaborative work at Woven by Toyota with my colleagues @ajithcnambiar and @tauxi

Copy link

netlify bot commented May 22, 2024

Deploy Preview for vcluster-docs canceled.

Built without sensitive environment variables

Name Link
🔨 Latest commit 60d2cc3
🔍 Latest deploy log https://app.netlify.com/sites/vcluster-docs/deploys/66a3475b5994e900080ac2dd

@heiko-braun
Copy link
Contributor

Thanks @imkira , would you mind to rebase this one?

@imkira imkira force-pushed the agora-multi-namespace-name-config branch from 890d322 to 8d72aa7 Compare July 8, 2024 12:41
@imkira
Copy link
Author

imkira commented Jul 9, 2024

@heiko-braun Hey, thanks for reaching out. Yes, I did the rebase. PTAL.

Support more flexible namespace naming format in the multi name space
mode via configuration.

Co-authored-by: Ajith Chandran <[email protected]>
Co-authored-by: Wojtek Koronski <[email protected]>
@imkira imkira force-pushed the agora-multi-namespace-name-config branch from 8d72aa7 to 60d2cc3 Compare July 26, 2024 06:51
@imkira
Copy link
Author

imkira commented Jul 26, 2024

@heiko-braun I rebased again (and worked through some conflicts). Could you please take another look and consider the merge? Thank you.

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