Skip to content
This repository has been archived by the owner on Apr 19, 2021. It is now read-only.

Commit

Permalink
Pull the definition of the 'namespace' variable out
Browse files Browse the repository at this point in the history
This should make it more obvious that this really is what should be used.

Related-to: #4
  • Loading branch information
ankon committed Feb 14, 2019
1 parent 6dda0a4 commit 516294d
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/bootstrap.js
Original file line number Diff line number Diff line change
Expand Up @@ -732,6 +732,7 @@ async function main() {
logger.debug(`Resolved properties: ${JSON.stringify(loadedProperties, null, 2)}`);

const mergedProperties = Object.assign({}, loadedProperties, {env: process.env});
const namespace = mergedProperties.environment;

let prepare;
let processResource;
Expand All @@ -742,7 +743,6 @@ async function main() {
};
} else {
const k8sClient = await k8s(argv.kubeconfig, argv.context, '');
const namespace = mergedProperties.environment;
prepare = ensureNamespace(k8sClient, namespace, {}).then(ns => {
if (argv.authorize) {
return authorizeK8s(argv.kubeconfig, ns.metadata.name, argv.serviceAccount, false, 'collaborne-registry');
Expand Down

0 comments on commit 516294d

Please sign in to comment.