Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

Bump Quarkus from 1.12.2.Final to 1.13.0.Final #258

Closed
wants to merge 2 commits into from

Conversation

rsvoboda
Copy link
Member

@rsvoboda rsvoboda commented Apr 2, 2021

Bump Quarkus from 1.12.2.Final to 1.13.0.Final

@rsvoboda
Copy link
Member Author

rsvoboda commented Apr 2, 2021

run tests

1 similar comment
@rsvoboda
Copy link
Member Author

rsvoboda commented Apr 2, 2021

run tests

@rsvoboda
Copy link
Member Author

rsvoboda commented Apr 2, 2021

%test.quarkus.grpc.clients.hello.port=9001 is not needed, causes fail in the run

[INFO] Quarkus OpenShift TS: Security: Keycloak Authentication + Authorization FAILURE [ 42.302 s] with master :/

Copy link
Contributor

@Sgitario Sgitario left a comment

Choose a reason for hiding this comment

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

I disagree about having the property quarkus.grpc.clients.hello.port as global property as it is only needed on modules having a hello grpc service.

@Sgitario
Copy link
Contributor

Sgitario commented Apr 6, 2021

run tests

4 similar comments
@Sgitario
Copy link
Contributor

Sgitario commented Apr 6, 2021

run tests

@Sgitario
Copy link
Contributor

Sgitario commented Apr 8, 2021

run tests

@Sgitario
Copy link
Contributor

Sgitario commented Apr 8, 2021

run tests

@Sgitario
Copy link
Contributor

run tests

@Sgitario
Copy link
Contributor

Upgrade will be done as part of #266. Closing.

@Sgitario Sgitario closed this Apr 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants