Skip to content
This repository has been archived by the owner on Oct 12, 2018. It is now read-only.

*: handle manifests with multiple resources #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ericchiang
Copy link
Owner

@ericchiang ericchiang commented Sep 19, 2018

Closes #4

@idcmp
Copy link

idcmp commented Oct 4, 2018

Is there a blocker with merging this PR?

@tustvold
Copy link

tustvold commented Oct 10, 2018

I'm currently working on an MR that makes you specify the name and namespace in the terraform code, and overrides the contents of the yml file to match. This is so that it can enforce ForceNew and prevent leaking the old resource. I don't know how people feel about this, it may be outside the scope of what this provider wants to do, but it seems problematic that as it stands this provider can leak resources.

It would be complex unify these two MRs...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants