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

JupyterHub Integration into Unity Marketplace: Refactor IAC code for marketplace deployments #490

Open
3 tasks
Tracked by #90
galenatjpl opened this issue Oct 1, 2024 · 0 comments
Open
3 tasks
Tracked by #90
Assignees
Labels

Comments

@galenatjpl
Copy link
Collaborator

galenatjpl commented Oct 1, 2024

After understanding and testing what exists (#489), you should now make changes that enable a seamless Unity Marketplace integration.

In the context of a Unity Marketplace deployment, the Management Console (MC) will execute the terraform code that the marketplace JSON points to (in this case, it will be under: https://github.com/unity-sds/unity-ads-deployment/tree/main/dev_env) . When running terraform apply, the MC will inject some variables automatically into the terraform, such as project and venue (among others). The ADS deployment IAC should be refactored to ensure that it leverages these variables to correctly namespace resources to the project/venue under which it is being deployed.

See: https://unity-sds.gitbook.io/docs/developer-docs/common-services/docs/users-guide/deployment/unity-marketplace
for more details about what variables are expected and get injected.

NOTE: this ticket is NOT actually getting the deployment into the marketplace, but just getting it ready for marketplace integration.

For example, there should be entries in a variables.tf (or similar):

Image

And resources names/IDs should be dynamically leveraging these values. For example:
Image

You should hopefully make changes to the ADS terraform in such a way that it still supports manual deployments outside of the MC, as well as deployments in the MC. For this you may need to slightly modify the deployment instructions and/or rely on a tfvars file.

Acceptance Criteria:

  • The terraform is refactored to support MC-injected variables
  • The terraform is still deployable/runnable with and without the MC/Marketplace
  • Any necessary procedure documentation is updated.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

2 participants