Skip to content

Commit b30996c

Browse files
committed
fix(plan): remove redundant puppet agent run task
- Removed the redundant task to run the puppet agent on all nodes after installation. - Simplified the install_test_cluster plan by eliminating unnecessary steps.
1 parent 0ae01f3 commit b30996c

File tree

2 files changed

+2
-5
lines changed

2 files changed

+2
-5
lines changed

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ jobs:
9797
bundle exec bolt plan run peadm_spec::install_test_cluster \
9898
--inventoryfile spec/fixtures/litmus_inventory.yaml \
9999
--modulepath spec/fixtures/modules \
100-
architecture="large" \
100+
architecture=${{ matrix.architecture }} \
101101
console_password=${{ secrets.CONSOLE_PASSWORD }} \
102102
version=${{ matrix.version }}
103103
- name: Wait as long as the file ${HOME}/pause file is present
@@ -135,7 +135,7 @@ jobs:
135135
--inventoryfile spec/fixtures/litmus_inventory.yaml \
136136
--modulepath spec/fixtures/modules \
137137
--no-host-key-check \
138-
architecture="large" \
138+
architecture=${{ matrix.architecture }} \
139139
version=${{ matrix.to_version }}
140140
- name: Tear down test cluster
141141
if: ${{ always() }}

spec/acceptance/peadm_spec/plans/install_test_cluster.pp

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -79,8 +79,5 @@
7979
$install_result =
8080
run_plan('peadm::install', $arch_params + $common_params)
8181

82-
# Run puppet agent on all nodes
83-
run_task('peadm::puppet_runonce', $t)
84-
8582
return($install_result)
8683
}

0 commit comments

Comments
 (0)