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

Ensure PULUMI_KUBERNETES_MANAGED_BY_LABEL doesn't cause diffs on updates #1508

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

stack72
Copy link
Contributor

@stack72 stack72 commented Mar 29, 2021

Fixes: #1357

Before:

▶ PULUMI_KUBERNETES_MANAGED_BY_LABEL=mycompany pulumi up
Previewing update (dev)

View Live: https://app.pulumi.com/stack72/test-kubernetes-go/dev/previews/9f709e63-1719-4493-8ed5-515882fcc70b

     Type                              Name                    Plan       Info
     pulumi:pulumi:Stack               test-kubernetes-go-dev
 ~   └─ kubernetes:apps/v1:Deployment  app-dep                 update     [diff: ~metadata]

Resources:
    ~ 1 to update
    1 unchanged

Do you want to perform this update? no
confirmation declined, not proceeding with the update

After:

▶ PULUMI_KUBERNETES_MANAGED_BY_LABEL=mycompany pulumi up
Previewing update (dev)

View Live: https://app.pulumi.com/stack72/test-kubernetes-go/dev/previews/91ef271f-3000-472f-b41e-b5470d4cca25

     Type                 Name                    Plan
     pulumi:pulumi:Stack  test-kubernetes-go-dev

Resources:
    2 unchanged

Proposed changes

Related issues (optional)

Fixes: #1357

Before:

```
▶ PULUMI_KUBERNETES_MANAGED_BY_LABEL=mycompany pulumi up
Previewing update (dev)

View Live: https://app.pulumi.com/stack72/test-kubernetes-go/dev/previews/9f709e63-1719-4493-8ed5-515882fcc70b

     Type                              Name                    Plan       Info
     pulumi:pulumi:Stack               test-kubernetes-go-dev
 ~   └─ kubernetes:apps/v1:Deployment  app-dep                 update     [diff: ~metadata]

Resources:
    ~ 1 to update
    1 unchanged

Do you want to perform this update? no
confirmation declined, not proceeding with the update
```

After:

```
▶ PULUMI_KUBERNETES_MANAGED_BY_LABEL=mycompany pulumi up
Previewing update (dev)

View Live: https://app.pulumi.com/stack72/test-kubernetes-go/dev/previews/91ef271f-3000-472f-b41e-b5470d4cca25

     Type                 Name                    Plan
     pulumi:pulumi:Stack  test-kubernetes-go-dev

Resources:
    2 unchanged
```
@stack72 stack72 self-assigned this Mar 29, 2021
@lblackstone lblackstone merged commit df20c05 into master Mar 29, 2021
@pulumi-bot pulumi-bot deleted the fix-managed-by-label-diffs branch March 29, 2021 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Enhancement Request: Be able to control the app.kubernetes.io/managed-by label
2 participants