Label node resources with cloud based on integrator
Bug #1830119 reported by
Cory Johns
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Charmed Kubernetes Testing |
Fix Released
|
Undecided
|
George Kraft | ||
| Kubernetes Worker Charm |
Fix Released
|
Low
|
George Kraft | ||
Bug Description
For Juju to provide the most useful error messages and hints when adding k8s models, it needs CDK to indicate which cloud is backing the cluster based on the presence of the integrator charms. The requested label on the node resource is:
juju.io/cloud: [ec2, azure, gce, openstack]
| Changed in charm-kubernetes-worker: | |
| assignee: | nobody → Cory Johns (johnsca) |
| status: | New → Triaged |
| description: | updated |
| summary: |
- Annotate nodes with cloud based on integrator + Label node resources with cloud based on integrator |
| Changed in charm-kubernetes-worker: | |
| importance: | Undecided → Low |
| summary: |
- Label node resources with cloud based on integrator + Annotate node resources with cloud based on integrator |
Revision history for this message
| Ian Booth (wallyworld) wrote | #1 |
| summary: |
- Annotate node resources with cloud based on integrator + Label node resources with cloud based on integrator |
| Changed in charm-kubernetes-worker: | |
| assignee: | Cory Johns (johnsca) → George Kraft (cynerva) |
| status: | Triaged → In Progress |
Revision history for this message
| George Kraft (cynerva) wrote | #2 |
| Changed in charmed-kubernetes-testing: | |
| assignee: | nobody → George Kraft (cynerva) |
| status: | New → In Progress |
| Changed in charm-kubernetes-worker: | |
| status: | In Progress → Fix Committed |
| milestone: | none → 1.17 |
| Changed in charmed-kubernetes-testing: | |
| status: | In Progress → Fix Committed |
| Changed in charm-kubernetes-worker: | |
| status: | Fix Committed → Fix Released |
| Changed in charmed-kubernetes-testing: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Just to fully clarify after I double checked the implementation for existing clouds, the "juju.io/cloud" value is set as a node label, not an annotation.