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

Update to Maven Docker Plugin 0.43.4 #9779

Merged
merged 2 commits into from
Sep 28, 2023
Merged

Update to Maven Docker Plugin 0.43.4 #9779

merged 2 commits into from
Sep 28, 2023

Conversation

poikilotherm
Copy link
Contributor

@poikilotherm poikilotherm commented Aug 14, 2023

What this PR does / why we need it:
With this new version, a problem on M1 MACs not being able to build with Docker because of a non-existing config file was worked around by the DMP devs.

Which issue(s) this PR closes:

Special notes for your reviewer:
There is a bug report that 0.43.2 was no longer allowing to push images. We'll see if this works again with this version.

Suggestions on how to test this:
@sekmiller is the one having the troubles. Hard to test.

Does this PR introduce a user interface change? If mockups are available, please link/include them here:

Is there a release notes update needed for this change?:

Additional documentation:

@poikilotherm poikilotherm added the Component: Containers Anything related to cloudy Dataverse, shipped in containers. label Aug 14, 2023
@poikilotherm poikilotherm added this to the 6.0 milestone Aug 14, 2023
@poikilotherm poikilotherm self-assigned this Aug 14, 2023
@poikilotherm poikilotherm changed the title build(deps): update to Maven Docker Plugin 0.43.3 #9771 Update to Maven Docker Plugin 0.43.4 Aug 18, 2023
@github-actions

This comment has been minimized.

@poikilotherm poikilotherm marked this pull request as ready for review August 18, 2023 07:44
@poikilotherm poikilotherm added the Size: 0.5 A percentage of a sprint. 0.35 hours label Aug 22, 2023
@poikilotherm poikilotherm removed this from the 6.0 milestone Sep 8, 2023
@cmbz cmbz added this to the 6.1 milestone Sep 18, 2023
@cmbz
Copy link

cmbz commented Sep 18, 2023

2023/09/18: Moved to Sprint Ready as per discussion in prioritization meeting.

@poikilotherm poikilotherm removed their assignment Sep 20, 2023
With this new version, a problem on M1 MACs not being able to build
with Docker because of a non-existing config file was worked around
by the DMP devs.

See also: fabric8io/docker-maven-plugin#1678
@pdurbin pdurbin force-pushed the 9771-fix-docker-build branch from 198344a to c93b180 Compare September 26, 2023 20:41
Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

Works fine on my machine. Approved.

@coveralls
Copy link

Coverage Status

coverage: 20.047%. remained the same when pulling c93b180 on 9771-fix-docker-build into 5fc7b30 on develop.

@github-actions
Copy link

📦 Pushed preview images as

ghcr.io/gdcc/dataverse:9771-fix-docker-build
ghcr.io/gdcc/configbaker:9771-fix-docker-build

🚢 See on GHCR. Use by referencing with full name as printed above, mind the registry name.

@kcondon kcondon self-assigned this Sep 28, 2023
@kcondon kcondon merged commit f8e8d82 into develop Sep 28, 2023
@kcondon kcondon deleted the 9771-fix-docker-build branch September 28, 2023 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Containers Anything related to cloudy Dataverse, shipped in containers. Size: 0.5 A percentage of a sprint. 0.35 hours
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

DOCKER> unknown flag: --driver with M1 Mac
5 participants