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

Allow configuring enablePlatformTags via plugins #4037

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

eifinger
Copy link

@eifinger eifinger commented Jun 1, 2023

Thank you for your interest in contributing! For general guidelines, please refer to
the contributing guide.

Before filing a pull request, make sure to do the following:

This helps to reduce the chance of having a pull request rejected.

Fixes #3518🛠️

This PR tries to pick up #3710
I added the requested integration test testSteps_forBuildToDockerRegistry_multiplePlatforms

@google-cla
Copy link

google-cla bot commented Jun 1, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@eifinger eifinger changed the title Enable platform tags plugin Allow configuring enablePlatformTags via plugins Jun 1, 2023
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch 3 times, most recently from 06a505d to 06b7b90 Compare June 13, 2023 07:31
@eifinger
Copy link
Author

Is there anything I can do to get the conversation started?

@eifinger eifinger force-pushed the enable_platform_tags_plugin branch from 06b7b90 to a9dbff8 Compare June 14, 2023 09:11
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch 2 times, most recently from f9df0ba to 374a46f Compare July 3, 2023 08:38
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch from 374a46f to 98788fb Compare July 12, 2023 13:33
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch from 98788fb to a5cb0ec Compare July 31, 2023 08:40
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch 2 times, most recently from 4a1d81a to ef66b4f Compare October 20, 2023 08:44
@msvticket
Copy link

Maybe this is something you could look at @mpeddada1 ?

tylerscoville and others added 2 commits May 31, 2024 23:58
Creates a new config parameter jib.to.enablePlatformTags and passes it through to the build context.
@eifinger eifinger force-pushed the enable_platform_tags_plugin branch from 5559f4d to 475b7e3 Compare May 31, 2024 21:58
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.

Add option to tag child images when publishing multi platform
3 participants