Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

chore(release): cut v1.1.0-rc.1 #4630

Merged
merged 7 commits into from
Apr 4, 2022

Conversation

schristoff
Copy link
Contributor

  • [ x] I have cherry-picked any changes labelled needs-cherry-pick-vX.Y.Z

  • [ x] I have made all of the following version and patch updates:

    1. Updated the container image tag in charts/osm/values.yaml
    2. Updated the chart and app version in charts/osm/Chart.yaml
    3. Updated the image tags used in the automated demo
    4. Regenerated the Helm chart README.md
  • [x ] I have checked that the base branch for this PR is correct as defined by the release guide

Is this a release candidate?

Signed-off-by: Sarah Christoff <[email protected]>
Signed-off-by: Sarah Christoff <[email protected]>
@@ -10,7 +10,7 @@ jobs:
env:
DOCKER_USER: ${{ secrets.RELEASE_DOCKER_USER }}
DOCKER_PASS: ${{ secrets.RELEASE_DOCKER_PASS }}
CTR_TAG: latest-main
CTR_TAG: v1.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

All of the CI config should stay the same

charts/osm/values.yaml Outdated Show resolved Hide resolved
tests/e2e/e2e_upgrade_test.go Outdated Show resolved Hide resolved
@@ -43,7 +43,7 @@ var testPresetMeshConfigMap *corev1.ConfigMap = &corev1.ConfigMap{
"logLevel": "error",
"maxDataPlaneConnections": 0,
"envoyImage": "envoyproxy/envoy-alpine:v1.19.3@sha256:874e699857e023d9234b10ffc5af39ccfc9011feab89638e56ac4042ecd4b0f3",
"initContainerImage": "openservicemesh/init:latest-main",
"initContainerImage": "openservicemesh/init:v1.1.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

This test will still pass as-is, so we don't need to change this. If the test passes without this line at all, I'd also be in favor of removing it.

@codecov-commenter
Copy link

codecov-commenter commented Apr 4, 2022

Codecov Report

Merging #4630 (4ad9fbc) into release-v1.1 (30e5362) will not change coverage.
The diff coverage is n/a.

❗ Current head 4ad9fbc differs from pull request most recent head f068de5. Consider uploading reports for the commit f068de5 to get more accurate results

@@              Coverage Diff              @@
##           release-v1.1    #4630   +/-   ##
=============================================
  Coverage         68.97%   68.97%           
=============================================
  Files               213      213           
  Lines             14861    14861           
=============================================
  Hits              10250    10250           
  Misses             4558     4558           
  Partials             53       53           
Flag Coverage Δ
unittests 68.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/ticker/ticker.go 83.33% <0.00%> (-3.85%) ⬇️
pkg/messaging/workqueue.go 100.00% <0.00%> (+10.71%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 30e5362...f068de5. Read the comment docs.

charts/osm/Chart.yaml Outdated Show resolved Hide resolved
@@ -18,7 +18,7 @@ version: 1.0.0

# This is the version number of the application being deployed. This version number should be
# incremented each time you make changes to the application.
appVersion: latest-main
appVersion: v1.1.0
Copy link
Contributor

Choose a reason for hiding this comment

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

Are we cutting an RC first? v1.1.0-rc.1

@@ -18,7 +18,7 @@ version: 1.0.0

Copy link
Contributor

Choose a reason for hiding this comment

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

The version field should also be updated

Signed-off-by: Sarah Christoff <[email protected]>
@shashankram shashankram changed the title chore(release): cut v1.1.0 chore(release): cut v1.1.0-rc.1 Apr 4, 2022
Signed-off-by: Sarah Christoff <[email protected]>
@@ -1,6 +1,6 @@
# Open Service Mesh Helm Chart

![Version: 1.0.0](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: latest-main](https://img.shields.io/badge/AppVersion-latest--main-informational?style=flat-square)
![Version: 1.0.0](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v1.1.0](https://img.shields.io/badge/AppVersion-v1.1.0-informational?style=flat-square)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
![Version: 1.0.0](https://img.shields.io/badge/Version-1.0.0-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v1.1.0](https://img.shields.io/badge/AppVersion-v1.1.0-informational?style=flat-square)
![Version: 1.1.0-rc.1](https://img.shields.io/badge/Version-1.1.0--rc.1-informational?style=flat-square) ![Type: application](https://img.shields.io/badge/Type-application-informational?style=flat-square) ![AppVersion: v1.1.0-rc.1](https://img.shields.io/badge/AppVersion-v1.1.0--rc.1-informational?style=flat-square)

@nojnhuh nojnhuh merged commit 2820d82 into openservicemesh:release-v1.1 Apr 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants