Skip to content

Commit d43edaf

Browse files
committed
Merge remote-tracking branch 'origin/main' into observability/tracing-attr/url.full
2 parents 495aa48 + dea24db commit d43edaf

2,017 files changed

Lines changed: 555313 additions & 13162 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
4.68 KB
Binary file not shown.

.github/scripts/release_manager_merge_bot.go

Lines changed: 23 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,7 +213,16 @@ func main() {
213213
}
214214

215215
state = *status.State
216-
log.Printf("Overall status: %s", state)
216+
log.Printf("Overall commit status: %s", state)
217+
if pr.MergeableState != nil {
218+
log.Printf("PR mergeable state: %s", *pr.MergeableState)
219+
if *pr.MergeableState == "dirty" {
220+
fatalError("PR #%d has merge conflicts (dirty).", prNumber)
221+
}
222+
if *pr.MergeableState == "draft" {
223+
fatalError("PR #%d is a draft.", prNumber)
224+
}
225+
}
217226

218227
switch state {
219228
case "failure":
@@ -230,7 +239,19 @@ func main() {
230239
}
231240
retryCount++
232241
case "success":
233-
log.Println("All checks have passed. Merging the pull request...")
242+
// Check if the PR is truly mergeable. MergeableState can be "blocked" if required
243+
// check runs (e.g. GitHub Actions) or reviews are missing, even if CombinedStatus is "success".
244+
// If it's nil, GitHub is still calculating mergeability.
245+
if pr.MergeableState == nil || *pr.MergeableState == "blocked" || *pr.MergeableState == "behind" {
246+
stateStr := "nil"
247+
if pr.MergeableState != nil {
248+
stateStr = *pr.MergeableState
249+
}
250+
log.Printf("Commit status is success, but PR mergeable state is %q. Waiting...", stateStr)
251+
goto wait
252+
}
253+
254+
log.Println("All checks have passed and PR is mergeable. Merging the pull request...")
234255
commitMessage := fmt.Sprintf("Merge pull request #%d from %s/%s", prNumber, owner, repo)
235256
mergeResult, _, err := client.PullRequests.Merge(ctx, owner, repo, prNumber, commitMessage, &github.PullRequestOptions{
236257
MergeMethod: "squash",

.kokoro/common.sh

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -344,20 +344,20 @@ function run_graalvm_tests() {
344344
function generate_graalvm_presubmit_modules_list() {
345345
modules_assigned_list=()
346346
generate_modified_modules_list
347-
if [[ ${#modified_module_list[@]} -gt 4 ]]; then
348-
# Too many modules modified, run a subset
349-
echo "Too many modules modified, running a subset"
350-
module_list="java-aiplatform,java-compute"
351-
elif [[ ${#modified_module_list[@]} -gt 0 ]]; then
347+
if [[ ${#modified_module_list[@]} -gt 0 && ${#modified_module_list[@]} -lt 5 ]]; then
352348
# If only a few modules have been modified, focus presubmit testing only on them.
353349
module_list=$(
354350
IFS=,
355351
echo "${modified_module_list[*]}"
356352
)
357353
else
358-
# no modules modified
359-
echo "No modules modified"
360-
module_list=""
354+
# If no modules have been modified or if too many have been modified, just test the modules
355+
# specified in the MAVEN_MODULES env var.
356+
if [ -z "${MAVEN_MODULES}" ]; then
357+
echo "MAVEN_MODULES not defined in environment."
358+
exit 1
359+
fi
360+
module_list=${MAVEN_MODULES}
361361
fi
362362
}
363363

.kokoro/presubmit/graalvm-native-a-presubmit.cfg

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,8 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34+
35+
env_vars: {
36+
key: "MAVEN_MODULES"
37+
value: "java-aiplatform,java-compute"
38+
}

.kokoro/presubmit/graalvm-native-b-presubmit.cfg

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,8 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34+
35+
env_vars: {
36+
key: "MAVEN_MODULES"
37+
value: "java-aiplatform,java-compute"
38+
}

.kokoro/presubmit/graalvm-native-c-presubmit.cfg

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,3 +31,8 @@ env_vars: {
3131
key: "SECRET_MANAGER_KEYS"
3232
value: "cloud-java-ci-it-service-account, java-bigqueryconnection-samples-secrets"
3333
}
34+
35+
env_vars: {
36+
key: "MAVEN_MODULES"
37+
value: "java-aiplatform,java-compute"
38+
}

generation_config.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
gapic_generator_version: 2.68.0
2-
googleapis_commitish: 7e8511aad98db8480de0fd594c1cfd09a32b1cf3
2+
googleapis_commitish: 77c78e90aeeb1c4612e9ab772c90bc40748a7a19
33
libraries_bom_version: 26.78.0
44
libraries:
55
- api_shortname: accessapproval

java-backupdr/README.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,20 +42,20 @@ If you are using Maven without the BOM, add this to your dependencies:
4242
<dependency>
4343
<groupId>com.google.cloud</groupId>
4444
<artifactId>google-cloud-backupdr</artifactId>
45-
<version>0.46.0</version>
45+
<version>0.47.0</version>
4646
</dependency>
4747
```
4848

4949
If you are using Gradle without BOM, add this to your dependencies:
5050

5151
```Groovy
52-
implementation 'com.google.cloud:google-cloud-backupdr:0.46.0'
52+
implementation 'com.google.cloud:google-cloud-backupdr:0.47.0'
5353
```
5454

5555
If you are using SBT, add this to your dependencies:
5656

5757
```Scala
58-
libraryDependencies += "com.google.cloud" % "google-cloud-backupdr" % "0.46.0"
58+
libraryDependencies += "com.google.cloud" % "google-cloud-backupdr" % "0.47.0"
5959
```
6060

6161
## Authentication
@@ -175,7 +175,7 @@ Java is a registered trademark of Oracle and/or its affiliates.
175175
[javadocs]: https://cloud.google.com/java/docs/reference/google-cloud-backupdr/latest/overview
176176
[stability-image]: https://img.shields.io/badge/stability-stable-green
177177
[maven-version-image]: https://img.shields.io/maven-central/v/com.google.cloud/google-cloud-backupdr.svg
178-
[maven-version-link]: https://central.sonatype.com/artifact/com.google.cloud/google-cloud-backupdr/0.46.0
178+
[maven-version-link]: https://central.sonatype.com/artifact/com.google.cloud/google-cloud-backupdr/0.47.0
179179
[authentication]: https://github.com/googleapis/google-cloud-java#authentication
180180
[auth-scopes]: https://developers.google.com/identity/protocols/oauth2/scopes
181181
[predefined-iam-roles]: https://cloud.google.com/iam/docs/understanding-roles#predefined_roles

java-backupdr/google-cloud-backupdr/src/main/java/com/google/cloud/backupdr/v1/stub/BackupDRStubSettings.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1770,6 +1770,7 @@ protected LibraryMetadata getLibraryMetadata() {
17701770
return LibraryMetadata.newBuilder()
17711771
.setArtifactName("com.google.cloud:google-cloud-backupdr")
17721772
.setRepository("googleapis/google-cloud-java")
1773+
.setVersion(Version.VERSION)
17731774
.build();
17741775
}
17751776

java-backupdr/google-cloud-backupdr/src/main/java/com/google/cloud/backupdr/v1/stub/BackupDrProtectionSummaryStubSettings.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -424,6 +424,7 @@ protected LibraryMetadata getLibraryMetadata() {
424424
return LibraryMetadata.newBuilder()
425425
.setArtifactName("com.google.cloud:google-cloud-backupdr")
426426
.setRepository("googleapis/google-cloud-java")
427+
.setVersion(Version.VERSION)
427428
.build();
428429
}
429430

0 commit comments

Comments
 (0)