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

add azure-cli deployment image #10

Merged
merged 7 commits into from
Jul 27, 2022
Merged

Conversation

DeusAvalon
Copy link
Contributor

@DeusAvalon DeusAvalon commented Jul 26, 2022

Contributes to #https://github.com/sidestream-tech/textiltiger/pull/174

Dockerfiles which provides the azure-cli needed for the textiltiger production deployment to an azure kubernetes cluster.

Tried to make it as small as possible but for some reason is the azure-cli for alpine riughly still 1 GB big, this Issue Azure/azure-cli#19591 is still open so they are working on making it smaller.

Checklist:

  • issue number linked above after pound (#)
    • replace "Closes " with "Contributes to" or other if this PR does not close the issue
  • issue checkboxes are all addressed
  • manually checked my feature / not applicable
  • wrote tests / not applicable
  • attached screenshots / not applicable

@DeusAvalon DeusAvalon requested a review from BracketJohn July 26, 2022 18:13
@DeusAvalon DeusAvalon self-assigned this Jul 26, 2022
@@ -0,0 +1,16 @@
FROM segment/chamber:2.10.6 AS chamber
# FROM mcr.microsoft.com/azure-cli:2.9.1 AS azure-cli
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this can go / needs to come back?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes this was just for testing purposes which didnt work out in the end.

make=4.3-r0
RUN pip install --no-cache-dir pip==22.2
RUN pip install --no-cache-dir azure-cli==2.38.0
RUN apk del --purge build
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about an entrypoint cmd command or similar?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

as we have 3 used cli's in this image i just went with a simple /bin/sh as a CMD for atleast a working docker run if needed.

@BracketJohn BracketJohn merged commit d194261 into main Jul 27, 2022
@BracketJohn BracketJohn deleted the add-azure-deployment-image branch July 27, 2022 11:40
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

Successfully merging this pull request may close these issues.

2 participants