Merge "[CVP,master] cvp-perf pipeline must run container with the same name"
diff --git a/cvp-shaker.groovy b/cvp-shaker.groovy
index 08f9315..1f04bc1 100644
--- a/cvp-shaker.groovy
+++ b/cvp-shaker.groovy
@@ -154,9 +154,11 @@
}
stage('Run Shaker tests') {
- if (! salt_testing.setupDockerAndTest(configRun)) {
- common.warningMsg('Docker contrainer failed to run Shaker')
- currentBuild.result = 'FAILURE'
+ timeout(time: 10, unit: 'HOURS') {
+ if (! salt_testing.setupDockerAndTest(configRun)) {
+ common.warningMsg('Docker contrainer failed to run Shaker')
+ currentBuild.result = 'FAILURE'
+ }
}
}
diff --git a/generate-cookiecutter-products.groovy b/generate-cookiecutter-products.groovy
index d4be6cc..50c38b2 100644
--- a/generate-cookiecutter-products.groovy
+++ b/generate-cookiecutter-products.groovy
@@ -75,34 +75,19 @@
// because each of them, might be 'refs/' variable, we need to add some tricky trigger of using
// 'release/XXX' logic. This is totall guess - so,if even those one failed, to definitely must pass
// correct variable finally!
- [context.get('cookiecutter_template_branch'), context.get('shared_reclass_branch'), context.get('mcp_common_scripts_branch')].any { branch ->
+ [ context.get('cookiecutter_template_branch'), context.get('shared_reclass_branch'), context.get('mcp_common_scripts_branch') ].any { branch ->
if (branch.toString().startsWith('release/')) {
gitGuessedVersion = branch
return true
}
}
- // Use mcpVersion git tag if not specified branch for cookiecutter-templates
- if (!context.get('cookiecutter_template_branch')) {
- context['cookiecutter_template_branch'] = gitGuessedVersion ?: context['mcp_version']
- }
- // Don't have n/t/s for cookiecutter-templates repo, therefore use master
- if (["nightly", "testing", "stable"].contains(context['cookiecutter_template_branch'])) {
- context['cookiecutter_template_branch'] = 'master'
- }
- if (!context.get('shared_reclass_branch')) {
- context['shared_reclass_branch'] = gitGuessedVersion ?: context['mcp_version']
- }
- // Don't have nightly/testing for reclass-system repo, therefore use master
- if (["nightly", "testing", "stable"].contains(context['shared_reclass_branch'])) {
- context['shared_reclass_branch'] = 'master'
- }
- if (!context.get('mcp_common_scripts_branch')) {
- // Pin exactly to CC branch, since it might use 'release/XXX' format
- context['mcp_common_scripts_branch'] = gitGuessedVersion ?: context['mcp_version']
- }
- // Don't have n/t/s for mcp-common-scripts repo, therefore use master
- if (["nightly", "testing", "stable"].contains(context['mcp_common_scripts_branch'])) {
- context['mcp_common_scripts_branch'] = 'master'
+
+ [ 'cookiecutter_template_branch', 'shared_reclass_branch', 'mcp_common_scripts_branch' ].each { repoName ->
+ if (context['mcp_version'] in [ "nightly", "testing", "stable" ]) {
+ context[repoName] = 'master'
+ } else if (! context.get(repoName)) {
+ context[repoName] = gitGuessedVersion ?: "release/${context['mcp_version']}".toString()
+ }
}
//
distribRevision = context['mcp_version']
diff --git a/upgrade-mcp-release.groovy b/upgrade-mcp-release.groovy
index d1614eb..e865a06 100644
--- a/upgrade-mcp-release.groovy
+++ b/upgrade-mcp-release.groovy
@@ -456,7 +456,7 @@
salt.enforceState(venvPepper, "I@salt:minion:ca", 'salt.minion.ca', true)
salt.enforceState(venvPepper, "I@salt:minion", 'salt.minion.cert', true)
// run `salt.minion` to refresh all minion configs (for example _keystone.conf)
- salt.enforceState([saltId: venvPepper, target: "I@salt:minion ${extra_tgt}", state: ['salt.minion'], read_timeout: 60, retries: 2])
+ salt.enforceState([saltId: venvPepper, target: "I@salt:minion", state: ['salt.minion'], read_timeout: 60, retries: 2])
// updating users and keys
salt.enforceState(venvPepper, "I@linux:system", 'linux.system.user', true)
salt.enforceState(venvPepper, "I@linux:system", 'openssh', true)