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

Unknown extensible attribute: Cloud API Owned #79

Closed
dvob opened this issue Jun 20, 2019 · 7 comments · Fixed by #107
Closed

Unknown extensible attribute: Cloud API Owned #79

dvob opened this issue Jun 20, 2019 · 7 comments · Fixed by #107

Comments

@dvob
Copy link
Contributor

dvob commented Jun 20, 2019

I would like to use the AllocateIP function on a on-prem infoblox environment with WAPI Version 2.3.1 but i get the error: Unknown extensible attribute: Cloud API Owned.
It would be nice if the object_manager functions could be used on on-prem instances as well.

@saiprasannasastry
Copy link
Contributor

Hi, can you please address the comments on the PR

@dhawal55
Copy link
Contributor

Why are these attributes hard-coded in code?

func (objMgr *ObjectManager) getBasicEA(cloudAPIOwned Bool) EA {
ea := make(EA)
ea["Cloud API Owned"] = cloudAPIOwned
ea["CMP Type"] = objMgr.cmpType
ea["Tenant ID"] = objMgr.tenantID
return ea
}

The go-client should allow users to set the required attributes instead of the ones in the code above

@somashekhar
Copy link
Contributor

These are the default EAs we get once CNA license is being installed and used. So we have made sure to get them in this method, so that subsequent methods can use them.

@dhawal55
Copy link
Contributor

Thank you @somashekhar. Is the CNA license required in order to use this library?

@somashekhar
Copy link
Contributor

If you don't have CNA then you should add these EAs manually and get library working.

@dhawal55
Copy link
Contributor

dhawal55 commented Oct 22, 2020 via email

@somashekhar
Copy link
Contributor

somashekhar commented Oct 22, 2020

Agree. That would be a better solution. Do make sure nothing breaks and also add respective UTs.
@AvRajath can you once confirm this proposal does not break anything in our existing Terraform plugin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants