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

feat(provider/kubernetes): namespace deployManifest #7016

Merged

Conversation

ethanfrogers
Copy link
Contributor

@ethanfrogers ethanfrogers commented May 16, 2019

adds fields for namespaceOverride to allow users to define the namespace
their manifests are deployed into (particularly for manifests baked from
Helm charts).

Default - no override

image

Namespace override enabled

image

Override enabled w/ SpEL expression

image

adds fields for namespaceOverride to allow users to define the namespace
their manifests are deployed into (particularly for manifests baked from
Helm charts).
Copy link
Contributor

@maggieneterval maggieneterval left a comment

Choose a reason for hiding this comment

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

😸

@@ -19,7 +19,7 @@ export function ManifestBasicSettings({
selectedAccount,
}: IManifestBasicSettingsProps) {
return (
<div className="container-fluid form-horizontal">
Copy link
Contributor

Choose a reason for hiding this comment

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

was this class just not doing anything?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It was causing some alignment issues if things were added underneath it in other forms.

@ethanfrogers ethanfrogers merged commit 4f4c32b into spinnaker:master May 17, 2019
@ethanfrogers ethanfrogers deleted the deploy-manifest-namespace branch May 17, 2019 12:24
maggieneterval added a commit to maggieneterval/deck that referenced this pull request Jun 3, 2019
89e4e78 fix(runJob/kubernetes): reliably display logs (spinnaker#7060)
f0287a1 fix(runJob/kubernetes): use explicit pod name (spinnaker#7039)
3fd183e fix(kubernetes): fix runjob stage init (spinnaker#7029)
4f4c32b feat(provider/kubernetes): namespace deployManifest (spinnaker#7016)
8e97d59 refactor(kubernetes): namespace selector component (spinnaker#7012)
53cb422 refactor(kubernetes): convert deploy manifest stage to react (spinnaker#7002)
0194821 feat(runjob): capture output ui (spinnaker#6978)
maggieneterval added a commit that referenced this pull request Jun 3, 2019
89e4e78 fix(runJob/kubernetes): reliably display logs (#7060)
f0287a1 fix(runJob/kubernetes): use explicit pod name (#7039)
3fd183e fix(kubernetes): fix runjob stage init (#7029)
4f4c32b feat(provider/kubernetes): namespace deployManifest (#7016)
8e97d59 refactor(kubernetes): namespace selector component (#7012)
53cb422 refactor(kubernetes): convert deploy manifest stage to react (#7002)
0194821 feat(runjob): capture output ui (#6978)
Mahito added a commit to Mahito/spinnaker that referenced this pull request Oct 25, 2019
Adds fields for namespaceOverride to deployManifest.

Ref. feat(provider/kubernetes): namespace deployManifest
spinnaker/deck#7016
Mahito added a commit to Mahito/spinnaker that referenced this pull request Nov 12, 2019
Adds fields for namespaceOverride to deployManifest.

Ref. feat(provider/kubernetes): namespace deployManifest
spinnaker/deck#7016
louisjimenez pushed a commit to spinnaker/spinnaker that referenced this pull request Nov 12, 2019
)

Adds fields for namespaceOverride to deployManifest.

Ref. feat(provider/kubernetes): namespace deployManifest
spinnaker/deck#7016
ttomsu pushed a commit to ttomsu/sponnet that referenced this pull request Feb 5, 2020
…065)

Adds fields for namespaceOverride to deployManifest.

Ref. feat(provider/kubernetes): namespace deployManifest
spinnaker/deck#7016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants