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

Conversation

shashankram
Copy link
Member

Backports e94c48b to release-v1.2

Updates the scripts to be in sync with the manifests
used in the manual demo documented on the website and
to be compatible with changes made to the bookstore app
labels in 5f54056.

Also fixes #4906 and updates the release notes for
v1.2.

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project? no

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change? no

  3. Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)? yes

…mesh#4910)

Updates the scripts to be in sync with the manifests
used in the manual demo documented on the website and
to be compatible with changes made to the bookstore app
labels in 5f54056.

Also fixes openservicemesh#4906 and updates the release notes for
v1.2.

Signed-off-by: Shashank Ram <shashr2204@gmail.com>
Copy link
Contributor

@nojnhuh nojnhuh left a comment

Choose a reason for hiding this comment

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

Let's include the release notes changes here too.

@shashankram
Copy link
Member Author

Let's include the release notes changes here too.

It was already included in https://github.com/openservicemesh/osm/pull/4905/files that was merged into release-v1.2.

Copy link
Contributor

@jaellio jaellio left a comment

Choose a reason for hiding this comment

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

Should these lint errors (unrelated to this change) be addressed before release?

@shashankram
Copy link
Member Author

Should these lint errors (unrelated to this change) be addressed before release?

I see the same lint errors on the main branch as well. Not sure what caused it, but my guess is the golangci-lint version used in the CI updated to a newer version, flagging some gosec warnings.

@shashankram shashankram merged commit 14d457b into openservicemesh:release-v1.2 Jul 18, 2022
@shashankram shashankram deleted the rel-v1.2 branch July 18, 2022 18:49
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.

3 participants