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

fix: app outdir property cannot be different from CLI or cdk8s.yaml file (#783) #786

Merged
merged 1 commit into from
Oct 4, 2022

Conversation

cdk8s-automation
Copy link
Contributor

Backport

This will backport the following commits from 2.x to 1.x:

Questions ?

Please refer to the Backport tool documentation

… file (#783)

Currently setting the `outdir` property for CDK8s app throws an error,
```
ERROR: synthesis failed, app expected to create "dist"
```

This is due to the CLI not able to recognize this change in the `outdir` in the application property.
The value specified in the `outdir` needs to be similar to what is mentioned either in CLI's `--output` argument or the `output` property in the cdk8s.yaml file.

Signed-off-by: Vinayak Kukreja <[email protected]>

Resolves cdk8s-team/cdk8s-cli#210

(cherry picked from commit 05bf5f4)
Signed-off-by: Vinayak Kukreja <[email protected]>
@mergify mergify bot merged commit f1fbb0a into 1.x Oct 4, 2022
@mergify mergify bot deleted the backport/1.x/pr-783 branch October 4, 2022 08:11
@cdk8s-automation
Copy link
Contributor Author

⚪ Backport skipped

The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.

Manual backport

To create the backport manually run:

backport --pr 786

Questions ?

Please refer to the Backport tool documentation

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-approve Pull requests that should be auto approved
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants