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

feat[chart]: Release v1.8.0 #2571

Merged
merged 1 commit into from
Mar 28, 2022

Conversation

stevehipwell
Copy link
Contributor

@stevehipwell stevehipwell commented Feb 3, 2022

Description

This PR will release the merged chart changes from #2477, #2564 & #2626 that haven't been released yet. It will also add a CHANGELOG and update the chart contribution guide to refer to this. I've updated the CI tooling and added the functionality to add release notes to the GH chart release which should make chart change discovery better.

Closes #2570

Checklist

  • Unit tests updated
  • End user documentation updated

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Feb 3, 2022
@stevehipwell stevehipwell marked this pull request as ready for review February 17, 2022 17:21
@k8s-ci-robot k8s-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 17, 2022
Signed-off-by: Steve Hipwell <steve.hipwell@gmail.com>
@stevehipwell
Copy link
Contributor Author

/assign @Raffo

@stevehipwell
Copy link
Contributor Author

@Raffo could we get this merged?

@stevehipwell
Copy link
Contributor Author

@Raffo this is still waiting for a LGTM.

@Raffo
Copy link
Contributor

Raffo commented Feb 24, 2022

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Feb 24, 2022
@stevehipwell
Copy link
Contributor Author

@Raffo I think you also need to approve this as I've made a change to a file I'm not an owner of.

@MrRedHead
Copy link

@Raffo Please approve.

@stevehipwell
Copy link
Contributor Author

@njuettner or @seanmalloy could one of you approve this?

@Raffo
Copy link
Contributor

Raffo commented Mar 28, 2022

/approve

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Raffo, stevehipwell

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 28, 2022
@k8s-ci-robot k8s-ci-robot merged commit 2751dd0 into kubernetes-sigs:master Mar 28, 2022
@stevehipwell stevehipwell deleted the chart-v1-8-0 branch March 28, 2022 17:45
@stevehipwell
Copy link
Contributor Author

@Raffo the CI changes have an error, I'll open a new PR to fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release new chart version
4 participants