Skip to content

Conversation

@everettraven
Copy link
Contributor

Description of the change:
Prep for the v1.29.0 release

Motivation for the change:
Release v1.29.0 of the operator-sdk

Checklist

If the pull request includes user-facing changes, extra documentation is required:

@openshift-ci openshift-ci bot requested review from anik120 and oceanc80 May 31, 2023 18:58
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 18:58 — with GitHub Actions Inactive
### Bug Fixes

- (docs) Update the go version in the developer guide. The documentation wasn't updated when the go version was bumped to v1.19. ([#6101](https://github.com/operator-framework/operator-sdk/pull/6101))
- Compress the bundle content, to avoid the configMap exceed max length error. The error will look like this:
Copy link
Contributor

Choose a reason for hiding this comment

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

IMHO this was a feature, not a bug.
Purely academic and non-blocking.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fair enough. This would have been generated as a bug fix since that is what the changelog fragment specified it as such here

Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

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

/lgtm

### Changes

- Update kuttl to v0.15.0 in the scorecard-test-kuttl image. ([#6401](https://github.com/operator-framework/operator-sdk/pull/6401))
- Bump the golang base image version in the ansible-operator Dockerfiles from 1.18 to 1.19. ([#6398](https://github.com/operator-framework/operator-sdk/pull/6398))
Copy link
Member

Choose a reason for hiding this comment

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

Nit:

Suggested change
- Bump the golang base image version in the ansible-operator Dockerfiles from 1.18 to 1.19. ([#6398](https://github.com/operator-framework/operator-sdk/pull/6398))
- [ansible/v1] Bump the golang base image version in the ansible-operator Dockerfiles from 1.18 to 1.19. ([#6398](https://github.com/operator-framework/operator-sdk/pull/6398))

@openshift-ci openshift-ci bot added lgtm Indicates that a PR is ready to be merged. and removed lgtm Indicates that a PR is ready to be merged. labels May 31, 2023
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:09 — with GitHub Actions Inactive
Signed-off-by: Bryce Palmer <[email protected]>
Signed-off-by: Bryce Palmer <[email protected]>
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:13 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:13 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:14 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:15 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:16 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:16 — with GitHub Actions Inactive
@everettraven everettraven temporarily deployed to deploy May 31, 2023 19:16 — with GitHub Actions Inactive
Copy link
Member

@rashmigottipati rashmigottipati left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 31, 2023
@everettraven
Copy link
Contributor Author

Overriding the sanity/docs check and merging this in

@everettraven everettraven merged commit 78c5643 into operator-framework:master May 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm Indicates that a PR is ready to be merged.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants