Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stage 3 hangs in integration testing (possibly due to tuned module) #1349

Closed
smithfarm opened this issue Sep 5, 2018 · 3 comments
Closed
Assignees
Labels

Comments

@smithfarm
Copy link
Contributor

Description of Issue/Question

For no apparent reason, Stage 3 hangs (i.e. enters an endless loop) transiently during integration tests. Examination of the minion logs shows the last thing that happened before the hang was related to tuned.

Upon further examination and discussion, it was concluded that the tuned Salt module is known to be buggy and it's possible that Salt is hanging (looping endlessly) in that module.

To address this, I propose to implement a --tuned option to health-ok.sh which will default to "off", meaning that, by default, the tuned orchestration will be disabled before running Stage 3. That way, once we are ready to investigate this further, we can implement a --tuned=on test case.

@smithfarm smithfarm added the QA label Sep 5, 2018
@smithfarm smithfarm self-assigned this Sep 5, 2018
@smithfarm
Copy link
Contributor Author

@mogeb @jschmid1 @rjfd

@smithfarm smithfarm changed the title Stage 3 hangs in integration testing Stage 3 hangs in integration testing (possibly due to tuned module) Sep 5, 2018
smithfarm added a commit that referenced this issue Sep 5, 2018
Fixes: #1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit that referenced this issue Sep 5, 2018
Fixes: #1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@jschmid1
Copy link
Contributor

jschmid1 commented Sep 5, 2018

I think the less intrusive option is to add a --tuned=off/on option and add the version with --tuned=off in tier0 whilst adding tuned=on in tier2.

smithfarm added a commit that referenced this issue Sep 5, 2018
Fixes: #1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
@smithfarm
Copy link
Contributor Author

@jschmid1 tier2 is run by the deepsea nightly CI test and by the Staging CI test. Having --tuned=on in tier2 would mean those CI runs are exposed to the (presumed) bug in Salt's tuned module. Not sure I like that idea.

I was going to propose that we add a "tuned" test case to tier3 (which is not run by any automated CI, only triggered manually).

Once the Salt tuned module is reported to be fixed, then we could turn tuned back on. What do you think?

smithfarm added a commit that referenced this issue Sep 5, 2018
Fixes: #1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
smithfarm added a commit to smithfarm/DeepSea that referenced this issue Sep 5, 2018
Fixes: SUSE#1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit 79d1d04)

Conflicts:
	qa/suites/basic/health-ok.sh
smithfarm added a commit to smithfarm/DeepSea that referenced this issue Sep 6, 2018
Fixes: SUSE#1349
Signed-off-by: Nathan Cutler <ncutler@suse.com>
(cherry picked from commit 79d1d04)

Conflicts:
	qa/suites/basic/health-ok.sh
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants