Skip to content

Add a kustomize function to remove namespace#1446

Merged
k8s-ci-robot merged 3 commits into
kubeflow:masterfrom
jlewi:fix_management
Aug 2, 2020
Merged

Add a kustomize function to remove namespace#1446
k8s-ci-robot merged 3 commits into
kubeflow:masterfrom
jlewi:fix_management

Conversation

@jlewi

@jlewi jlewi commented Jul 31, 2020

Copy link
Copy Markdown
Contributor

* This is needed to produce YAMLs that are compatible with ACM

related to

* GoogleCloudPlatform/kubeflow-distribution#27 kustomize function to remove namespace
* GoogleCloudPlatform/kubeflow-distribution#4 instructions for ACM
@k8s-ci-robot

Copy link
Copy Markdown
Contributor

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@kubeflow-bot

Copy link
Copy Markdown
Contributor

This change is Reviewable

@jlewi jlewi marked this pull request as ready for review August 1, 2020 19:07
@jlewi

jlewi commented Aug 1, 2020

Copy link
Copy Markdown
Contributor Author

/assign @Bobgy

@k8s-ci-robot k8s-ci-robot added size/M and removed size/S labels Aug 1, 2020
@Bobgy

Bobgy commented Aug 2, 2020

Copy link
Copy Markdown
Contributor

/lgtm

@jlewi

jlewi commented Aug 2, 2020

Copy link
Copy Markdown
Contributor Author

/approve

@k8s-ci-robot

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: jlewi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@@ -0,0 +1,20 @@
# Define a transform to remove namespace from cluster scoped resources

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need this because of a bug in kustomize (cannot know CRD scope)?
Can we link related issue in comment?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants