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 goreleaser error and update README with --no-taint flag #108

Merged
merged 2 commits into from
Jan 18, 2024

Conversation

isaacnboyd
Copy link
Contributor

Hello. Very cool project! I am new to open source and would love to get involved.

Goreleaser published the deprecation of archives.replacements here

The solution is to include the replacement in the name_template.

I was able to get the goreleaser working locally with this change.

Doc update is related to #91

Goreleaser depricated archives.replacements on 2023-06-06 (v1.19.0).
Replacement config provided by goreleaser "Deprication notices" page
on goreleaser.com
Updating doc to reflect work done on commit
a68c7d4871db91edf7f7dbd69ee273cbbc8181af1
Copy link
Owner

@robscott robscott left a comment

Choose a reason for hiding this comment

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

Thanks for fixing this @isaacnboyd!

@@ -146,6 +146,7 @@ kube-capacity --pods --containers --util --output yaml
--kubeconfig string kubeconfig file to use for Kubernetes config
-n, --namespace string only include pods from this namespace
--namespace-labels string labels to filter namespaces with
--no-taint exclude nodes with taints
Copy link
Owner

Choose a reason for hiding this comment

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

I don't think this PR has merged yet (#98). Mind leaving this change out until I can figure out how to handle that PR and/or maybe just rebase it myself?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

the --no-taint flag was added in this commit. PR #101

Flag is initialized here. You can see it on main currently with -h.

Apologies I shouldn't have linked issue #91 because as you pointed out that has open PR #98 which is related to excluding specific taints.

Copy link
Owner

Choose a reason for hiding this comment

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

🤦 you're completely right, thanks!

@robscott
Copy link
Owner

I am new to open source and would love to get involved.

If you're interested in taking a more active role in this project, I could definitely use some help, feel free to ping me on k8s Slack if interested and we can figure out the details.

Copy link
Owner

@robscott robscott left a comment

Choose a reason for hiding this comment

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

Thanks @isaacnboyd!

@robscott robscott merged commit c675757 into robscott:main Jan 18, 2024
3 checks passed
robscott pushed a commit that referenced this pull request Feb 16, 2024
name_template was updated by #108 but the update included a bug.
The bug changed the asset names.

https://github.com/robscott/kube-capacity/releases/tag/v0.8.0-rc1

"amd64" weren't replaced with "x86_64".
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.

None yet

2 participants