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

Update Helm and client-go deps #1662

Merged
merged 7 commits into from
Jul 30, 2021
Merged

Update Helm and client-go deps #1662

merged 7 commits into from
Jul 30, 2021

Conversation

lblackstone
Copy link
Member

Proposed changes

Related issues (optional)

Fix #1661

@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@lblackstone lblackstone marked this pull request as draft July 29, 2021 18:41
@lblackstone
Copy link
Member Author

lblackstone commented Jul 29, 2021

Apparently this change broke the Kustomize SDK, so I'll need to do some more work to figure out that dependency.

I'll move this change into the next release since it's possible that it could change behavior.

Here's the error message:

2021/07/29 18:34:39 This binary cannot use k8s.io code; it must use kyaml.

@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

1 similar comment
@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@github-actions
Copy link

Does the PR have any schema changes?

Looking good! No breaking changes found.
No new resources/functions.

@lblackstone lblackstone marked this pull request as ready for review July 30, 2021 17:14
@lblackstone lblackstone merged commit bebc483 into master Jul 30, 2021
@pulumi-bot pulumi-bot deleted the lblackstone/helm-dep branch July 30, 2021 17:16
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.

Upgrade helm dependency to >=3.6.1
2 participants