Skip to content

Commit 4937c22

Browse files
Cr 1539 add http proxy acceptance test to runner init (#586)
1 parent 9c9b329 commit 4937c22

File tree

3 files changed

+68
-17
lines changed

3 files changed

+68
-17
lines changed

lib/interface/cli/commands/hybrid/helper.js

Lines changed: 39 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -417,10 +417,24 @@ async function downloadHybridComponents(location) {
417417
console.log(`Kubernetes registrator installer downloaded successfully ${location}`);
418418
}
419419

420-
async function runClusterAcceptanceTests({ kubeNamespace, kubeContextName, kubeConfigPath }) {
420+
async function runClusterAcceptanceTests({
421+
apiHost,
422+
kubeNamespace,
423+
kubeConfigPath,
424+
kubeContextName,
425+
dockerRegistry,
426+
envVars,
427+
verbose,
428+
valuesFile, // --values
429+
setValue, // --set-value
430+
setFile, // --set-file
431+
}) {
421432
const binLocation = await downloadVeonona();
422433
const componentRunner = new Runner(binLocation);
423434
const cmd = ['test', '--log-formtter', DefaultLogFormatter];
435+
if (apiHost) {
436+
cmd.push(`--api-host=${apiHost}`);
437+
}
424438
if (kubeNamespace) {
425439
cmd.push('--kube-namespace');
426440
cmd.push(kubeNamespace);
@@ -433,6 +447,30 @@ async function runClusterAcceptanceTests({ kubeNamespace, kubeContextName, kubeC
433447
cmd.push('--kube-config-path');
434448
cmd.push(kubeConfigPath);
435449
}
450+
if (process.env.NODE_TLS_REJECT_UNAUTHORIZED === '0') {
451+
cmd.push('--insecure');
452+
}
453+
if (dockerRegistry) {
454+
cmd.push(`--set-value=DockerRegistry=${dockerRegistry}`);
455+
}
456+
if (verbose) {
457+
cmd.push(`--verbose=${verbose}`);
458+
}
459+
if (envVars) {
460+
envVars.forEach((item) => {
461+
const parts = item.split('=');
462+
cmd.push(`--set-value=EnvVars.${parts[0]}=${parts[1]}`);
463+
});
464+
}
465+
if (valuesFile) {
466+
cmd.push(`--values=${valuesFile}`);
467+
}
468+
if (setValue) {
469+
cmd.push(`--set-value=${setValue}`);
470+
}
471+
if (setFile) {
472+
cmd.push(`--set-file=${setFile}`);
473+
}
436474
await componentRunner.run(components.venona, cmd);
437475
}
438476

lib/interface/cli/commands/hybrid/init.cmd.js

Lines changed: 28 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -232,12 +232,12 @@ const initCmd = new Command({
232232
values: valuesFile,
233233
'set-value': setValue,
234234
'set-file': setFile,
235-
'skip-cluster-test': skipClusterTest,
236235
'app-proxy-ingress-class': appProxyIngressClass,
237236
'env-vars': envVars,
238237
} = _argv;
239238
let {
240239
yes: noQuestions,
240+
'skip-cluster-test': skipClusterTest,
241241
'kube-context-name': kubeContextName,
242242
'kube-namespace': kubeNamespace,
243243
'set-default-runtime': shouldMakeDefaultRe,
@@ -262,19 +262,19 @@ const initCmd = new Command({
262262
valuesObj = YAML.parse(valuesFileStr);
263263
noQuestions = true;
264264

265-
if (!kubeNamespace && valuesObj.Namespace) {
265+
if (!kubeNamespace && _.has(valuesObj, 'Namespace')) {
266266
kubeNamespace = valuesObj.Namespace;
267267
}
268-
if (!kubeContextName && valuesObj.Context) {
268+
if (!kubeContextName && _.has(valuesObj, 'Context')) {
269269
kubeContextName = valuesObj.Context;
270270
}
271-
if (!url && valuesObj.CodefreshHost) {
271+
if (!url && _.has(valuesObj, 'CodefreshHost')) {
272272
url = valuesObj.CodefreshHost;
273273
}
274-
if (!token && valuesObj.Token) {
274+
if (!token && _.has(valuesObj, 'Token')) {
275275
token = valuesObj.Token;
276276
}
277-
if (!name && valuesObj.AgentId) {
277+
if (!name && _.has(valuesObj, 'AgentId')) {
278278
name = valuesObj.AgentId;
279279
}
280280
if (typeof _.get(valuesObj, 'Monitor.Enabled') !== 'undefined') {
@@ -301,6 +301,9 @@ const initCmd = new Command({
301301
if (_.has(valuesObj, 'DockerRegistry') && !dockerRegistry) {
302302
dockerRegistry = _.get(valuesObj, 'DockerRegistry');
303303
}
304+
if (_.has(valuesObj, 'SkipClusterTest') && _.isUndefined(skipClusterTest)) {
305+
skipClusterTest = _.get(valuesObj, 'SkipClusterTest');
306+
}
304307
}
305308
if (!url) {
306309
url = DEFAULTS.URL;
@@ -480,15 +483,25 @@ const initCmd = new Command({
480483
installationPlan.addContext('argv', _argv);
481484

482485
// run cluster acceptance tests
483-
if (!_.get(valuesObj, 'SkipClusterTest')) {
484-
installationPlan.addStep({
485-
name: 'run cluster acceptance tests',
486-
func: runClusterAcceptanceTests,
487-
arg: { kubeNamespace, kubeConfigPath, kubeContextName },
488-
installationEvent: installationProgress.events.ACCEPTANCE_TESTS_RAN,
489-
condition: !skipClusterTest,
490-
});
491-
}
486+
installationPlan.addStep({
487+
name: 'run cluster acceptance tests',
488+
func: runClusterAcceptanceTests,
489+
arg: {
490+
apiHost: sdk.config.context.url,
491+
kubeNamespace,
492+
kubeConfigPath,
493+
kubeContextName,
494+
dockerRegistry,
495+
envVars,
496+
verbose,
497+
valuesFile,
498+
setValue,
499+
setFile,
500+
},
501+
installationEvent: installationProgress.events.ACCEPTANCE_TESTS_RAN,
502+
condition: !skipClusterTest,
503+
});
504+
492505
// generate new agent name
493506
installationPlan.addContext('agentName', name ? name.trim() : name);
494507
installationPlan.addStep({

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
22
"name": "codefresh",
3-
"version": "0.73.17",
3+
"version": "0.73.18",
44
"description": "Codefresh command line utility",
55
"main": "index.js",
66
"preferGlobal": true,

0 commit comments

Comments
 (0)