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(kubernetes): disable project cluster filtration by stack/detail #7562

Merged
merged 4 commits into from
Oct 25, 2019

Conversation

maggieneterval
Copy link
Contributor

@maggieneterval maggieneterval commented Oct 24, 2019

Closes spinnaker/spinnaker#4880, spinnaker/spinnaker#3490

When configuring a Spinnaker project with Kubernetes V2 accounts, only filtering by * for stack and detail will return any clusters. This is because unlike with the VM-based providers, an application cluster itself is not associated with a single stack/detail, and each of its workloads can be associated with different stacks/details depending on the moniker annotations added to each manifest. This violates some assumptions in the Clouddriver logic shared by all providers and would be non-trivial to fix. We brought this issues to the SIG and there was not a ton of interest in further investment in the Projects view at this time, so this fix is a bandaid so at least the limitations of configuring projects with Kubernetes V2 accounts is clear at configuration time.

kqGgZddefqB

Copy link
Contributor

@ezimanyi ezimanyi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚫 🖥️ 🖥️ 🖥️ ☕ 📚 ✔️ 👍

@maggieneterval maggieneterval added the ready to merge Reviewed and ready for merge label Oct 25, 2019
@mergify mergify bot merged commit 114303a into spinnaker:master Oct 25, 2019
@mergify mergify bot added the auto merged Merged automatically by a bot label Oct 25, 2019
christopherthielen added a commit to christopherthielen/deck that referenced this pull request Oct 28, 2019
d71daa5 fix(core/pipeline): fully re-render list of trigger configs after a delete (spinnaker#7571)
629a98f fix(core/pipeline): make revision dropdown usable, layout tweaks (spinnaker#7569)
ca176fc feat(provider/aws): Functions (listing and searching) (spinnaker#7568)
e49ffaf Revert "feat(provider/aws): Functions (listing and searching) (spinnaker#7536)" (spinnaker#7567)
114303a fix(kubernetes): disable project cluster filtration by stack/detail (spinnaker#7562)
86a365b feat(provider/aws): Functions (listing and searching) (spinnaker#7536)
6236a9f fix(core/pipeline): make UX less bad when a pipeline stage never happened (spinnaker#7563)
christopherthielen added a commit that referenced this pull request Oct 28, 2019
d71daa5 fix(core/pipeline): fully re-render list of trigger configs after a delete (#7571)
629a98f fix(core/pipeline): make revision dropdown usable, layout tweaks (#7569)
ca176fc feat(provider/aws): Functions (listing and searching) (#7568)
e49ffaf Revert "feat(provider/aws): Functions (listing and searching) (#7536)" (#7567)
114303a fix(kubernetes): disable project cluster filtration by stack/detail (#7562)
86a365b feat(provider/aws): Functions (listing and searching) (#7536)
6236a9f fix(core/pipeline): make UX less bad when a pipeline stage never happened (#7563)
Jammy-Louie pushed a commit to pivotal/deck that referenced this pull request Nov 8, 2019
d71daa5 fix(core/pipeline): fully re-render list of trigger configs after a delete (spinnaker#7571)
629a98f fix(core/pipeline): make revision dropdown usable, layout tweaks (spinnaker#7569)
ca176fc feat(provider/aws): Functions (listing and searching) (spinnaker#7568)
e49ffaf Revert "feat(provider/aws): Functions (listing and searching) (spinnaker#7536)" (spinnaker#7567)
114303a fix(kubernetes): disable project cluster filtration by stack/detail (spinnaker#7562)
86a365b feat(provider/aws): Functions (listing and searching) (spinnaker#7536)
6236a9f fix(core/pipeline): make UX less bad when a pipeline stage never happened (spinnaker#7563)
yunzhangit pushed a commit to yunzhangit/deck that referenced this pull request Mar 28, 2021
d71daa5 fix(core/pipeline): fully re-render list of trigger configs after a delete (spinnaker#7571)
629a98f fix(core/pipeline): make revision dropdown usable, layout tweaks (spinnaker#7569)
ca176fc feat(provider/aws): Functions (listing and searching) (spinnaker#7568)
e49ffaf Revert "feat(provider/aws): Functions (listing and searching) (spinnaker#7536)" (spinnaker#7567)
114303a fix(kubernetes): disable project cluster filtration by stack/detail (spinnaker#7562)
86a365b feat(provider/aws): Functions (listing and searching) (spinnaker#7536)
6236a9f fix(core/pipeline): make UX less bad when a pipeline stage never happened (spinnaker#7563)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto merged Merged automatically by a bot ready to merge Reviewed and ready for merge target-release/1.17
Projects
None yet
4 participants