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

Adapt to latest changes at Portainer & GitHub #9

Open
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

efrecon
Copy link
Contributor

@efrecon efrecon commented Sep 8, 2023

This provides a few improvements to adapt to changes at dependencies of this repo:

  • kompose has fallen out of the Portainer image, the Dockefile uses wildcards when copying the binaries to cope with cases where kompose is present, and cases where it is not (as it will not be present in recent versions of the image).
  • Versions of actions used in the workflow have been bumped up to avoid running against deprecated features/deprecated binaries (node).
  • Bumped up the underlying Alpine implementation to benefit from all security improvements, but also its ability to resolve all DNS queries (TCP).

The workflows have been tweak so they can run within a fork and be properly tested. This has no implications on the original project.

Use wildcards for most of the binaries that are copied to avoid errors
when building the latest images (kompose has been removed). Also adding
helm to the list of binaries that should be added.
Use the value from the github context so the workflow can build in forks
This is to avoid warnings and running on deprecated versions of node,
and avoid deprecated GHA features
@efrecon
Copy link
Contributor Author

efrecon commented Sep 8, 2023

And ... the workflows tweaks had unexpected implications... Let me fix that. I have converted this to a draft.

@efrecon efrecon marked this pull request as draft September 8, 2023 07:38
This because the GitHub repository might contain uppercase letter,
while Docker image names need to contain lowercase letters
@efrecon efrecon marked this pull request as ready for review September 8, 2023 07:55
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.

1 participant