Merge lp:~thedac/ubuntu-openstack-ci/lp1728680 into lp:ubuntu-openstack-ci

Proposed by David Ames
Status: Rejected
Rejected by: David Ames
Proposed branch: lp:~thedac/ubuntu-openstack-ci/lp1728680
Merge into: lp:ubuntu-openstack-ci
Diff against target: 11 lines (+1/-1)
1 file modified
run/job-parts/osci_job_common.sh (+1/-1)
To merge this branch: bzr merge lp:~thedac/ubuntu-openstack-ci/lp1728680
Reviewer Review Type Date Requested Status
Ryan Beisner (community) Disapprove
Review via email: mp+333001@code.launchpad.net

Description of the change

Allow passing of operating system USER env var

Closes-Bug: #1728680

To post a comment you must log in.
Revision history for this message
Ryan Beisner (1chb1n) wrote :

This code is debug/informational only, and has no effect on builds. It just dumps env vars to a file for inspection if necessary.

review: Disapprove

Unmerged revisions

393. By David Ames

Allow passing of operating system USER env var

Closes-Bug: #1728680

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'run/job-parts/osci_job_common.sh'
2--- run/job-parts/osci_job_common.sh 2017-04-26 22:43:21 +0000
3+++ run/job-parts/osci_job_common.sh 2017-10-30 18:42:01 +0000
4@@ -81,7 +81,7 @@
5
6 function f_env_no_pass() {
7 # Dump environment variables, except for password vars
8- env | egrep -v "PASSWORD|password|COOKIE|cookie|OS_|USER|user|PROXY|proxy" | \
9+ env | egrep -v "PASSWORD|password|COOKIE|cookie|OS_|PROXY|proxy" | \
10 sort 2>&1 | tee $WORKSPACE/env.txt
11 }
12

Subscribers

People subscribed via source and target branches