-
Notifications
You must be signed in to change notification settings - Fork 17
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
Enhancement: add support for /api/startup-status #513
Comments
@ecanault what is the purpose of allowing Terraform to access this API endpoint? It's a one time use utility in the majority of cases and does not control any resources. |
Hi @thejoeker12, Getting informations regarding this endpoint, especially the Once this setup has been done, this key could then be used as a condition in Terraform to build the rest of an instance (settings, management items, ...). I agree that it can be an overkill load of work for possibly a one time usage for those 2 endpoints, but my goal is to be able to build (and maintain) multiple Jamf Pro instances based on a template with the maximum of automations. For the moment, assuming that all SDK capabilities could be integrated in the provider in a near future, the only manual actions I see are: ADE / VPP / APNs / IdP integrations that are linked to external services and FILEVAULT2COMM certificate that can not be downloaded via API (you can vote my feature request here for that 😉 https://ideas.jamf.com/ideas/JPRO-I-797). Regards, |
hi @ecanault , i don't think this should live within the provider as there is no resource to manage to speak of. i think for your usecase within your pipeline you need a script to run to monitor the init of a jamfpro server and upon that completion, carry out any other pre flight checks before running your tf. i put this together - https://github.com/deploymenttheory/go-api-sdk-jamfpro/blob/main/recipes/jamf_pro_initialization_server/InitializeAndcheckStatus.go to give you an idea. thanks |
Hi @ShocOne I totally agree that implementing those 2 endpoints ( That being said, and on the other hand, why not? In my opinion, if a product can be managed with Terraform, it has to be complete regardless the fact that some datas or resources may or may not be used the way you may want or think. Let people build new ideas 😉! I think also that it is an advantage to be able to completely build a Jamf Pro management with the same tool (Terraform in our case): this will avoid multiple languages, and therefore errors, and will permit to centralize in the same place (Terraform states files) the whole status of an instance. If we agree that the 2 endpoints come together as they are related to similar datas/resources steps, let me give you another example for the Thanks for your reading, and hope I was able to convince you! |
I'm still with the opinion that you should have a preflight script run within your pipeline that checks for a jamf pro server initialisation and upon a successful check then allow the terraform step in the pipeline to occur. Could you describe all of the steps you perform (manual / automated) when you stand up a new jamf pro instance ? The context might give me some ideas 💡. Thanks Dafydd |
Hi @ShocOne, Here is the actual workflow:
The main goal is to avoid the mix of "tools" (preflight script first and terraform after) if, for example, we want to run/maintain all that stuff outside GitLab and inside Terraform Cloud. Obviously, being also able to perform the 2nd "By script" section with Terraform will be mandatory for that. Finally, having Emmanuel |
Hi,
Following the implementation of
/api/startup-status
in the SDK (deploymenttheory/go-api-sdk-jamfpro#603), is it possible to add support in the terraform provider?Thanks in advance,
Emmanuel
The text was updated successfully, but these errors were encountered: