Repository navigation

SRE-2257 ci: no proxy option for docker - #16551

Merged

daltonbohning merged 39 commits into

grom72/sre-2257-no-proxy-for-docker

Nov 25, 2025

Merged

SRE-2257 ci: no proxy option for docker#16551
daltonbohning merged 39 commits into
masterfrom
grom72/sre-2257-no-proxy-for-docker

grom72 commented Jun 28, 2025 •

edited

Loading

Contributor

DAOS_NO_PROXY variable is used for the list of hosts/network that should not be accessed via proxy.

The variable is propagated to Dockers build procedure via pipeline-lib #472

Requires:

SRE-2257 ci: Use Artifactory for PyPi #16546

Steps for the author:

Commit message follows the guidelines.
Appropriate Features or Test-tag pragmas were used.
Appropriate Functional Test Stages were run.
At least two positive code reviews including at least one code owner from each category referenced in the PR.
Testing is complete. If necessary, forced-landing label added and a reason added in a comment.

After all prior steps are complete:

Gatekeeper requested (daos-gatekeeper added as a reviewer).

grom72 added 27 commits

June 27, 2025 12:07


          SRE-2257 ci: Use Artifactory for PyPi

4710c0d

Skip-func-test: true
Skip-test: true
Skip-unit-tests: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Add build on el 9 test

bd25cd6

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Add leap 15 build with gcc

c111390

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Switch to rockylinux

964cca5

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Add Leap 15.6 build

2130fe5

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Move PyPi to repo-helper

56c82ab

Cancel-prev-build: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Fix - use '--proxy=""' to avoid proxy usage

67154c3

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Test with fake proxy

ae25855

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Test with empty proxy

5b109eb

Cancel-prev-build: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Too many stages for Jenkinse to process

8482efa

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Test without empty proxy for el9

ab401b0

Cancel-prev-build: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Test with fake proxy on el9

2e935eb

Cancel-prev-build: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Last test with el9

2afb9b7

Cancel-prev-build: false
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Fix no_proxy

6ec3305

Cancel-prev-buil: false

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Fix for empty no_proxy

bba5fe2

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Yet another fix for el9

c32dc06

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Yet another fix for el9 2nd

230b591

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Finall solution ready for integration

eddc7a5

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Finall solution ready for integration (2nd)

289cf31

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          el9 does not accept proxy from pip.conf file

24db345

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          el9 failed build debug

75312f0

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Debugging el9 - proxy="" only in pip.conf

91c9b37

Skip-test: true
Skip-unit-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Debugging el9 - proxy="" only in pip.conf in Dockerfile

638b25a

Skip-test: true
Skip-unit-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


           Debugging el9 - proxy="" only in pip.conf in Dockerfile - fix

c41395a

Skip-test: true
Skip-unit-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Debugging el9 - proxy= only in pip.conf in repo-helper

007866e

Skip-test: true
Skip-unit-tests: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Final solution cleanup 2nd ;)

293b57a

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Final solution cleanup 3rd ;)

d0414a8

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

github-actions Bot commented Jun 28, 2025 •

edited

Loading

Errors are Unable to load ticket data
https://daosio.atlassian.net/browse/SRE-2257

grom72 force-pushed the grom72/sre-2257-no-proxy-for-docker branch from 1ccaccb to be07305 Compare

June 28, 2025 22:23

daosbuild3 commented Jun 28, 2025

Collaborator

Test stage NLT on EL 8.8 completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos/job/PR-16551/7/display/redirect


          DAOS_NO_PROXY instead of NO_PROXY

a2a885c

Skip-test: true
Skip-unit-tests: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

grom72 force-pushed the grom72/sre-2257-no-proxy-for-docker branch from d9ae3b1 to 6c2aa28 Compare

June 29, 2025 13:24


          Final tune

5d9f788

Priority: 2

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

grom72 force-pushed the grom72/sre-2257-no-proxy-for-docker branch from 6c2aa28 to 5d9f788 Compare

June 29, 2025 13:25

daosbuild3 commented Jun 30, 2025

Collaborator

Test stage Functional Hardware Medium Verbs Provider MD on SSD completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net//job/daos-stack/job/daos/view/change-requests/job/PR-16551/17/execution/node/1595/log

grom72 marked this pull request as ready for review

June 30, 2025 18:32

grom72 requested a review from a team as a code owner

June 30, 2025 18:32

grom72 mentioned this pull request

SRE-2257 - introduce NO_PROXY support for Dockers daos-stack/pipeline-lib#472

Merged

grom72 requested review from JohnMalmberg, phender and ryon-jensen

June 30, 2025 18:37

JohnMalmberg previously approved these changes

View reviewed changes

JohnMalmberg left a comment

Contributor

Assuming the the library command is removed before landing.


          Merge remote-tracking branch 'origin/master' into grom72/sre-2257-no-…

51800d4

…proxy-for-docker

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

grom72 dismissed JohnMalmberg’s stale review via

51800d4

July 3, 2025 20:59


          No proxy is implemented via 'no_proxy' variable'

f98391b

There is no need to use unset HTTPS_PROXY as proxy is selectively
disabled for preconfigured list of hosts.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

grom72 changed the title ~~sre 2257 no proxy for docker~~ SRE-2257 ci: no proxy option for docker


          pipeline-lib changes has landed

c572687

Final validation on VMs only.
Skip-func-hw-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

grom72 requested a review from JohnMalmberg

July 17, 2025 12:10

grom72 commented Jul 17, 2025

Contributor Author

Assuming the the library command is removed before landing.

Done: c572687

grom72 added 2 commits

September 17, 2025 08:38


          Merge remote-tracking branch 'origin/master' into grom72/sre-2257-no-…

42f691f

…proxy-for-docker

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>


          Merge remote-tracking branch 'origin/master' into grom72/sre-2257-no-…

7e1051d

…proxy-for-docker

Only limited tests scope to confirm that Dockers are properly used and
provide proper results

Skip-unit-test-memcheck: true

Allow-unstable-test: true

Skip-fault-injection-test: true

Skip-func-hw-test: true

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

daosbuild3 commented Sep 22, 2025

Collaborator

Test stage NLT on EL 8.8 completed with status UNSTABLE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/daos//view/change-requests/job/PR-16551/22/testReport/

osalyk approved these changes

View reviewed changes


          Merge remote-tracking branch 'origin/master' into grom72/sre-2257-no-…

02e6063

…proxy-for-docker

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>

JohnMalmberg approved these changes

View reviewed changes

JohnMalmberg left a comment

Contributor

I am wondering if the differences between repo-helper-el[8,9] and future 10 are big enough to keep them separate instead of using a generic el file?

grom72 requested a review from a team

November 24, 2025 09:59

daltonbohning merged commit 75fb15c into master

60 checks passed

daltonbohning deleted the grom72/sre-2257-no-proxy-for-docker branch

November 25, 2025 17:42

grom72 mentioned this pull request

SRE-3743 ci: force direct access to artifactory (#18048) #18050

Merged

4 of 6 tasks

daltonbohning pushed a commit that referenced this pull request


          SRE-3743 ci: force direct access to artifactory (#18048) (#18050)

b724a85

Use NO_PROXY to bypass an https proxy when accessing local artifactory.

Backport of: #16551 #17522

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Ryon Jensen <ryon.jensen@hpe.com>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet