Commit d6b001e
authored
Improve CI reliability and developer productivity through test scheduling optimizations, mirror stability fixes, and a new artifact reuse feature. (#1379)
* Document disk-intensive test placement in greenplum_schedule
Add comment explaining why autovacuum-template0-segment and profile tests
are positioned early in the test schedule. These tests consume significant
disk space through WAL generation, XID consumption, and autovacuum operations.
Running them early when ~20GB disk space is available (vs ~10GB later) helps
avoid disk exhaustion issues during test execution.
* Fix Rocky Linux mirror instability in CI
Add repository metadata refresh and retry logic to handle transient
mirror failures during RPM installation. This addresses frequent 404
errors from Rocky Linux mirrors that cause CI failures.
Changes:
- Run 'dnf clean all' and 'dnf makecache --refresh' before installation
- Add '--setopt=retries=10' to dnf install command
- Apply fix to both rpm-install-test and test jobs
This improves CI reliability without changing functionality.
* Add artifact reuse feature for faster test iteration
Enable reusing build artifacts from previous workflow runs to speed up
test iteration by ~50-70 minutes. This is useful for debugging test
failures without rebuilding.
Changes:
- Add 'reuse_artifacts_from_run_id' workflow input parameter
- Skip build job when reusing artifacts from specified run
- Skip rpm-install-test job when reusing artifacts
- Update artifact download steps to support cross-run downloads
- Add proper job conditionals to handle skipped build job
Usage:
Manually trigger workflow and specify a previous run ID in the
'reuse_artifacts_from_run_id' input field. Leave empty to build fresh.
This maintains backward compatibility - default behavior unchanged.
* Add GitHub Actions workflow documentation for developers
Create comprehensive documentation for GitHub Actions workflows, focusing
on features that help developers iterate faster when debugging CI issues.
Key sections:
- Manual workflow triggers and input parameters
- Artifact reuse feature with step-by-step guide
- Running workflows in forked repositories
- Troubleshooting common issues
This documentation enables developers to:
- Reuse build artifacts to save ~50-70 minutes per test iteration
- Run CI validation in their forks before submitting PRs
- Understand available workflow options and test selections
- Debug test failures more efficiently
* Pin Rocky Linux repos to stable 9.x release
Use --releasever=9 to pin dnf to stable Rocky Linux 9.x repos instead
of bleeding-edge point releases (e.g., 9.6) that may not be fully synced
across all mirrors.
Rocky Linux maintains binary compatibility within major versions, so
pinning to 9 ensures we get stable, widely-mirrored packages while
remaining compatible with the 9.6 container OS.
This complements the earlier retry/refresh logic by addressing the root
cause: new point releases have metadata sync lag across mirror network.
* Move all autovacuum tests to early execution
Move autovacuum and autovacuum-segment tests alongside
autovacuum-template0-segment to run early in the schedule when more
disk space is available.
All three autovacuum tests are disk-intensive and benefit from running
when ~20GB is available rather than later when space may be constrained.
This grouping also improves test organization by keeping related tests
together.
* Clarify secrets configuration in workflow documentation
Update README to clarify that no manual secret configuration is required
for normal development workflows:
- GITHUB_TOKEN is automatically provided by GitHub
- Only used for artifact reuse feature (downloading previous run artifacts)
- DockerHub secrets only needed for custom container image builds
(advanced/maintainer use case)
This removes confusion about required setup steps for fork users.1 parent 9d86458 commit d6b001e
3 files changed
Lines changed: 303 additions & 10 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
| 166 | + | |
| 167 | + | |
| 168 | + | |
| 169 | + | |
| 170 | + | |
| 171 | + | |
| 172 | + | |
| 173 | + | |
| 174 | + | |
| 175 | + | |
| 176 | + | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
| 192 | + | |
| 193 | + | |
| 194 | + | |
| 195 | + | |
| 196 | + | |
| 197 | + | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
| 201 | + | |
| 202 | + | |
| 203 | + | |
| 204 | + | |
| 205 | + | |
| 206 | + | |
| 207 | + | |
| 208 | + | |
| 209 | + | |
| 210 | + | |
| 211 | + | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
| 219 | + | |
| 220 | + | |
| 221 | + | |
| 222 | + | |
| 223 | + | |
| 224 | + | |
| 225 | + | |
| 226 | + | |
| 227 | + | |
| 228 | + | |
| 229 | + | |
| 230 | + | |
| 231 | + | |
| 232 | + | |
| 233 | + | |
| 234 | + | |
| 235 | + | |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
| 252 | + | |
| 253 | + | |
| 254 | + | |
| 255 | + | |
| 256 | + | |
| 257 | + | |
| 258 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
113 | 113 | | |
114 | 114 | | |
115 | 115 | | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
| 119 | + | |
| 120 | + | |
116 | 121 | | |
117 | 122 | | |
118 | 123 | | |
| |||
412 | 417 | | |
413 | 418 | | |
414 | 419 | | |
| 420 | + | |
415 | 421 | | |
416 | 422 | | |
417 | 423 | | |
| |||
687 | 693 | | |
688 | 694 | | |
689 | 695 | | |
| 696 | + | |
| 697 | + | |
| 698 | + | |
| 699 | + | |
690 | 700 | | |
691 | 701 | | |
692 | 702 | | |
| |||
710 | 720 | | |
711 | 721 | | |
712 | 722 | | |
| 723 | + | |
| 724 | + | |
713 | 725 | | |
714 | 726 | | |
715 | 727 | | |
| |||
814 | 826 | | |
815 | 827 | | |
816 | 828 | | |
| 829 | + | |
| 830 | + | |
| 831 | + | |
| 832 | + | |
| 833 | + | |
817 | 834 | | |
818 | 835 | | |
819 | 836 | | |
820 | | - | |
| 837 | + | |
| 838 | + | |
821 | 839 | | |
822 | | - | |
| 840 | + | |
823 | 841 | | |
824 | 842 | | |
825 | 843 | | |
| |||
858 | 876 | | |
859 | 877 | | |
860 | 878 | | |
| 879 | + | |
| 880 | + | |
| 881 | + | |
861 | 882 | | |
862 | 883 | | |
863 | 884 | | |
| |||
1087 | 1108 | | |
1088 | 1109 | | |
1089 | 1110 | | |
| 1111 | + | |
| 1112 | + | |
1090 | 1113 | | |
1091 | 1114 | | |
1092 | 1115 | | |
| |||
1095 | 1118 | | |
1096 | 1119 | | |
1097 | 1120 | | |
| 1121 | + | |
| 1122 | + | |
1098 | 1123 | | |
1099 | 1124 | | |
1100 | 1125 | | |
| |||
1186 | 1211 | | |
1187 | 1212 | | |
1188 | 1213 | | |
| 1214 | + | |
| 1215 | + | |
| 1216 | + | |
| 1217 | + | |
| 1218 | + | |
1189 | 1219 | | |
1190 | 1220 | | |
1191 | 1221 | | |
1192 | | - | |
| 1222 | + | |
| 1223 | + | |
1193 | 1224 | | |
1194 | | - | |
| 1225 | + | |
1195 | 1226 | | |
1196 | 1227 | | |
1197 | 1228 | | |
| |||
0 commit comments