Skip to content

Conversation

@bvolpato
Copy link
Contributor

Parameterized / parallel tests have been suffering with flakes because of job name collision when only suffixed with timestamp:

com.google.api.client.googleapis.json.GoogleJsonResponseException: 409 Conflict
POST https://dataflow.googleapis.com/v1b3/projects/{PROJECT}/locations/us-central1/flexTemplates:launch
{
  "code": 409,
  "errors": [
    {
      "domain": "global",
      "message": "(78f666ea16e50f79): There is already an active job named test-jdbc-to-pubsub-20231115093811749. If you want to  submit a second job, try again by setting a different name.",
      "reason": "alreadyExists"
    }
  ],
  "message": "(78f666ea16e50f79): There is already an active job named test-jdbc-to-pubsub-20231115093811749. If you want to  submit a second job, try again by setting a different name.",
  "status": "ALREADY_EXISTS"
}

This will add more entropy and avoid that collision.

@github-actions
Copy link
Contributor

Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:

R: @damondouglas added as fallback since no labels match configuration

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@damondouglas
Copy link
Contributor

LGTM Thank you, @bvolpato

@damondouglas damondouglas merged commit 9b1040b into apache:master Nov 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants