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

Add option to ignore license check #23

Closed
wants to merge 3 commits into from
Closed

Add option to ignore license check #23

wants to merge 3 commits into from

Conversation

jpereira
Copy link

@jpereira jpereira commented Oct 5, 2018

That fix is necessary to use with Infoblox instances running in-house.

@alibasim86
Copy link

Hi Everyone, Can we get this pull request merged pls, beside any test instances issues the same core code can work with infoblox that does not have cloud license installed

@saiprasannasastry
Copy link
Contributor

Hi This has not been merged because they want the cloud licence check to be a integral part of terraform.I will get back on this ASAP

@dvob
Copy link
Contributor

dvob commented Aug 18, 2019

Why is the license check requreid? In my opinion the license check should be removed entirely. The only thing the client (terraform-provider) does is using the WAPI, but the WAPI can also be used without the CNA licenese.

@saiprasannasastry
Copy link
Contributor

saiprasannasastry commented Aug 18, 2019

i've told our PM to look at the comments on goclient as well regarding the same. i will talk to him and get an update

@jpereira
Copy link
Author

Well, as usual. the PM being as a PM.

@saiprasannasastry
Copy link
Contributor

still waiting on his response

@saiprasannasastry
Copy link
Contributor

saiprasannasastry commented Aug 28, 2019

@jpereira Pm told to remove the cloud license check all together. quick question ,The value of LICENSE_NOCHECK passed should be a string?

@jpereira
Copy link
Author

yes

@dvob
Copy link
Contributor

dvob commented Aug 28, 2019

In my opinion it would make more sense to remove the license check entirely, then we don't need the LICENSE_NOCHECK option. See my proposal in #36
Or are there cases when the license check is required?

@saiprasannasastry
Copy link
Contributor

Will remove it.

@saiprasannasastry
Copy link
Contributor

closing this PR as we have decided to remove license check

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.

4 participants