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

New Operator v0.5.2 Release #144

Merged
merged 10 commits into from
Apr 25, 2024
Merged

New Operator v0.5.2 Release #144

merged 10 commits into from
Apr 25, 2024

Conversation

mazin-s
Copy link
Contributor

@mazin-s mazin-s commented Apr 22, 2024

Performed the following changes

  • Bumped up version from 0.5.1 to 0.5.2
  • Bumped up appVersion from v0.5.0 to v0.5.1
  • Added some annotations that describe the changes we did (did not include all changes, included user related changes). Didn't include changes related to testing, logging for tests, etc as they don't effect user.
  • Removed the ability to specify the tag

Copy link
Contributor

@manavrajvanshi manavrajvanshi left a comment

Choose a reason for hiding this comment

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

LGTM, left a minor comment. Let me know what you think.

@@ -75,15 +75,14 @@ spec:
drop:
- ALL
securityContext:
runAsNonRoot: true
runAsNonRoot: true
Copy link
Collaborator

Choose a reason for hiding this comment

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

trailing space to remove

@@ -26,7 +26,7 @@ spec:
- --health-probe-bind-address=:8081
- --metrics-bind-address=127.0.0.1:8080
- --leader-elect
image: {{ .Values.image.repository }}:{{ .Values.image.tag | default $.Chart.AppVersion }}
image: {{ .Values.image.repository }}:{{ $.Chart.AppVersion }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

usually we keep tag exemple of customer who are using his own registry and who add tag suffix once image are validated or compressed

Copy link
Collaborator

@tuxtof tuxtof left a comment

Choose a reason for hiding this comment

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

LGTM

@tuxtof tuxtof merged commit 2d2507a into nutanix:master Apr 25, 2024
1 check passed
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.

3 participants