Skip to content

Conversation

@grom72
Copy link
Contributor

@grom72 grom72 commented Jun 11, 2025

The HTTPS_PROXY variable set in Jenkins may force any shell script to use a proxy server.
This is not expected, as we want to avoid using a proxy unless it is explicitly needed by the DAOS tests or the build process.

grom72 added 3 commits June 11, 2025 10:36
The HTTPS_PROXY variable set in Jenkins may force any shell script to use a proxy server.
This is not expected, as we want to avoid using a proxy unless it is explicitly needed by
the DAOS tests or the build process.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@grom72 grom72 requested a review from JohnMalmberg June 11, 2025 08:42
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/1/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/1/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/1/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/2/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/2/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/2/display/redirect

Pass HTTPS_PROXY to docekr build command

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/3/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/3/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/3/display/redirect

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/4/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/4/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/4/display/redirect

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/5/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/5/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/5/display/redirect

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/6/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/6/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/6/display/redirect

JohnMalmberg
JohnMalmberg previously approved these changes Jun 11, 2025
Copy link
Collaborator

@JohnMalmberg JohnMalmberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is good enough, but at some time we will want to take a second pass and cleanup a few nits.

Comment on lines 3 to 6
/*
* Copyright (C) 2020-2024 Intel Corporation
* Copyright (C) 2025 Hewlett Packard Enterprise Development LP
*/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The (C) actually should not be in Copyright messages. It has no legal meaning, and we should only have it in places where we have to match a known boilerplate that has it.

I have been removing '(C)' when I update copyright lines in code modules.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

// vars/dockerBuildArgs.groovy
/*
* Copyright (C) 2020-2024 Intel Corporation
* Copyright (C) 2025 Hewlett Packard Enterprise Development LP
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 4 to 5
* Copyright (C) 2020-2024 Intel Corporation
* Copyright (C) 2025 Hewlett Packard Enterprise Development LP
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ditto

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Comment on lines 102 to 105
if (env.DAOS_HTTPS_PROXY) {
script += ' --proxy "' + env.DAOS_HTTPS_PROXY + '"'
}
script += ' --user --upgrade launchable~=1.0'
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The pip program searches a number of locations for a config file name pip.conf or something similar.
When we setup a docker container or a system to use pip, we should be updating that file instead of adding --proxy to the pip command.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@grom72 grom72 requested a review from JohnMalmberg June 11, 2025 14:37
@grom72 grom72 marked this pull request as ready for review June 11, 2025 14:37
@grom72 grom72 added the forced-landing The PR has known failures or intentionally reduced testing, but should still be landed label Jun 11, 2025
@grom72 grom72 requested a review from ryon-jensen June 11, 2025 18:06
grom72 added 2 commits June 11, 2025 20:29
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/9/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/9/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/9/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/10/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/10/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/10/display/redirect

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/11/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/11/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/11/display/redirect

tanabarr
tanabarr previously approved these changes Jun 12, 2025
Copy link

@tanabarr tanabarr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice to create a groovy helper to perform the proxy checks rather than duplicating but otherwise LGTM.

Signed-off-by: Tomasz Gromadzki <tomasz.gromadzki@hpe.com>
@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/12/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/12/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/12/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 9 with master Repo completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/13/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on EL 8 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/13/display/redirect

@daosbuild3
Copy link
Collaborator

Test stage provisionNodes on Leap 15 with slurm completed with status FAILURE. https://jenkins-3.daos.hpc.amslabs.hpecorp.net/job/daos-stack/job/pipeline-lib/job/PR-470/13/display/redirect

@grom72 grom72 merged commit 4df343a into master Jun 13, 2025
4 of 10 checks passed
@grom72 grom72 deleted the grom72/sre-3104 branch June 13, 2025 11:24
@grom72 grom72 removed the forced-landing The PR has known failures or intentionally reduced testing, but should still be landed label Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants