Skip to content

Commit d0f3642

Browse files
committed
feat(workflow): add grouped logging for inventory and peadm config
- Added grouped logging for inventory and peadm config retrieval steps. - Redacted password in inventory file for security. - Displayed peadm_config.json content for debugging. - Added grouped logging for smoke test step.
1 parent 990997b commit d0f3642

File tree

1 file changed

+8
-0
lines changed

1 file changed

+8
-0
lines changed

.github/workflows/test-upgrade-legacy.yaml

+8
Original file line numberDiff line numberDiff line change
@@ -126,18 +126,26 @@ jobs:
126126
- name: Check if compiler is converted
127127
timeout-minutes: 120
128128
run: |
129+
echo ::group::inventory
130+
sed -e 's/password: .*/password: "[redacted]"/' < spec/fixtures/litmus_inventory.yaml || true
131+
echo ::endgroup::
132+
echo ::group::get_peadm_config
129133
primary=$(yq '.groups[].targets[] | select(.vars.role == "primary") | .name' spec/fixtures/litmus_inventory.yaml)
130134
compiler=$(yq '.groups[].targets[] | select(.vars.role == "compiler") | .name' spec/fixtures/litmus_inventory.yaml | head -n 1)
131135
bundle exec bolt task run peadm::get_peadm_config \
132136
-t $primary \
133137
--modulepath spec/fixtures/modules \
134138
--no-host-key-check \
135139
--format json > peadm_config.json
140+
cat peadm_config.json
141+
echo ::endgroup::
142+
echo ::group::smoke_test
136143
legacy_compiler=$(yq '.items[0].value.params.legacy_compilers[0]' peadm_config.json)
137144
if [ "$compiler" != "$legacy_compiler" ]; then
138145
echo "Compiler conversion failed"
139146
exit 1
140147
fi
148+
echo ::endgroup::
141149
- name: Tear down test cluster
142150
if: ${{ always() }}
143151
continue-on-error: true

0 commit comments

Comments
 (0)