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

Charts[Gateway] Unable to install chart in a new namespace #353

Closed
3 tasks done
Tracked by #111
leninmehedy opened this issue Sep 18, 2023 · 1 comment · Fixed by #363
Closed
3 tasks done
Tracked by #111

Charts[Gateway] Unable to install chart in a new namespace #353

leninmehedy opened this issue Sep 18, 2023 · 1 comment · Fixed by #363
Assignees
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs released

Comments

@leninmehedy
Copy link
Member

leninmehedy commented Sep 18, 2023

Bill of Materials or CLI Version

No response

Execution Environment

JUnit Extensions

Slf4j Logging Implementation

None

To Reproduce

  1. Deploy chart in namespace-1
  2. Attempt to deploy helm chart in namespace-2

It errors with message:

Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "pod-monitoring-role" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata;

Describe the bug

There are two issues as below:

  1. We are currently creating "pod-monitoring-role" ClusterRole to be able to run helm chart test container with necessary access. However, we don't need to create the ClusterRole every time we deploy the chart.

Therefore, "pod-monitoring-role" ClusterRole needs to be deployed only once as shared resource. Ideally, we should also be able to remove after helm chart test is executed.

  1. Besides the shared GatewayClass also exhibit the same issue and therefore needs to be installed as shared resource.

Describe the expected behavior

A user should be able to deploy the helm chart to multiple namespaces

Whole JUnit/CLI Logs

helm install fst-namespace-test2 -n fst-lenin2  hedera-network                          
                                                                                                                                     
Error: INSTALLATION FAILED: rendered manifests contain a resource that already exists. Unable to continue with install: ClusterRole "pod-monitoring-role" in namespace "" exists and cannot be imported into the current release: invalid ownership metadata; annotation validation error: key "meta.helm.sh/release-name" must equal "fst-namespace-test2": current value is "fst-namespace-test"; annotation validation error: key "meta.helm.sh/release-namespace" must equal "fst-lenin2": current value is "fst-lenin"

Additional Context

No response

Tasks

  1. Bug released
    leninmehedy
  2. Bug released
    leninmehedy
  3. New Feature released
    leninmehedy
@leninmehedy leninmehedy added Bug A error that causes the feature to behave differently than what was expected based on design docs Pending Triage New issue that needs to be triaged by the team labels Sep 18, 2023
@leninmehedy leninmehedy self-assigned this Sep 18, 2023
@leninmehedy leninmehedy moved this to 🏗 In progress in FST Suite Sep 18, 2023
@leninmehedy leninmehedy changed the title Charts[Tester] Cannot install chart because of ClusterRole pod-monitor-role Charts[Tester] Cannot Install Chart In A New Namespace Because of Previous ClusterRole pod-monitor-role Sep 19, 2023
@leninmehedy leninmehedy removed the Pending Triage New issue that needs to be triaged by the team label Sep 19, 2023
@leninmehedy leninmehedy changed the title Charts[Tester] Cannot Install Chart In A New Namespace Because of Previous ClusterRole pod-monitor-role Charts[Tester] Unable to install chart in a new namespace Sep 19, 2023
@leninmehedy leninmehedy changed the title Charts[Tester] Unable to install chart in a new namespace Charts[Gateway] Unable to install chart in a new namespace Sep 19, 2023
@leninmehedy leninmehedy moved this from 🏗 In progress to 👀 In review in FST Suite Sep 20, 2023
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in FST Suite Sep 21, 2023
@swirlds-automation
Copy link
Contributor

🎉 This issue has been resolved in version 0.12.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug A error that causes the feature to behave differently than what was expected based on design docs released
Projects
Status: ✅ Done
2 participants