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

quarkus.grpc.clients.test-port property

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • None
    • None
    • None
    • 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
    • Undefined

    Description

      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`

       

       

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: