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

Version 0.48.0 introduced bug/regression when installing using kubeconfig to separate cluster. #1332

Closed
GrahamDumpleton opened this issue Sep 23, 2023 · 1 comment · Fixed by #1333
Labels
bug This issue describes a defect or unexpected behavior carvel-accepted This issue should be considered for future work and that the triage process has been completed priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.

Comments

@GrahamDumpleton
Copy link

What steps did you take:

Using latest 0.48.0. This problem doesn't arise with 0.47.0.

Using App resources to install stuff into a separate cluster using a kubeconfig. Eg. something like:

metadata:
  name: vcluster-session-educates-cli-w03-s001
  namespace: educates-cli-w03
spec:
  cluster:
    kubeconfigSecretRef:
      key: config
      name: vc-shared-vcluster
    namespace: default

What happened:

The package will not be installed and fails with a message:

stderr: 'kapp: Error: namespaces "educates-cli-w03" not found'
stdout: 'Target cluster ''https://shared-vcluster.educates-cli-w03.svc.cluster.local/''(nodes: educates-control-plane)'

In other words, kapp-controller is now expecting a namespace with name the same as the namespace containing the App in the originating cluster, to exist in the target cluster, even though this should not be required.

In 0.47.0 there was no such requirement and everything worked fine.

Setting the new defaultNamespace property added in 0.48.0 doesn't workaround the regression.

What did you expect:

Should have worked like in 0.47.0 when using kubeconfig to install in a different cluster.

Anything else you would like to add:

Nope.

Environment:

  • kapp Controller version: 0.48.0

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@GrahamDumpleton GrahamDumpleton added bug This issue describes a defect or unexpected behavior carvel-triage This issue has not yet been reviewed for validity labels Sep 23, 2023
@praveenrewar praveenrewar added carvel-accepted This issue should be considered for future work and that the triage process has been completed priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. and removed carvel-triage This issue has not yet been reviewed for validity labels Sep 26, 2023
@GrahamDumpleton
Copy link
Author

Confirmed that appears to fix issue was experiencing.

@github-actions github-actions bot added the carvel-triage This issue has not yet been reviewed for validity label Sep 28, 2023
@praveenrewar praveenrewar removed the carvel-triage This issue has not yet been reviewed for validity label Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior carvel-accepted This issue should be considered for future work and that the triage process has been completed priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants