Skip to content
This repository has been archived by the owner on Jul 18, 2023. It is now read-only.

Regression : SAT_HOST is exported before RHEV_SAT_HOST #1495

Open
ntkathole opened this issue May 1, 2019 · 8 comments
Open

Regression : SAT_HOST is exported before RHEV_SAT_HOST #1495

ntkathole opened this issue May 1, 2019 · 8 comments
Assignees
Labels
CI-Regression Regresssion Introduced by some enhancement or new feature added to CI

Comments

@ntkathole
Copy link
Contributor

export SAT_HOST="${RHEV_SAT_HOST}"

Another regression from PR #1442 . We are trying to export SAT_HOST before we have value exported of RHEV_SAT_HOST. This causes value of SAT_HOST to None and later fab tasks gets failed.

See satellite6-upgrader job 1178.

@ntkathole ntkathole added the CI-Regression Regresssion Introduced by some enhancement or new feature added to CI label May 1, 2019
@san7ket
Copy link
Contributor

san7ket commented May 22, 2019

Is this still Valid

@ntkathole
Copy link
Contributor Author

Yes it is valid

@san7ket
Copy link
Contributor

san7ket commented May 23, 2019

Is @jyejare going to work on this ?

@jyejare
Copy link
Member

jyejare commented May 24, 2019

Will fix it today , Thanks guys for bringing it up

@jyejare
Copy link
Member

jyejare commented May 24, 2019

I am really amazed to see that while executing, it sourced compute resources conf file but not the upgrade conf file below that:

source config/sat6_upgrade.conf

@jyejare
Copy link
Member

jyejare commented May 24, 2019

I dont know how, but somehow:
In Preupgrade - setupPrerequisites: source config/sat6_upgrade.conf isnt happening / delaying.
and
InUpgrade - setupPrerequisites: source config/compute_resources.conf isnt happening / delaying.

Jenkins issue maybe. We can try adding sleep for some seconds.

@san7ket
Copy link
Contributor

san7ket commented May 28, 2019

Don't think it's a issue with jenkins since it doesn't happen any where else

@jyejare
Copy link
Member

jyejare commented May 28, 2019

I tried sourcing through the Jenkins system later, it sourced as expected.

Dont know what's wrong with it while running job.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI-Regression Regresssion Introduced by some enhancement or new feature added to CI
Projects
None yet
Development

No branches or pull requests

3 participants