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

kubectl-rook-ceph 0.8.0 (new formula) #171289

Merged
merged 2 commits into from May 18, 2024
Merged

Conversation

Skaronator
Copy link
Contributor

  • Have you followed the guidelines for contributing?
  • Have you ensured that your commits follow the commit style guide?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

This adds the Rook.io CLI Plugin.

Main Repo: https://github.com/rook/rook
Plugin Repo: https://github.com/rook/kubectl-rook-ceph

While this go binary doesn't need kubectl to run, it's the recommended way of interacting with this tool. The name of the binary must be "kubectl-rook_ceph" in order for kubectl to find it correctly.

https://kubernetes.io/docs/tasks/extend-kubectl/kubectl-plugins/#naming-a-plugin

@github-actions github-actions bot added autosquash Automatically squash pull request commits according to Homebrew style. go Go use is a significant feature of the PR or issue new formula PR adds a new formula to Homebrew/homebrew-core labels May 9, 2024
@chenrui333 chenrui333 changed the title Add rook-ceph kubectl plugin kubectl-rook-ceph 0.8.0 (new formula) May 9, 2024
@Skaronator Skaronator force-pushed the rook-plugin branch 4 times, most recently from 169c3b8 to 5f8af9b Compare May 9, 2024 16:34
@github-actions github-actions bot added autosquash Automatically squash pull request commits according to Homebrew style. and removed autosquash Automatically squash pull request commits according to Homebrew style. labels May 9, 2024
@github-actions github-actions bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label May 9, 2024
Copy link
Contributor

github-actions bot commented May 9, 2024

Thanks for contributing to Homebrew! 🎉 It looks like you're having trouble with a CI failure. See our contribution guide for help. You may be most interested in the section on dealing with CI failures. You can find the CI logs in the Checks tab of your pull request.

@Skaronator Skaronator force-pushed the rook-plugin branch 2 times, most recently from ed4ea1c to 23cf064 Compare May 9, 2024 16:58
@github-actions github-actions bot added the autosquash Automatically squash pull request commits according to Homebrew style. label May 9, 2024
@github-actions github-actions bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label May 9, 2024
@Skaronator Skaronator force-pushed the rook-plugin branch 2 times, most recently from 029404b to f512339 Compare May 9, 2024 17:48
@chenrui333 chenrui333 force-pushed the rook-plugin branch 2 times, most recently from 2ebc1d9 to a300aab Compare May 9, 2024 19:08
@github-actions github-actions bot added the automerge-skip `brew pr-automerge` will skip this pull request label May 9, 2024
@chenrui333
Copy link
Member

Looks like there is some build discrepancy with goreleaser build, raise a PR in here, rook/kubectl-rook-ceph#290

Copy link
Contributor

@chenrui333 chenrui333 added the ready to merge PR can be merged once CI is green label May 18, 2024
@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label May 18, 2024
@chenrui333
Copy link
Member

need a rebase laster for the autobump part

Skaronator and others added 2 commits May 18, 2024 09:49
Adds binary with the normal kubectl plugin naming of "kubectl-rook_ceph"

Co-authored-by: Rui Chen <rui@chenrui.dev>

kubectl-rook-ceph: minor updates

Signed-off-by: Rui Chen <rui@chenrui.dev>

autobump: add kubectl-rook-ceph

Signed-off-by: Rui Chen <rui@chenrui.dev>
@BrewTestBot BrewTestBot added this pull request to the merge queue May 18, 2024
Merged via the queue into Homebrew:master with commit 27c5b23 May 18, 2024
15 checks passed
@Skaronator Skaronator deleted the rook-plugin branch May 18, 2024 14:11
@Skaronator
Copy link
Contributor Author

Awesome, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autobump automerge-skip `brew pr-automerge` will skip this pull request CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. go Go use is a significant feature of the PR or issue new formula PR adds a new formula to Homebrew/homebrew-core ready to merge PR can be merged once CI is green
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants