Skip to content

bump to v1.9.0 - #2177

Merged
FxKu merged 5 commits into
masterfrom
bump-v1.9.0
Jan 30, 2023
Merged

FxKu merged 5 commits into
masterfrom
bump-v1.9.0

Conversation

@FxKu

@FxKu FxKu commented Jan 17, 2023

Copy link
Copy Markdown
Contributor
  • bump operator image in deployment manifest
  • bump logical backup default image
  • update version in helm charts and create new packages
  • drop v1.6.x tgz packages from helm folders

@FxKu FxKu added this to the 1.9 milestone Jan 17, 2023
registry: registry.opensource.zalan.do
repository: acid/postgres-operator-ui
tag: v1.8.2
tag: v1.9.0

@dmvolod dmvolod Jan 17, 2023

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@FxKu do you mind that master branch should contains latest tag and specific version tag should be present inside the chart archive only. Otherwise is not possible to install chart from the current master in the development (with some modifications manifests values).
wdyt about it?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

well my idea is to tag a release as soon as this PR is merged :)
but we can also think of changing it to latest so that issues like #2153 stop popping up.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Yes, indeed. We could always have the latest tag and imagePullPolicy: Always on the master and create release script to automate this operation.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Keep in mind! The default imagePullPolicy always pulls the :latest tag! As such, it should not be specified for this usecase.

https://kubernetes.io/docs/concepts/containers/images/#imagepullpolicy-defaulting

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@katlol I know about it, thank you. But for this case, it's better to specified it to simplify release scripts for helm chart if this approach could be implemented.

@monotek

monotek commented Jan 24, 2023

Copy link
Copy Markdown

Before that release it would be realy nice to add an ci step, which pushes the chart to some real helm repo.

This can be achived very easily by using: https://github.com/helm/chart-releaser-action

@FxKu

FxKu commented Jan 26, 2023

Copy link
Copy Markdown
Contributor Author

@monotek we will do it for the next release when we have CI for multi-arch operator image

@idanovinda

Copy link
Copy Markdown
Collaborator

👍

1 similar comment
@FxKu

FxKu commented Jan 30, 2023

Copy link
Copy Markdown
Contributor Author

👍

@FxKu
FxKu merged commit 30b6124 into master Jan 30, 2023
@FxKu
FxKu deleted the bump-v1.9.0 branch January 30, 2023 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants