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

2.10.3 backports 1 #26790

Merged
merged 26 commits into from Jul 19, 2022
Merged

2.10.3 backports 1 #26790

merged 26 commits into from Jul 19, 2022

Conversation

gsmet
Copy link
Member

@gsmet gsmet commented Jul 18, 2022

Please don't merge, I will merge it myself.

geoand and others added 26 commits July 18, 2022 15:53
Follows up on quarkusio#26285

(cherry picked from commit 80bd1c8)
Fixes quarkusio#26573
If container.start() is missing, it will fail with "java.lang.IllegalStateException: Mapped port can only be obtained
after the container is started", or can get a NPE at a later phase of execution, if no call to retrieving port info is
made

(cherry picked from commit 46eb5b7)
(cherry picked from commit 2b8b9b5)
- there is no easy way to reliably identify the column of an expression
used as a section param, therefore, we'll report the column of the
containing section/block for the moment
- related to quarkusio#26479

(cherry picked from commit d588d8b)
…sion following Quarkus example

(cherry picked from commit da4dd38)
(cherry picked from commit 9cda251)
It has been fixed by one of the latest Hibernate Reactive upgrades.
Probably by hibernate/hibernate-reactive#975

(cherry picked from commit 3ee180a)
The init test state is called several times depending whether JUnit 5 is configured with the lifecycle method based or class based.

Clearing out the outer instances every time the main class is initialized, ensure that the outer instances are always up to date.

(cherry picked from commit 28fc5cf)
PR quarkusio#13963 mistakenly stopped stripping debug information from the native
executable when debug info generation is enabled resulting in both the
native executable and the corresponding `.debug` file containing the
debug information. Which also results in the native executable being
larger when `-Dquarkus.native.debug.enabled` is used:

```
$ du -hs quarkus-integration-test-main-999-SNAPSHOT-runner-nodebug
108M	quarkus-integration-test-main-999-SNAPSHOT-runner-nodebug

$ du -hs quarkus-integration-test-main-999-SNAPSHOT-runner-debug
182M	quarkus-integration-test-main-999-SNAPSHOT-runner-debug

$ du -hs quarkus-integration-test-main-999-SNAPSHOT-runner-debug.debug
75M	quarkus-integration-test-main-999-SNAPSHOT-runner-debug.debug
```

Inspecting `quarkus-integration-test-main-999-SNAPSHOT-runner-debug` and
`quarkus-integration-test-main-999-SNAPSHOT-runner-debug` with
`readelf --debug-dump=info` we observe that both files are containing
debug information while only the latter should contain them.

(cherry picked from commit f69d98d)
(cherry picked from commit 053a39f)
(cherry picked from commit 906ebcb)
Signed-off-by: Phillip Kruger <phillip.kruger@gmail.com>
(cherry picked from commit 48787de)
@gsmet gsmet added the area/infra internal and infrastructure related issues label Jul 18, 2022
@gsmet gsmet self-assigned this Jul 18, 2022
@gsmet gsmet marked this pull request as ready for review July 18, 2022 14:13
@quarkus-bot

This comment has been minimized.

@gsmet gsmet merged commit 6fc7f96 into quarkusio:2.10 Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet