Skip to content

feat(understack-workflows): Redacts sensitive information in workflow output - #2487

Merged
nicholaskuechler merged 1 commit into
mainfrom
PUC-2098-20261014
Oct 14, 2026
Merged

nicholaskuechler merged 1 commit into
mainfrom
PUC-2098-20261014

Conversation

@nicholaskuechler

Copy link
Copy Markdown
Collaborator

What does this change do?

Small patch to redact sensitive information in workflow output such as passwords and tokens.

Upgrade impact

None.

@github-actions

Copy link
Copy Markdown

Python test results

497 tests  +434   497 ✅ +434   3s ⏱️ -5s
  1 suites ±  0     0 💤 ±  0 
  1 files   ±  0     0 ❌ ±  0 

Results for commit b13dda7. ± Comparison against base commit 97535a0.

This pull request removes 63 and adds 497 tests. Note that renamed tests count towards both.
tests.test_argo_client.TestArgoClient ‑ test_generate_workflow_name
tests.test_argo_client.TestArgoClient ‑ test_init_with_ssl_verify
tests.test_argo_client.TestArgoClient ‑ test_init_with_token
tests.test_argo_client.TestArgoClient ‑ test_init_without_token
tests.test_argo_client.TestArgoClient ‑ test_kubernetes_token_property
tests.test_argo_client.TestArgoClient ‑ test_run_playbook_creation_failure
tests.test_argo_client.TestArgoClient ‑ test_run_playbook_success
tests.test_argo_client.TestArgoClient ‑ test_run_playbook_with_empty_extra_vars
tests.test_argo_client.TestArgoClient ‑ test_url_stripping
tests.test_argo_client.TestArgoClient ‑ test_wait_for_completion_api_error
…
tests.test_framework ‑ test_a_deleted_cr_does_not_wedge_the_reconcile
tests.test_framework ‑ test_a_recreate_after_a_delete_in_the_same_batch_is_reconciled
tests.test_framework ‑ test_added_event_reconciles_only_the_changed_resource
tests.test_framework ‑ test_cleanup_policy_boolean_accessors_report_policy_decisions
tests.test_framework ‑ test_collapsed_batch_still_prunes_against_the_whole_snapshot
tests.test_framework ‑ test_cr_without_a_spec_is_unreadable
tests.test_framework ‑ test_crontab_does_not_enable_a_disabled_hook
tests.test_framework ‑ test_deleted_event_batch_does_not_log_zero_changed_crs
tests.test_framework ‑ test_deleted_event_cancels_an_earlier_change_in_the_same_batch
tests.test_framework ‑ test_deleted_event_reconciles_nothing_but_prunes
…

@nicholaskuechler
nicholaskuechler added this pull request to the merge queue Oct 14, 2026
Merged via the queue into main with commit 5ac2db8 Oct 14, 2026
67 checks passed
@nicholaskuechler
nicholaskuechler deleted the PUC-2098-20261014 branch October 14, 2026 18:32
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