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.

Sign up for GitHub

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

Jump to bottom

Re-examine if we need the Docker Mixin #2771

Closed
tamirkamara opened this issue Oct 23, 2022 · 0 comments · Fixed by #2889
Closed

Re-examine if we need the Docker Mixin #2771

tamirkamara opened this issue Oct 23, 2022 · 0 comments · Fixed by #2889
Assignees
@tamirkamara

Comments

@tamirkamara
Copy link
Collaborator

The Docker Mixin (porter) is only installed in the Innereye bundle, but doesn't seem to be used.
If we indeed don't need it we should remove it from:

  1. porter.yaml of the Innereye bundle
  2. devcontainer
  3. Resource Processor image
@tamirkamara tamirkamara added story Stories are the smallest unit of work to be done for a project. and removed story Stories are the smallest unit of work to be done for a project. labels Oct 23, 2022
@tamirkamara tamirkamara self-assigned this Nov 17, 2022
@tamirkamara tamirkamara moved this from Up Next to In Progress in Azure TRE - Engineering Nov 17, 2022
@tamirkamara tamirkamara mentioned this issue Nov 17, 2022
Merged
@tamirkamara tamirkamara moved this from In Progress to PR in Azure TRE - Engineering Nov 17, 2022
Repository owner moved this from PR to Done in Azure TRE - Engineering Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tamirkamara tamirkamara

Labels
None yet
Projects
Azure TRE - Engineering
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove docker mixin microsoft/AzureTRE
1 participant
@tamirkamara