Skip to content

Conversation

@mmajerni
Copy link
Contributor

Release Note

NONE

@squakez
Copy link
Contributor

squakez commented Jun 17, 2024

Backport to main as well if this is a general fix. Thanks!

@squakez
Copy link
Contributor

squakez commented Jun 17, 2024

❌ TestSyntheticIntegrationFromDeployment (1m21.39s)
      synthetic_test.go:123: 
          Timed out after 60.000s.
          Expected
              <v1.ConditionStatus>: False
          to equal
              <v1.ConditionStatus>: True

This one was failing. I've restarted the check but it does not look to be a flaky one, if you want to give a look while the second check attempt is running.

@mmajerni
Copy link
Contributor Author

TestSyntheticIntegrationFromDeployment is located in subfolder advanced and the change is made in cli subfolder so other tests except these in CLI folder should not be affected.

@squakez
Copy link
Contributor

squakez commented Jun 17, 2024

TestSyntheticIntegrationFromDeployment is located in subfolder advanced and the change is made in cli subfolder so other tests except these in CLI folder should not be affected.

True. I see the second attempt cleared. You can disregard such a comment, thanks.

@squakez squakez merged commit 6c22eb6 into apache:release-2.3.x Jun 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants