Uploaded image for project: 'Quarkus'
  1. Quarkus
  2. QUARKUS-909

quarkus.grpc.clients.test-port property

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • None
    • None
    • None
    • False
    • False
    • Undefined
    • Hide

      Quarkus-quick-start / grp without this `%test.quarkus.grpc.clients.hello.port=9001` property

      Show
      Quarkus-quick-start / grp without this `%test.quarkus.grpc.clients.hello.port=9001` property

      In version 1.11 the only required configuration that needed, in order to run a quarkus app with Grpc was

      quarkus.grpc.clients.hello.host=localhost
      

      But in upstream we had a port collision, and we had to add a new property (quarkus.grpc.clients.<service_name>.port) to the application.properties file

      quarkus.grpc.clients.hello.host=localhost
      %test.quarkus.grpc.clients.hello.port=9001
      

      Driven by this issue: https://github.com/quarkusio/quarkus/issues/15934

      If this is the final approach we are going to introduce breaking changes that must be documented. 

       

      We suggest adding a new property to `Quarkus-grpc`  in order to avoid this breaking change. Something like we already have: `quarkus.http.port` and `quarkus.http.test-port`

       

      Suggested property: `quarkus.grpc.clients.<service_name>.test-port` default port `9001`

       

       

              Unassigned Unassigned
              rhn-support-pagonzal Pablo Gonzalez Granados (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: