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

Quarkus NativeImageBuildStep fails with perm denied with docker rootless #26892

Merged
merged 1 commit into from
Sep 14, 2022
Merged

Quarkus NativeImageBuildStep fails with perm denied with docker rootless #26892

merged 1 commit into from
Sep 14, 2022

Conversation

reda-alaoui
Copy link
Contributor

Fix #26890

@geoand geoand requested a review from zakkak July 26, 2022 09:10
zakkak
zakkak reviewed Jul 26, 2022
View reviewed changes
Copy link
Contributor

@zakkak zakkak left a comment

Choose a reason for hiding this comment

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

As mentioned in #26890 this is related to moby/moby#42134 (comment)
Unfortunately though there is no proper solution there. There seem to be two options for this issue:

  1. Run as root inside the container as this PR does (I wonder if that results in files been created inside the container to be owned by root on the host)
  2. Adjust the permissions of the volumes being mounted on the host to match the actual user/group id of the user in the container.

The first approach is certainly simpler, but results in running the builder image container as root, which is not desirable in general.
The second approach is a bit more complex and requires careful handling to make sure permissions are set properly before and after the builder image runs.

I need to setup docker in rootless mode in order to toy around with this options, which I will probably not have the time to do in the following weeks, so if there is anyone else willing to experiment and review this PR in the meantime please do so.

@reda-alaoui
Copy link
Contributor Author

Could someone at least trigger the build?

@geoand
Copy link
Contributor

geoand commented Sep 2, 2022

@zakkak what should we do with this one?

@zakkak
Copy link
Contributor

zakkak commented Sep 2, 2022

@zakkak what should we do with this one?

Setting up a rootless docker to test this out and see if we can do better (as discussed in #26892 (review)) is in my TODO list but certainly not a priority.

I suggest we wait till next Friday to see if I will manage to have a better look at it, and if I fail we can merge it as is and come up with a follow up PR later if we find a better solution. WDYT?

@geoand
Copy link
Contributor

geoand commented Sep 2, 2022

Sounds good

@zakkak zakkak self-assigned this Sep 2, 2022
zakkak
zakkak requested changes Sep 9, 2022
View reviewed changes
Copy link
Contributor

@zakkak zakkak left a comment

Choose a reason for hiding this comment

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

I managed to find some time and tried the PR with docker rootless and it works as expected, but requires setting the DOCKER_HOST env variable which is not necessary in order to use docker rootless see https://docs.docker.com/engine/security/rootless/#client.

Note: Unfortunately I will probably not be able to explore alternative options before the 19th of September.

...ment/src/main/java/io/quarkus/deployment/pkg/steps/NativeImageBuildLocalContainerRunner.java Outdated Show resolved Hide resolved
@reda-alaoui reda-alaoui force-pushed the fix-26890 branch 2 times, most recently from 0c47727 to a401d3e Compare September 11, 2022 10:12
@reda-alaoui reda-alaoui requested a review from zakkak September 11, 2022 10:15
@reda-alaoui reda-alaoui force-pushed the fix-26890 branch 5 times, most recently from d1cbd00 to 31eec4b Compare September 11, 2022 10:59
@reda-alaoui reda-alaoui requested a review from zakkak September 12, 2022 09:05
zakkak
zakkak approved these changes Sep 12, 2022
View reviewed changes
Copy link
Contributor

@zakkak zakkak left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Although I am not a big fan of running the container image as root I confirm that the PR fixes the issue and is good for merging till we figure out a better alternative.

Thank you for your contribution @reda-alaoui!

@reda-alaoui
Copy link
Contributor Author

reda-alaoui commented Sep 13, 2022 •
edited
Loading

Are we waiting for something/someone to merge? :)

@geoand
Copy link
Contributor

geoand commented Sep 13, 2022

Can you please squash and rebase onto main?

@reda-alaoui reda-alaoui marked this pull request as draft September 13, 2022 18:40
@reda-alaoui reda-alaoui marked this pull request as ready for review September 13, 2022 18:40
@reda-alaoui
Copy link
Contributor Author

Can you please squash and rebase onto main?

Done 👍

@geoand
Copy link
Contributor

geoand commented Sep 13, 2022

@zakkak I'll let you merge

@reda-alaoui reda-alaoui requested a review from zakkak September 13, 2022 18:47
@zakkak zakkak merged commit 37f266d into quarkusio:main Sep 14, 2022
@quarkus-bot quarkus-bot bot added this to the 2.14 - main milestone Sep 14, 2022
@geoand
Copy link
Contributor

geoand commented Sep 14, 2022

I suppose we want this in 2.13? If so, please add the triage/backport label.

@zakkak
Copy link
Contributor

zakkak commented Sep 14, 2022

I suppose we want this in 2.13? If so, please add the triage/backport label.

I think it would indeed be nice to have.

Dairdevil pushed a commit to Dairdevil/quarkus that referenced this pull request Sep 14, 2022
Squashed commit of the following:
d4305e3 
commit 62657c0
Merge: f7b6773 2928919
Author: Alasdair Preston <[email protected]>
Date:   Wed Sep 14 11:08:51 2022 +0100

    Merge branch 'bugfix/26587-openshift-route-named-port' of github.com:Dairdevil/quarkus into bugfix/26587-openshift-route-named-port

commit f7b6773
Author: Michal Vavřík <[email protected]>
Date:   Tue Sep 13 21:10:54 2022 +0200

    Generate Quarkus Maven Plugin Config Docs

    resolves: quarkusio#1204

commit 9bfd447
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Fri Sep 9 21:09:37 2022 +0000

    Bump kafka3.version from 3.2.1 to 3.2.2

    Bumps `kafka3.version` from 3.2.1 to 3.2.2.

    Updates `kafka-clients` from 3.2.1 to 3.2.2

    Updates `kafka-streams` from 3.2.1 to 3.2.2

    Updates `kafka-streams-test-utils` from 3.2.1 to 3.2.2

    Updates `kafka_2.13` from 3.2.1 to 3.2.2

    ---
    updated-dependencies:
    - dependency-name: org.apache.kafka:kafka-clients
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka-streams
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka-streams-test-utils
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka_2.13
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit ebc0112
Author: Sanne Grinovero <[email protected]>
Date:   Tue Sep 13 17:28:18 2022 +0100

    Transform DB2 driver to Jakarta APIs

commit 3bc918c
Author: Réda Housni Alaoui <[email protected]>
Date:   Fri Jul 22 22:15:40 2022 +0200

    Quarkus NativeImageBuildStep fails with perm denied with docker rootless

commit 604bab7
Author: Auri Munoz <[email protected]>
Date:   Fri Sep 9 16:02:49 2022 +0200

    update Stork version to align with k8s client 6.x

    Related to quarkusio#6588

commit 59c4822
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue Sep 13 23:01:39 2022 +0000

    Bump actions/download-artifact from 1 to 3

    Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 1 to 3.
    - [Release notes](https://github.com/actions/download-artifact/releases)
    - [Commits](actions/download-artifact@v1...v3)

    ---
    updated-dependencies:
    - dependency-name: actions/download-artifact
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit d1c8566
Author: Zheng Feng <[email protected]>
Date:   Wed Jun 15 08:58:13 2022 +0800

    Improve Narayana recovery manager service and integrate it with agroal

commit beab85f
Author: Holly Cummins <[email protected]>
Date:   Fri Sep 9 20:26:25 2022 +0100

    Cross-link to quarkiverse wiki

commit 701b930
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue Sep 13 09:55:11 2022 +0000

    Bump liquibase.version from 4.15.0 to 4.16.0

    Bumps `liquibase.version` from 4.15.0 to 4.16.0.

    Updates `liquibase-core` from 4.15.0 to 4.16.0
    - [Release notes](https://github.com/liquibase/liquibase/releases)
    - [Changelog](https://github.com/liquibase/liquibase/blob/master/changelog.txt)
    - [Commits](liquibase/liquibase@v4.15.0...v4.16.0)

    Updates `liquibase-mongodb` from 4.15.0 to 4.16.0
    - [Release notes](https://github.com/liquibase/liquibase-mongodb/releases)
    - [Changelog](https://github.com/liquibase/liquibase-mongodb/blob/main/RELEASE.md)
    - [Commits](liquibase/liquibase-mongodb@liquibase-mongodb-4.15.0...liquibase-mongodb-4.16.0)

    ---
    updated-dependencies:
    - dependency-name: org.liquibase:liquibase-core
      dependency-type: direct:production
      update-type: version-update:semver-minor
    - dependency-name: org.liquibase.ext:liquibase-mongodb
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit 22c71e9
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Mon Sep 12 21:22:57 2022 +0000

    Bump micrometer-bom from 1.9.3 to 1.9.4

    Bumps [micrometer-bom](https://github.com/micrometer-metrics/micrometer) from 1.9.3 to 1.9.4.
    - [Release notes](https://github.com/micrometer-metrics/micrometer/releases)
    - [Commits](micrometer-metrics/micrometer@v1.9.3...v1.9.4)

    ---
    updated-dependencies:
    - dependency-name: io.micrometer:micrometer-bom
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit ebf6e16
Author: Martin Kouba <[email protected]>
Date:   Tue Sep 13 14:19:16 2022 +0200

    ArC - support non-producer methods annotated with @startup

    - resolves quarkusio#11710

commit 5b677dc
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 15:25:04 2022 +0300

    Properly support generic bounds for ParamConverterProvider classes

    Fixes: quarkusio#27892

commit f2382b1
Author: Holly Cummins <[email protected]>
Date:   Tue Sep 13 15:22:21 2022 +0100

    Allow some PRs to run workflows, even when repo settings require approval for all non-committer PRs

commit 492b184
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:45:04 2022 +0200

    Add more info about releasing to CONTRIBUTING.md

commit f6053e0
Author: Réda Housni Alaoui <[email protected]>
Date:   Thu Aug 4 21:36:36 2022 +0200

    Document the release process

    Fix quarkusio#26901

commit db141a2
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 14:50:18 2022 +0300

    Ensure that Mongo DNS lookup does not happen on the event loop

    This is done by performing the DNS lookup at application startup
    and caching the results

    Fixes: quarkusio#27627

commit 8baab4e
Author: Katia Aresti <[email protected]>
Date:   Fri Sep 9 12:33:14 2022 +0200

    Create Infinispan caches on first access with the minimal config if they don't exist

commit 24bcf7c
Author: Ozan Gunalp <[email protected]>
Date:   Tue Aug 23 19:37:47 2022 +0300

    Kafka dev ui as a dev console route

    minor fixes for UI

commit 73d2c6f
Author: Emiliia Nesterovych <[email protected]>
Date:   Wed Aug 3 13:17:41 2022 +0200

    extend kafka dev ui

commit d27a75a
Author: Oleksiy Lukin <[email protected]>
Date:   Mon Aug 1 12:45:58 2022 +0200

    implement basic kafka dev ui

commit 11e984f
Author: Roberto Cortez <[email protected]>
Date:   Mon Sep 5 17:43:03 2022 +0100

    Properly generate documents from config groups in ConfigMapping

commit 36f1509
Author: xstefank <[email protected]>
Date:   Fri Sep 9 11:07:19 2022 +0200

    Upgrade SmallRye Health to 3.3.0

commit 6fd7f1c
Author: Tomas Hofman <[email protected]>
Date:   Fri Sep 9 14:54:02 2022 +0200

    Rest client config: Remove incorrect note about the proxyAddress property being only applicable to reactive clients

commit efddfeb
Author: Tomas Hofman <[email protected]>
Date:   Fri Sep 9 14:46:49 2022 +0200

    Add RestClientBuilder factories

    These factories should make it easier for developers to obtain a
    RestClientBuilder instance preconfigured according to current
    application config. The builder can be further tweaked and then used to
    programatically build a rest client proxy.

commit 28e2306
Author: Tomas Hofman <[email protected]>
Date:   Thu Sep 8 14:47:56 2022 +0200

    Rest client classic: make configuration methods in RestClientBase protected

    This is to accommodate kogito extension that accesses these methods.

commit 8e4d836
Author: Loïc Mathieu <[email protected]>
Date:   Wed Aug 31 11:13:57 2022 +0200

    Improve command mode guide

commit a982c88
Author: vsevel <[email protected]>
Date:   Tue Sep 13 09:52:25 2022 +0200

    Bump jboss-logmanager 1.1.1

commit 1bfd5e5
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Sep 12 10:16:13 2022 +0300

    Return proper response code when an HTTP request contains an invalid Q value

    Closes: quarkusio#27852

commit af22e7c
Author: Clement Escoffier <[email protected]>
Date:   Tue Sep 13 10:27:57 2022 +0200

    Use default host and port when the service instance does not set them

commit 7cb775e
Author: Clement Escoffier <[email protected]>
Date:   Tue Sep 13 09:03:49 2022 +0200

    Fix request context leak in the Funqy Knative runtime.

commit a861e54
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Sep 12 13:13:03 2022 +0300

    Allow setting a custom logging filter on logging handlers

    Resolves: quarkusio#25981

commit 2928919
Merge: 71ae072 9402450
Author: Alasdair Preston <[email protected]>
Date:   Wed Sep 14 11:04:37 2022 +0100

    Merge branch 'main' of github.com:Dairdevil/quarkus into bugfix/26587-openshift-route-named-port

commit 9402450
Merge: b963b7e dba1b1e
Author: Guillaume Smet <[email protected]>
Date:   Wed Sep 14 10:42:59 2022 +0200

    Merge pull request quarkusio#26369 from michalvavrik/feature/mvn-docs-generator

    Generate Quarkus Maven Plugin Config Docs

commit b963b7e
Merge: 3ab09ab 06caa54
Author: Clement Escoffier <[email protected]>
Date:   Wed Sep 14 10:32:57 2022 +0200

    Merge pull request quarkusio#27847 from quarkusio/dependabot/maven/kafka3.version-3.2.2

    Bump kafka3.version from 3.2.1 to 3.2.2

commit 3ab09ab
Merge: 37f266d ec6d2cf
Author: Guillaume Smet <[email protected]>
Date:   Wed Sep 14 10:29:42 2022 +0200

    Merge pull request quarkusio#27910 from Sanne/DB2JakartaNative

    Allow transforming the DB2 JDBC driver to Jakarta APIs during Augmentation

commit 37f266d
Merge: 6e10b21 565f4d4
Author: Foivos <[email protected]>
Date:   Wed Sep 14 11:18:52 2022 +0300

    Merge pull request quarkusio#26892 from Cosium/fix-26890

    Quarkus NativeImageBuildStep fails with perm denied with docker rootless

commit 71ae072
Author: Alasdair Preston <[email protected]>
Date:   Tue Sep 13 11:23:38 2022 +0100

        Override target port used when creating openshift route

commit ec6d2cf
Author: Sanne Grinovero <[email protected]>
Date:   Tue Sep 13 17:28:18 2022 +0100

    Transform DB2 driver to Jakarta APIs

commit 6e10b21
Merge: 6049d8f eee3dc2
Author: Georgios Andrianakis <[email protected]>
Date:   Wed Sep 14 10:21:17 2022 +0300

    Merge pull request quarkusio#27839 from aureamunoz/upgrade-stork-version

    Update Stork version to align with k8s client 6.x

commit 6049d8f
Merge: bc5fb5a d90af0f
Author: Guillaume Smet <[email protected]>
Date:   Wed Sep 14 08:16:16 2022 +0200

    Merge pull request quarkusio#27909 from quarkusio/dependabot/github_actions/actions/download-artifact-3

    Bump actions/download-artifact from 1 to 3

commit bc5fb5a
Merge: 6461dd5 52c8f4d
Author: George Gastaldi <[email protected]>
Date:   Wed Sep 14 01:04:18 2022 -0300

    Merge pull request quarkusio#26161 from zhfeng/narayana_recovery_service

commit d90af0f
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue Sep 13 23:01:39 2022 +0000

    Bump actions/download-artifact from 1 to 3

    Bumps [actions/download-artifact](https://github.com/actions/download-artifact) from 1 to 3.
    - [Release notes](https://github.com/actions/download-artifact/releases)
    - [Commits](actions/download-artifact@v1...v3)

    ---
    updated-dependencies:
    - dependency-name: actions/download-artifact
      dependency-type: direct:production
      update-type: version-update:semver-major
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit dba1b1e
Author: Michal Vavřík <[email protected]>
Date:   Tue Sep 13 21:10:54 2022 +0200

    Generate Quarkus Maven Plugin Config Docs

    resolves: quarkusio#1204

commit 565f4d4
Author: Réda Housni Alaoui <[email protected]>
Date:   Fri Jul 22 22:15:40 2022 +0200

    Quarkus NativeImageBuildStep fails with perm denied with docker rootless

commit 6461dd5
Merge: 5c5e7ce 223b6cc
Author: George Gastaldi <[email protected]>
Date:   Tue Sep 13 15:14:33 2022 -0300

    Merge pull request quarkusio#27846 from holly-cummins/holly-wiki-links

commit 223b6cc
Author: Holly Cummins <[email protected]>
Date:   Fri Sep 9 20:26:25 2022 +0100

    Cross-link to quarkiverse wiki

commit 5c5e7ce
Merge: 0ddffcf b8405f3
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 19:17:29 2022 +0200

    Merge pull request quarkusio#27863 from quarkusio/dependabot/maven/liquibase.version-4.16.0

    Bump liquibase.version from 4.15.0 to 4.16.0

commit 0ddffcf
Merge: a675c2e eaedc32
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 19:16:32 2022 +0200

    Merge pull request quarkusio#27880 from quarkusio/dependabot/maven/io.micrometer-micrometer-bom-1.9.4

    Bump micrometer-bom from 1.9.3 to 1.9.4

commit a675c2e
Merge: 329af5a 78191c0
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 19:16:15 2022 +0200

    Merge pull request quarkusio#27897 from mkouba/issue-11710

    Support non-producer methods annotated with @startup

commit 329af5a
Merge: 20cea48 67eb7b5
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 18:40:25 2022 +0300

    Merge pull request quarkusio#27898 from geoand/quarkusio#27892

    Properly support generic bounds for ParamConverterProvider classes

commit 20cea48
Merge: 602ed25 fd811f3
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 16:28:03 2022 +0200

    Merge pull request quarkusio#27899 from holly-cummins/holly-enable-workflow-bot

    Auto-approve workflow runs from established contributors or low-risk files

commit fd811f3
Author: Holly Cummins <[email protected]>
Date:   Tue Sep 13 15:22:21 2022 +0100

    Allow some PRs to run workflows, even when repo settings require approval for all non-committer PRs

commit 602ed25
Merge: 20d9c4f e3f4dfe
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 16:19:20 2022 +0200

    Merge pull request quarkusio#27142 from reda-alaoui/patch-1

    Document the release process

commit 20d9c4f
Merge: c04644b e04b11f
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 16:10:58 2022 +0300

    Merge pull request quarkusio#27896 from geoand/quarkusio#27627

    Ensure that Mongo DNS lookup does not happen on the event loop

commit c04644b
Merge: 5e6a3c9 c18c4d0
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 16:03:31 2022 +0300

    Merge pull request quarkusio#27838 from karesti/minimal-viable-config

    Create Infinispan caches on first access with the minimal config

commit e3f4dfe
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:45:04 2022 +0200

    Add more info about releasing to CONTRIBUTING.md

commit a9e0f55
Author: Réda Housni Alaoui <[email protected]>
Date:   Thu Aug 4 21:36:36 2022 +0200

    Document the release process

    Fix quarkusio#26901

commit 67eb7b5
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 15:25:04 2022 +0300

    Properly support generic bounds for ParamConverterProvider classes

    Fixes: quarkusio#27892

commit 78191c0
Author: Martin Kouba <[email protected]>
Date:   Tue Sep 13 14:19:16 2022 +0200

    ArC - support non-producer methods annotated with @startup

    - resolves quarkusio#11710

commit 5e6a3c9
Merge: 7cfcecc 1750554
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:29:35 2022 +0200

    Merge pull request quarkusio#26998 from alukin/feature/kafka-client-dev-ui-squashed

    Introduce a Kafka Client DevUI component

commit 7cfcecc
Merge: 599d415 01499ac
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:22:45 2022 +0200

    Merge pull request quarkusio#27756 from radcortez/fix-27668

    Properly generate documents from config groups in ConfigMapping

commit 599d415
Merge: 45e97cc 07fee83
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:20:53 2022 +0200

    Merge pull request quarkusio#27829 from xstefank/health-3.3.0

    Upgrade SmallRye Health to 3.3.0

commit 45e97cc
Merge: 7e79e40 58c0966
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:20:11 2022 +0200

    Merge pull request quarkusio#27810 from TomasHofman/kogito-visibility-fix

    Rest client classic: revert to package private visibility for configuration methods

commit 7e79e40
Merge: 469fd21 6e121b2
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:18:43 2022 +0200

    Merge pull request quarkusio#27539 from loicmathieu/command-mode-guide

    Improve command mode guide

commit c18c4d0
Author: Katia Aresti <[email protected]>
Date:   Fri Sep 9 12:33:14 2022 +0200

    Create Infinispan caches on first access with the minimal config if they don't exist

commit 469fd21
Merge: 89fc74f c3250e0
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 14:14:27 2022 +0200

    Merge pull request quarkusio#27890 from vsevel/jboss-logmanager-1.1.1

    Bump jboss-logmanager 1.1.1

commit c3250e0
Author: vsevel <[email protected]>
Date:   Tue Sep 13 09:52:25 2022 +0200

    Bump jboss-logmanager 1.1.1

commit e04b11f
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 14:50:18 2022 +0300

    Ensure that Mongo DNS lookup does not happen on the event loop

    This is done by performing the DNS lookup at application startup
    and caching the results

    Fixes: quarkusio#27627

commit 89fc74f
Merge: 136ed6f f8b761c
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 13:47:28 2022 +0200

    Merge pull request quarkusio#27859 from geoand/quarkusio#27852

    Return proper response code when an HTTP request contains an invalid Q value

commit 136ed6f
Merge: 9de09fa 66a11c4
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 13:46:40 2022 +0200

    Merge pull request quarkusio#27891 from cescoffier/stork-filter-handle-unset-host-and-port

    Use default host and port when the service instance does not set them

commit 9de09fa
Merge: 709f2b0 af89f4f
Author: Guillaume Smet <[email protected]>
Date:   Tue Sep 13 13:21:00 2022 +0200

    Merge pull request quarkusio#27885 from cescoffier/fix-knative-funqy-context-issue

    Fix request context leak in the Funqy Knative runtime

commit 709f2b0
Merge: b60ad38 3fb466f
Author: Georgios Andrianakis <[email protected]>
Date:   Tue Sep 13 13:43:16 2022 +0300

    Merge pull request quarkusio#27864 from geoand/quarkusio#25981

    Allow setting a custom logging filter on logging handlers

commit b8405f3
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Tue Sep 13 09:55:11 2022 +0000

    Bump liquibase.version from 4.15.0 to 4.16.0

    Bumps `liquibase.version` from 4.15.0 to 4.16.0.

    Updates `liquibase-core` from 4.15.0 to 4.16.0
    - [Release notes](https://github.com/liquibase/liquibase/releases)
    - [Changelog](https://github.com/liquibase/liquibase/blob/master/changelog.txt)
    - [Commits](liquibase/liquibase@v4.15.0...v4.16.0)

    Updates `liquibase-mongodb` from 4.15.0 to 4.16.0
    - [Release notes](https://github.com/liquibase/liquibase-mongodb/releases)
    - [Changelog](https://github.com/liquibase/liquibase-mongodb/blob/main/RELEASE.md)
    - [Commits](liquibase/liquibase-mongodb@liquibase-mongodb-4.15.0...liquibase-mongodb-4.16.0)

    ---
    updated-dependencies:
    - dependency-name: org.liquibase:liquibase-core
      dependency-type: direct:production
      update-type: version-update:semver-minor
    - dependency-name: org.liquibase.ext:liquibase-mongodb
      dependency-type: direct:production
      update-type: version-update:semver-minor
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit 66a11c4
Author: Clement Escoffier <[email protected]>
Date:   Tue Sep 13 10:27:57 2022 +0200

    Use default host and port when the service instance does not set them

commit af89f4f
Author: Clement Escoffier <[email protected]>
Date:   Tue Sep 13 09:03:49 2022 +0200

    Fix request context leak in the Funqy Knative runtime.

commit 3fb466f
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Sep 12 13:13:03 2022 +0300

    Allow setting a custom logging filter on logging handlers

    Resolves: quarkusio#25981

commit eaedc32
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Mon Sep 12 21:22:57 2022 +0000

    Bump micrometer-bom from 1.9.3 to 1.9.4

    Bumps [micrometer-bom](https://github.com/micrometer-metrics/micrometer) from 1.9.3 to 1.9.4.
    - [Release notes](https://github.com/micrometer-metrics/micrometer/releases)
    - [Commits](micrometer-metrics/micrometer@v1.9.3...v1.9.4)

    ---
    updated-dependencies:
    - dependency-name: io.micrometer:micrometer-bom
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit 1750554
Author: Ozan Gunalp <[email protected]>
Date:   Tue Aug 23 19:37:47 2022 +0300

    Kafka dev ui as a dev console route

    minor fixes for UI

commit 119a046
Author: Emiliia Nesterovych <[email protected]>
Date:   Wed Aug 3 13:17:41 2022 +0200

    extend kafka dev ui

commit 1a8b36b
Author: Oleksiy Lukin <[email protected]>
Date:   Mon Aug 1 12:45:58 2022 +0200

    implement basic kafka dev ui

commit 58c0966
Author: Tomas Hofman <[email protected]>
Date:   Fri Sep 9 14:54:02 2022 +0200

    Rest client config: Remove incorrect note about the proxyAddress property being only applicable to reactive clients

commit c0e11c6
Author: Tomas Hofman <[email protected]>
Date:   Fri Sep 9 14:46:49 2022 +0200

    Add RestClientBuilder factories

    These factories should make it easier for developers to obtain a
    RestClientBuilder instance preconfigured according to current
    application config. The builder can be further tweaked and then used to
    programatically build a rest client proxy.

commit 07fee83
Author: xstefank <[email protected]>
Date:   Fri Sep 9 11:07:19 2022 +0200

    Upgrade SmallRye Health to 3.3.0

commit f8b761c
Author: Georgios Andrianakis <[email protected]>
Date:   Mon Sep 12 10:16:13 2022 +0300

    Return proper response code when an HTTP request contains an invalid Q value

    Closes: quarkusio#27852

commit eee3dc2
Author: Auri Munoz <[email protected]>
Date:   Fri Sep 9 16:02:49 2022 +0200

    update Stork version to align with k8s client 6.x

    Related to quarkusio#6588

commit 06caa54
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date:   Fri Sep 9 21:09:37 2022 +0000

    Bump kafka3.version from 3.2.1 to 3.2.2

    Bumps `kafka3.version` from 3.2.1 to 3.2.2.

    Updates `kafka-clients` from 3.2.1 to 3.2.2

    Updates `kafka-streams` from 3.2.1 to 3.2.2

    Updates `kafka-streams-test-utils` from 3.2.1 to 3.2.2

    Updates `kafka_2.13` from 3.2.1 to 3.2.2

    ---
    updated-dependencies:
    - dependency-name: org.apache.kafka:kafka-clients
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka-streams
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka-streams-test-utils
      dependency-type: direct:production
      update-type: version-update:semver-patch
    - dependency-name: org.apache.kafka:kafka_2.13
      dependency-type: direct:production
      update-type: version-update:semver-patch
    ...

    Signed-off-by: dependabot[bot] <[email protected]>

commit 52c8f4d
Author: Zheng Feng <[email protected]>
Date:   Wed Jun 15 08:58:13 2022 +0800

    Improve Narayana recovery manager service and integrate it with agroal

commit 23d2600
Author: Tomas Hofman <[email protected]>
Date:   Thu Sep 8 14:47:56 2022 +0200

    Rest client classic: make configuration methods in RestClientBase protected

    This is to accommodate kogito extension that accesses these methods.

commit 01499ac
Author: Roberto Cortez <[email protected]>
Date:   Mon Sep 5 17:43:03 2022 +0100

    Properly generate documents from config groups in ConfigMapping

commit 6e121b2
Author: Loïc Mathieu <[email protected]>
Date:   Wed Aug 31 11:13:57 2022 +0200

    Improve command mode guide
@reda-alaoui reda-alaoui deleted the fix-26890 branch September 14, 2022 17:03
@gsmet gsmet modified the milestones: 2.14 - main, 2.13.0.Final Sep 20, 2022
zakkak added a commit to zakkak/quarkus that referenced this pull request Sep 27, 2022
@zakkak
Fix support for rootless docker context
71a0501 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892
@zakkak zakkak mentioned this pull request Sep 27, 2022
Merged
gsmet pushed a commit to gsmet/quarkus that referenced this pull request Oct 3, 2022
@zakkak @gsmet
Fix support for rootless docker context
e1199cd 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892

(cherry picked from commit 71a0501)
igorregis pushed a commit to igorregis/quarkus that referenced this pull request Oct 16, 2022
@zakkak @igorregis
Fix support for rootless docker context
f5f5c6e 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892
igorregis pushed a commit to igorregis/quarkus that referenced this pull request Oct 16, 2022
@zakkak @igorregis
Fix support for rootless docker context
1d10321 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892
igorregis pushed a commit to igorregis/quarkus that referenced this pull request Oct 16, 2022
@zakkak @igorregis
Fix support for rootless docker context
ac3afa0 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892
igorregis pushed a commit to igorregis/quarkus that referenced this pull request Oct 17, 2022
@zakkak @igorregis
Fix support for rootless docker context
9f2004b 
With the `'` it returns `'''unix:///run/user/1000/docker.sock'` which
fails the startsWith("unix://") check.

Fix up quarkusio#26892
@rsvoboda rsvoboda mentioned this pull request Oct 24, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zakkak zakkak zakkak approved these changes

Assignees

@zakkak zakkak

Labels
area/core kind/bugfix
Projects
None yet
Milestone
2.13.0.Final
Development

Successfully merging this pull request may close these issues.

Quarkus NativeImageBuildStep fails with perm denied with docker rootless
4 participants
@reda-alaoui @geoand @zakkak @gsmet