Skip to content

Add debug to helm ci#831

Merged
ignatiusm merged 13 commits into
devfrom
add-debug-to-helm-ci
Apr 28, 2026
Merged

Add debug to helm ci#831
ignatiusm merged 13 commits into
devfrom
add-debug-to-helm-ci

Conversation

@ignatiusm

Copy link
Copy Markdown
Contributor

Test Helm CI workflow script was failing with timeouts: https://github.com/Garvan-Data-Science-Platform/ctrl/actions/runs/25032757160

I added:

  • some flags to give minikube max resources
  • timouts to the helm install and upgrade commands (to lengthen it from 5 min default)
  • some debugging output in the case of failure (as you can't ssh to the runner to inspect pod status after a failed run)

Last run of this passed smoothly: https://github.com/Garvan-Data-Science-Platform/ctrl/actions/runs/25034455431

ignatiusm and others added 13 commits February 25, 2026 12:29
* Update minimatch and rollup

* Remove snyk exceptions

* Bump axios
* Bump tar

* bump vite in admin- and user-client

* bump mdast-util-to-hast

* bump lodash

* bump lodash-es

* bump body-parser

* bump js-yaml

* bump esbuild

* bump jsdiff
bump multer
Note: previous release was 1.2.1, however security alerts blocked merging updated Chart version number.
this PR fixes those security alerts
Merging main into the release branch to pre-resolve the 1.2.1 vs 1.2.2 version bump conflict.
This ensures the automated Promote Pre-release workflow will execute a clean merge.
* Adds audit-logs endpoint

* Appease the linter

* Adds audit-log UI

* Adds serverside pagination and sorting for auditlogs

* Tiny tweak for center align

* Fix capilatisation in admin settings panel

* Do not sort encrypted field

* AuditLogController -> AuditLogsController

* Adds list of backend tests

* Adds seedAuditLogs test helper

* Add AllowedAuditLogSortFields to make DRY

* Add defaultAuditLogsPageSize for DRY config

* Use new DRY config and type

* First draft of backend auditlog tests

* Randomise audit log test seed userId

* Adds audit-log cypress tests

* Fix typos, remove unneeded imports
@ignatiusm ignatiusm requested a review from plouka13 April 28, 2026 05:01
@ignatiusm ignatiusm changed the base branch from main to dev April 28, 2026 05:01

@plouka13 plouka13 left a comment

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.

Love this! Happy to have some debugging info on failure

@ignatiusm ignatiusm merged commit d58508b into dev Apr 28, 2026
14 checks passed
@ignatiusm ignatiusm deleted the add-debug-to-helm-ci branch April 28, 2026 05:08
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.

2 participants