Skip to content

Conversation

@oren-codefresh
Copy link
Contributor

No description provided.

installationEvent: installationProgress.events.RUNTIME_INSTALLED,
});

let appProxyIP;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no need for this

installationPlan.addStep({
name: 'install app-proxy',
func: async () => {
appProxyIP = await installAppProxy({
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do installationPlan.addContext('appProxyIp', appProxyIp);
this will make sure it will be available after restart

await sdk.runtimeEnvs.update({ name: reName }, _.merge(re, body));
console.log(`Runtime environment "${colors.cyan(reName)}" has been updated with the app proxy`);
},
condition: !!appProxy,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

condition: async () => installationPlan.getContext('appProxyIp')

const re = await sdk.runtimeEnvs.get({ name: reName });
const body = {
appProxy: {
externalIP: appProxyIP,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

installationPlan.getContext('appProxyIp')

@roi-codefresh
Copy link
Contributor

bump

@oren-codefresh oren-codefresh merged commit 3d8eba4 into master Sep 16, 2020
@noam-codefresh noam-codefresh deleted the SAAS-8690 branch August 13, 2021 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants