chore: [servicemanagement] Enable requesting numeric enums in "transport=rest" responses for services supporting this (Java, Go, Python, PHP, TypeScript, C#, and Ruby), even if they do not yet turn on REST transport

[gcf-owl-bot]

• Regenerate this pull request now.

chore: disallow "transport=rest" for services where numeric enums are not confirmed to be supported (except in PHP and Java)
PiperOrigin-RevId: 493113566

Source-Link: googleapis/googleapis@758f0d1

Source-Link: https://github.com/googleapis/googleapis-gen/commit/78bd8f05e1276363eb14eae70e91fe4bc20703ab
Copy-Tag: eyJwIjoiamF2YS1zZXJ2aWNlLW1hbmFnZW1lbnQvLk93bEJvdC55YW1sIiwiaCI6Ijc4YmQ4ZjA1ZTEyNzYzNjNlYjE0ZWFlNzBlOTFmZTRiYzIwNzAzYWIifQ==

feat: added google.api.Service.publishing and client libraries settings
feat: added google.api.JwtLocation.cookie
feat: new fields in enum google.api.ErrorReason
fix: deprecate google.api.Endpoint.aliases
fix: deprecate google.api.BackendRule.min_deadline
docs: minor updates to comments

PiperOrigin-RevId: 488484261

Source-Link: googleapis/googleapis@bd9aa78

Source-Link: https://github.com/googleapis/googleapis-gen/commit/019a6095f9c08c339a15b88088bf9625c2cd1de0
Copy-Tag: eyJwIjoiamF2YS1zZXJ2aWNlLW1hbmFnZW1lbnQvLk93bEJvdC55YW1sIiwiaCI6IjAxOWE2MDk1ZjljMDhjMzM5YTE1Yjg4MDg4YmY5NjI1YzJjZDFkZTAifQ==

[gcf-owl-bot]

Owl bot is regenerating pull request 8785...
Build name: MDVmZTYzM2QtMjNjZC00YjQ4LTk5MDEtOTNlMDBhYzg3NWJk

[gcf-owl-bot]

Owl bot is regenerating pull request 8785...
Build name: NWU2ODE0ZDktNWE1Zi00ZWJhLWIxODktYTExZjU3M2I1NjQz

[diegomarquezp]

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.10.1:testCompile (default-testCompile) on project google-cloud-service-management: Compilation failure: Compilation failure:
[ERROR] /tmpfs/src/github/google-cloud-java/java-service-management/google-cloud-service-management/src/test/java/com/google/cloud/api/servicemanagement/v1/ServiceManagerClientHttpJsonTest.java:[36,22] cannot find symbol
[ERROR]   symbol:   class Publishing
[ERROR]   location: package com.google.api

[lqiu96]

@blakeli0 I took a look at the protos and I don't think this is an issue with the protos in this module: (https://github.com/googleapis/googleapis/blob/b6b751420f6b055d9445e81d35f90c0a2ee16b05/google/api/servicemanagement/v1/servicemanager.proto). I'm leaning towards this being an issue with gapic-generator.

It affects the auto-generated unitTest getServiceConfigTest found at:

1. google-cloud-java/java-service-management/google-cloud-service-management/src/test/java/com/google/cloud/api/servicemanagement/v1/ServiceManagerClientHttpJsonTest.java

   Line 441 in f557c6b

   .setPublishing(Publishing.newBuilder().build())

2. google-cloud-java/java-service-management/google-cloud-service-management/src/test/java/com/google/cloud/api/servicemanagement/v1/ServiceManagerClientTest.java

   Line 500 in f557c6b

   .setPublishing(Publishing.newBuilder().build())

The Service.java implementation:
https://github.com/googleapis/java-common-protos/blob/c2cde010a89123375518340819b420358e192302/proto-google-common-protos/src/main/java/com/google/api/Service.java

doesn't have this publisher field and I'm not sure why this is being included.

Can you help us take a look at this?

[lqiu96]

We are missing this PR in java-common-protos: googleapis/java-common-protos#481. Path to resolve this PR:

• Merged into java-common-protos and release
• Update the java-common-protos version in shared-deps and release
• Updated shared-deps version in this repo and then merge this PR

[suztomo]

#8876 Java shared dependencies BOM 3.1.0 has the new common-protos library.

[gcf-owl-bot]

Owl bot is regenerating pull request 8785...
Build name: NThkZDRhMzctY2I1NC00OWUxLWFmYmQtODk1MDU1MTViYjYw