-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Use the appsv1 apigroup when building the reflector for replicasets. #816
Conversation
/lgtm |
/hold |
Can you add a changelog entry and bump to rc.1 already? |
Oh yes, will do! |
Done! |
cc @serathius @piosz @DirectXMan12 please stand by to publish the gcr.io image :) |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: brancz, tariq1890 The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@brancz Can we remove the hold? |
/hold cancel |
@tariq1890 this was not merged into master. |
@tariq1890 @brancz I believe it's been 7 days since the release candidate and no new issues were reported so we can do a final release? https://github.com/kubernetes/kube-state-metrics/releases/tag/v1.7.0-rc.1 |
No objections from me. Feel free to create a PR to update the release 1.7 branch if you wish. Once @brancz approves and pushes the stable image to quay and gcr, we can publish the stable release. |
This fixes #815