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

[UI] Allow namespace creation when deploying default chart in Rancher #319

Closed
mmartin24 opened this issue Aug 29, 2023 · 3 comments
Closed
Labels
area/embedded kind/enhancement New feature or request

Comments

@mmartin24
Copy link
Contributor

Is your feature request related to a problem? Please describe.
When deploying a default Epinio chart namespace creation is not possible; however, this is possible when deploying a custom Epinio chart.

Installation options when using default Epinio charts:
image

Installation options when using custom Epinio chart from repo:
image

Besides the inconsistency, this may be an issue while working with the new Epinio extension. We noticed that attempting to deploy the default Epinio chart without selecting the Epinio namespace (because it cannot be added) will end up with the application not being found in the cluster:
image

Describe the solution you'd like
Creating a namespace from the default chart should be possible

Describe alternatives you've considered

Additional context

@mmartin24 mmartin24 added the kind/enhancement New feature or request label Aug 29, 2023
@mmartin24 mmartin24 changed the title Allow namespace creation when deploying default chart [UI] Allow namespace creation when deploying default chart in Rancher Aug 29, 2023
@thehejik
Copy link
Contributor

it looks like the extension is expecting Epinio to be installed in epinio namespace. Epinio rancher app is using hardcoded namespace cattle-epinio-system.

@richard-cox
Copy link
Member

There's two parts to this

  1. Namespace is not shown when installing 102.0.3+up1.8.1 (rancher release), but is for 1.9.0 (dev epinio release)
    This is because the 1.8.1 chart contains the annotation catalog.cattle.io/release-name, which tells the UI / Rancher that the chart must be installed to it's a specific namespace. Going on @enrichman comment this requirement has been dropped. So the newer chart must not contain the annotation. This is working as expected.

  2. When installing epinio to a namespace other than epinio it's not recognised as an epinio cluster by the extension
    We hadn't tracked this in an issue, so created [ui] Ensure embedded UI works when Epinio is installed in namespaces other than epinio #322

@richard-cox
Copy link
Member

Given the above, i think we can close this one

@richard-cox richard-cox moved this to To Test in UI Aug 29, 2023
@github-project-automation github-project-automation bot moved this from To Test to Done in UI Aug 30, 2023
@andreas-kupries andreas-kupries moved this from Done to Archivable in Epinio planning Sep 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/embedded kind/enhancement New feature or request
Projects
Archived in project
Status: Done
Development

No branches or pull requests

4 participants