SRE-3984 Build: Fixups for OpenSUSE - #18944
Conversation
6d84ece to
feb13a1
Compare
|
Errors are Component should be lower-case,Unable to load ticket data |
feb13a1 to
d42b353
Compare
Test if the references to the experimental lua repo can be removed. FIX use and scope of proxy environment varialbles for SUSE. - Jenkinsfile: Add helper script until pipeline-lib fixed. - ci/provisioning/post_provision_config_nodes_LEAP.sh: Remove enabling lua repo. - utils/docker/Dockerfile.leap.15: - utils/scripts/install-leap15.sh: Move setup of repos needed to setup LUA into the install-leap15.sh Correct proxy environment variables. Fix rubygem install to actually use the artifact server, as a dnf install renamed and replaced the /etc/gemrc file. - utils/scripts/helpers/repo-helper-leap15.sh: Do not need -k and --no-proxy options for most of the module. Use DAOS_HTTPS_PROXY if no local repo. Get rid of some log noise. - ftest.sh: - src/tests/ftest/scripts/main.sh: Validate that DAOS_HTTPS_PROXY is not really needed. Signed-off-by: John E. Malmberg <john.malmberg@hpe.com>
d42b353 to
094845f
Compare
| # HTTPS_PROXY is passed in directly by ftest.sh. A high-level script like this | ||
| # should not be the one assigning HTTPS_PROXY=$DAOS_HTTPS_PROXY; revisit after | ||
| # smoke testing and document proper CI/lab proxy usage. | ||
| # if [ -n "$DAOS_HTTPS_PROXY" ]; then | ||
| # # shellcheck disable=SC2154 | ||
| # export HTTPS_PROXY="${DAOS_HTTPS_PROXY:-""}" | ||
| # fi |
There was a problem hiding this comment.
We should also run the dfuse build tests since they rely on the proxy env. Unfortunately one of them is currently failing, but the others should be working
Features: build
Allow-unstable-test: true
There was a problem hiding this comment.
I will add that to the next smoke run, thanks.
| # TEMP DISABLED escape hatch: uncomment to force DAOS_HTTPS_PROXY through as a quick fix. | ||
| # DAOS_HTTPS_PROXY=\"$_DAOS_HTTPS_PROXY\" |
There was a problem hiding this comment.
Shouldn't we remove this if it is not needed?
There was a problem hiding this comment.
I am still smoke testing this, you apparently got added automatically to the PR as a codeowner.
I am expecting a few passes at this which I will be doing forced pushes before I have something ready for an official review.
| // >>> TEMP PATCH: shift default functional testing from EL 9 to Leap 15/SLES 15 <<< | ||
| // >>> REVERT these 3 defaultValue lines back to EL9=true, Leap15=false, SLES15=false <<< |
There was a problem hiding this comment.
Why not use commit pragmas?
There was a problem hiding this comment.
Commit pragmas take too long to lookup and figure out the right combination to work, and if you for get to copy them to a new commit, a lot of test time get wasted.
Preferred solution to commit pragmas is an optional file that if present is processed by commit pragmas, but github actions blocks from landing if it is present. Has to be removed by PR user once testing is done.
JohnMalmberg
changed the title
|
Test stage Functional on Leap 15 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-18944/5/testReport/ |
|
Test stage Functional on SLES 15 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-18944/5/testReport/ |
3 participants
Test if the references to the experimental lua repo can be removed. FIX use and scope of proxy environment varialbles for SUSE.
Jenkinsfile: Add helper script until pipeline-lib fixed.
ci/provisioning/post_provision_config_nodes_LEAP.sh: Remove enabling lua repo.
utils/docker/Dockerfile.leap.15:
utils/scripts/install-leap15.sh: Move setup of repos needed to setup LUA into the install-leap15.sh Correct proxy environment variables.
utils/scripts/helpers/repo-helper-leap15.sh: Do not need -k and --no-proxy options for most of the module. Use DAOS_HTTPS_PROXY if no local repo. Get rid of some log noise.
Steps for the author:
After all prior steps are complete: