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

Helm release upgrades from v12.0.0 fail when changing the Mojaloop-Simulator Helm Chart version #2302

Closed
Tracked by #2151
mdebarros opened this issue Jun 17, 2021 · 3 comments
Assignees
Labels
bug Something isn't working or it has wrong behavior on a Mojaloop Core service oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it
Milestone

Comments

@mdebarros
Copy link
Member

mdebarros commented Jun 17, 2021

Summary:

Helm release upgrades from v12.0.0 fail when upgrading the Mojaloop-Simulator Helm Chart version:

The Deployment "dev3-sim-payeefsp-scheme-adapter" is invalid: MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable

Severity:
Medium

Priority:
Medium

Expected Behavior

Upgrades from 12.0.0 to 12.x.x should be successful.

Steps to Reproduce

  1. Install v12.0.0 with default config
  2. Upgrade v12.0.0 to v12.1.0 (WIP: feat(release): v13.0.0 wip (v12.1.0 earlier) helm#436) with default config

Specifications

  • Component (if known): Helm
  • Version: v12.x.x
  • Platform: Helm
  • Subsystem: Mojaloop-Simulators Helm Chart
  • Type of testing: Manual
  • Bug found/raised by: @mdebarros

Notes:

  • Severity when opened: Medium
  • Priority when opened: Medium
@mdebarros mdebarros added the bug Something isn't working or it has wrong behavior on a Mojaloop Core service label Jun 17, 2021
@mdebarros
Copy link
Member Author

mdebarros commented Jun 17, 2021

The issue is caused due to labels not following best practices, specifically:

Selector labels (such as app.kubernetes.io/name) should not contain any Helm version-specific or release-specific information as the selector-fields are immutable:

The Deployment "dev3-sim-payeefsp-scheme-adapter" is invalid: MatchExpressions:[]v1.LabelSelectorRequirement(nil)}: field is immutable

WIP [Helm release v12.1.0] includes changes to Mojaloop-Simulator specification to align labels to best practices -> mojaloop/helm@1b8c0bb.

This, unfortunately, means that v12.1.0 Helm Release will be breaking. As such the version was bumped up to v13.0.0 to reflect this.

@mdebarros mdebarros self-assigned this Jun 17, 2021
@elnyry-sam-k elnyry-sam-k added the oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it label Jun 18, 2021
@elnyry-sam-k elnyry-sam-k added this to the Sprint 14.4 milestone Jun 18, 2021
@elnyry-sam-k
Copy link
Member

@mdebarros would you recommend enhancing/fixing mojaloop simulator chart to avoid this issue in the future?

@mdebarros
Copy link
Member Author

@mdebarros would you recommend enhancing/fixing mojaloop simulator chart to avoid this issue in the future?

I have aligned the selectorLabels to best practices in Helm v13.0.0 release, and this issue should not impact upgrades going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working or it has wrong behavior on a Mojaloop Core service oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it
Projects
None yet
Development

No branches or pull requests

2 participants