Skip to content

CH-230: Update keycloak, nfs server and other npm / python deps to newer #844

Merged
filippomc merged 11 commits intodevelopfrom
feature/CH-230
Apr 23, 2026
Merged

CH-230: Update keycloak, nfs server and other npm / python deps to newer #844
filippomc merged 11 commits intodevelopfrom
feature/CH-230

Conversation

@alxbrd
Copy link
Copy Markdown
Contributor

@alxbrd alxbrd commented Apr 8, 2026

Closes CH-230

Implemented solution

Updated dependencies to newer versions.

How to test this PR

...

Sanity checks:

  • The pull request is explicitly linked to the relevant issue(s)
  • The issue is well described: clearly states the problem and the general proposed solution(s)
  • In this PR it is explicitly stated how to test the current change
  • The labels in the issue set the scope and the type of issue (bug, feature, etc.)
  • The relevant components are indicated in the issue (if any)
  • All the automated test checks are passing
  • All the linked issues are included in one Sprint
  • All the linked issues are in the Review state
  • All the linked issues are assigned

Breaking changes (select one):

  • The present changes do not change the preexisting api in any way
  • This PR and the issue are tagged as a breaking-change and the migration procedure is well described above

Possible deployment updates issues (select one):

  • There is no reason why deployments based on CloudHarness may break after the current update
  • This PR and the issue are tagged as alert:deployment

Test coverage (select one):

  • Tests for the relevant cases are included in this pr
  • The changes included in this pr are out of the current test coverage scope

Documentation (select one):

  • The documentation has been updated to match the current changes
  • The changes included in this PR are out of the current documentation scope

Nice to have (if relevant):

  • Screenshots of the changes
  • Explanatory video/animated gif

@alxbrd alxbrd marked this pull request as draft April 8, 2026 14:40
@alxbrd alxbrd marked this pull request as ready for review April 22, 2026 09:58
@alxbrd alxbrd requested review from Copilot and filippomc April 22, 2026 09:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates various runtime/build dependencies across the CloudHarness stack (Keycloak, NFS provisioner, frontend tooling, Python deps) and adjusts deployment/configuration files to match the newer components.

Changes:

  • Bumped multiple npm/python dependencies (e2e test deps, Samples frontend tooling, Samples backend Flask/Werkzeug/Connexion pins, Notifications Jinja2).
  • Updated NFS server build inputs (Go module + Dockerfile) and enabled NFS usage for Samples volume in default values.
  • Adjusted CI/deployment YAMLs and Helm templates (Codefresh pipeline tweaks, registry secret guard, Kafka data-dir migration, Elasticsearch image tag, Argo controller port list).

Reviewed changes

Copilot reviewed 15 out of 21 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
test/test-e2e/package-lock.json Updates e2e test node dependency resolutions (ts-jest/@types/jest/etc.).
deployment/codefresh-test.yaml Reformats and tweaks the Codefresh pipeline; adds clone of cloud-harness repo for install.
deployment-configuration/helm/templates/auto-deployments.yaml Tightens registry secret condition to require registry.secret.name.
applications/samples/frontend/package.json Bumps frontend toolchain versions (Vite, ESLint, plugin-react, etc.).
applications/samples/deploy/values.yaml Enables NFS-backed volume by default for Samples (usenfs: true).
applications/samples/deploy/values-test.yaml Adds nfsserver to Samples test dependencies.
applications/samples/backend/requirements.txt Simplifies/updates Flask/Werkzeug/Connexion version constraints.
applications/notifications/server/requirements.txt Updates Jinja2 version.
applications/nfsserver/nfs-subdir-external-provisioner/go.mod Updates Go version and dependency graph for the provisioner.
applications/nfsserver/Dockerfile Updates Go builder image and improves yum install layer; adds CMD.
applications/events/deploy/templates/deployments.yml Adds initContainer to migrate Kafka data directory; updates log dir and mount path.
applications/elasticsearch/deploy/charts/values.yaml Updates Elasticsearch image tag.
applications/argo/deploy/charts/templates/controller/workflow-controller-deployment.yaml Removes an extra container port entry.
applications/accounts/admin-event-listener/pom.xml Updates Kafka client and compression lib dependency management via properties.
applications/accounts/admin-event-listener/jar-module/pom.xml Relies on parent dependencyManagement for Kafka/compression deps (no explicit versions).
applications/accounts/Dockerfile Updates Keycloak base image tag.
Files not reviewed (1)
  • test/test-e2e/package-lock.json: Language not supported

Comment thread applications/samples/deploy/values.yaml Outdated
Comment thread applications/samples/frontend/package.json
Comment thread applications/nfsserver/nfs-subdir-external-provisioner/go.mod Outdated
Comment thread applications/nfsserver/nfs-subdir-external-provisioner/go.mod Outdated
alxbrd and others added 4 commits April 23, 2026 13:26
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@filippomc filippomc merged commit f4dad0b into develop Apr 23, 2026
10 of 11 checks 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