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

Unrestricted & Airlock Import Review workspace versions #3941

Closed
jonnyry opened this issue May 20, 2024 · 4 comments
Closed

Unrestricted & Airlock Import Review workspace versions #3941

jonnyry opened this issue May 20, 2024 · 4 comments

Comments

@jonnyry
Copy link
Collaborator

jonnyry commented May 20, 2024

The Unrestricted & Airlock Import Review workspaces reuse the terraform of the base workspace (so as not to unnecessarily duplicate the workspace terraform code)...

However it looks like the versions they are pulling are old, or at least out of sync with each other:

Unrestricted workspace
https://github.com/microsoft/AzureTRE/blob/main/templates/workspaces/unrestricted/Dockerfile.tmpl
TRE version = 0.14.0

Airlock Import Review
https://github.com/microsoft/AzureTRE/blob/main/templates/workspaces/airlock-import-review/Dockerfile.tmpl
TRE version = 0.15.2

Additional question - is it necessary to pull a versioned copy of the code from GitHub (and therefore risk being out of sync with the local copy)... rather than just copying the terraform locally from the ../base/terraform directory?

image

@jonnyry
Copy link
Collaborator Author

jonnyry commented May 20, 2024

Related:

#3658
#3617

@marrobi
Copy link
Member

marrobi commented May 20, 2024

@jonnyry the reason it is pinned rather than using the local version we needed to run a script before build as the files are outside of the docker context. Also if you update base, it might break the others, and then have to update them in the same PR. This was getting messy.

We should look at porter dependencies maybe: https://porter.sh/docs/development/authoring-a-bundle/working-with-dependencies/# There was an issue with dynamic parameters, but this this now works.

As per #3658 could do with spiking some alternatives and deciding on a way forward.

@jonnyry
Copy link
Collaborator Author

jonnyry commented May 20, 2024

Ok good to know - we have defined our own workspace so will bear that in mind.

@jonnyry
Copy link
Collaborator Author

jonnyry commented May 22, 2024

Closing as duplicate

@jonnyry jonnyry closed this as completed May 22, 2024
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

No branches or pull requests

2 participants