Skip to content

Conversation

@bluzarraga
Copy link
Member

We are using the future branches to make changes to the helm charts so we can continue to make changes to helm charts while code freeze is in place for 4.11 GA

YCShen1010 and others added 30 commits November 1, 2023 05:22
* Update OperandConfig with value reference

Signed-off-by: Daniel Fan <[email protected]>

* Add operator ns for EDB image list ConfigMap

Signed-off-by: Daniel Fan <[email protected]>

* add value reference for EDB license image

Signed-off-by: Daniel Fan <[email protected]>

* fix EDB operand image digest

Signed-off-by: Daniel Fan <[email protected]>

---------

Signed-off-by: Daniel Fan <[email protected]>
Signed-off-by: Ben Luzarraga <[email protected]>
for OperandConfig resources[*] array field

Signed-off-by: Daniel Fan <[email protected]>
* add storage class placeholder for EDB-Keycloak

* EDB-Keycloak storage configuration

Signed-off-by: YuChen <[email protected]>

---------

Signed-off-by: YuChen <[email protected]>
Co-authored-by: Piotr Godowski <[email protected]>
Co-authored-by: YuChen <[email protected]>
* Handle EDB Keycloak profile sizes

* Apply suggestions from code review

Co-authored-by: giacomoch <[email protected]>

* specify profile size for Keycloak

* EDB and keycloak size configuration

Signed-off-by: YuChen <[email protected]>

* align default EDB resources with the starterset

---------

Signed-off-by: YuChen <[email protected]>
Co-authored-by: Piotr Godowski <[email protected]>
Co-authored-by: giacomoch <[email protected]>
Co-authored-by: YuChen <[email protected]>
* Fix keycloak CatalogSource reference issue

Signed-off-by: Daniel Fan <[email protected]>

* implement Keycloak BindInfo via template reference

Signed-off-by: Daniel Fan <[email protected]>

---------

Signed-off-by: Daniel Fan <[email protected]>
Signed-off-by: Piotr Godowski <[email protected]>
Co-authored-by: Piotr Godowski <[email protected]>
for all namespace mode, move services Ns check into reconciliation logic

Signed-off-by: Daniel Fan <[email protected]>
* Bump version to 4.4.0

Signed-off-by: Daniel Fan <[email protected]>

* Add new IM v4.4 entry

Signed-off-by: Daniel Fan <[email protected]>

---------

Signed-off-by: Daniel Fan <[email protected]>
* introduce maintennance mode into cs operator

Signed-off-by: YuChen <[email protected]>

* add annotation and label change predicate

Signed-off-by: YuChen <[email protected]>

---------

Signed-off-by: YuChen <[email protected]>
Bumps golang from 1.21.3-bullseye to 1.21.4-bullseye.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Piotr Godowski <[email protected]>
* Add Keycloak truststore configuration option

* Add the Keycloak startup script to generate truststore

* import also default serviceaccount ca.crt

---------

Co-authored-by: Piotr Godowski <[email protected]>
when spec field does not exist

Signed-off-by: Daniel Fan <[email protected]>
Bumps golang from 1.21.4-bullseye to 1.21.5-bullseye.

---
updated-dependencies:
- dependency-name: golang
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
qpdpQ and others added 15 commits January 31, 2025 23:20
* added operatorVersion for helm chart

Signed-off-by: Henry Li <[email protected]>

* changed helm chart version to 4.11.0

Signed-off-by: Henry Li <[email protected]>

* changed image tag to use chart.appVersion

Signed-off-by: Henry Li <[email protected]>

---------

Signed-off-by: Henry Li <[email protected]>
* added CommonService CR to helm chart

Signed-off-by: Henry Li <[email protected]>

* removed optional namespace-scope configmap to avoid needing restart if NSS operator is not installed first

Signed-off-by: Henry Li <[email protected]>

---------

Signed-off-by: Henry Li <[email protected]>
* updated helm template to work with CICD automation

Signed-off-by: Henry Li <[email protected]>

* removed operatorVersion because it will have no function when charts are built

Signed-off-by: Henry Li <[email protected]>

---------

Signed-off-by: Henry Li <[email protected]>
* comment out values for transition to global values file

* global values transition

* replace values with global values

* update pull secret

* Update global values
* remove cluster rbac

* add cluster roles to cluster rbac file
* add storageclass config option

Signed-off-by: YuChen <[email protected]>

* Update helm/templates/cr.yaml

Co-authored-by: Henry Li <[email protected]>

---------

Signed-off-by: YuChen <[email protected]>
Co-authored-by: Henry Li <[email protected]>
* added new postgres parameter max_slot_wal_keep_size

Signed-off-by: Henry Li <[email protected]>

* quote value since EDB docs has value quoted

Signed-off-by: Henry Li <[email protected]>

* fixed incorrect value for max_slot_wal_keep_size

Signed-off-by: Henry Li <[email protected]>

---------

Signed-off-by: Henry Li <[email protected]>
* cs-operator refactor for no olm

Signed-off-by: Allen Li <[email protected]>

* add deployment status check

Signed-off-by: Allen Li <[email protected]>

* update status check

Signed-off-by: Allen Li <[email protected]>

* revert unused changes

Signed-off-by: Allen Li <[email protected]>

* add comment

Signed-off-by: Allen Li <[email protected]>

* update returned error message

Signed-off-by: Allen Li <[email protected]>

* typo in check condition

Signed-off-by: Allen Li <[email protected]>

* change forceupdate to true

Signed-off-by: Allen Li <[email protected]>

* add else condition

Signed-off-by: Allen Li <[email protected]>

* add error condition

Signed-off-by: Allen Li <[email protected]>

* lint issue

Signed-off-by: Allen Li <[email protected]>

---------

Signed-off-by: Allen Li <[email protected]>
@bluzarraga
Copy link
Member Author

/retest

Copy link
Member

@bitscuit bitscuit left a comment

Choose a reason for hiding this comment

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

/lgtm

@ibm-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bitscuit, bluzarraga

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [bitscuit,bluzarraga]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@bitscuit
Copy link
Member

/verify-owners

@ibm-ci-bot
Copy link
Contributor

The following users are mentioned in OWNERS file(s) but are untrusted for the following reasons. One way to make the user trusted is to add them as members of the IBM org. You can then trigger verification by writing /verify-owners in a comment.

  • pgodowski
    • User is not a member of the org. User is not a collaborator. Satisfy at least one of these conditions to make the user trusted.
    • OWNERS

@ibm-ci-bot ibm-ci-bot merged commit ebc68a5 into release-future Feb 12, 2025
5 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.