Skip to content

Fix bug conflicts when containers start simultaneously#3092

Merged
wu-sheng merged 2 commits into
apache:masterfrom
kezhenxu94:bugfix/e2e-container-name
Jul 16, 2019
Merged

Fix bug conflicts when containers start simultaneously#3092
wu-sheng merged 2 commits into
apache:masterfrom
kezhenxu94:bugfix/e2e-container-name

Conversation

@kezhenxu94
Copy link
Copy Markdown
Member

Please answer these questions before submitting pull request

  • Why submit this pull request?
  • Bug fix
  • New feature provided
  • Improve performance

The e2e containers' names of single mode and cluster mode
would conflict when they start up simultaneously in Jenkins job.

The e2e containers' names of single mode and cluster mode
would conflict when they start up simultaneously in Jenkins job.
@kezhenxu94 kezhenxu94 added bug Something isn't working and you are sure it's a bug! test Test requirements about performance, feature or before release. labels Jul 16, 2019
@kezhenxu94 kezhenxu94 added this to the 6.3.0 milestone Jul 16, 2019
@kezhenxu94 kezhenxu94 requested a review from wu-sheng July 16, 2019 01:17
@kezhenxu94
Copy link
Copy Markdown
Member Author

kezhenxu94 commented Jul 16, 2019

The e2e cluster test Jenkins jobs failed occasionally due to two main reasons, one of which is container name conflicts, which should be fixed by this patch, the other reason is timing out, and is under investigation

Copy link
Copy Markdown
Member

@wu-sheng wu-sheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wu-sheng
Copy link
Copy Markdown
Member

/run e2e

@wu-sheng wu-sheng merged commit 673b0bb into apache:master Jul 16, 2019
@kezhenxu94 kezhenxu94 deleted the bugfix/e2e-container-name branch July 16, 2019 02:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working and you are sure it's a bug! test Test requirements about performance, feature or before release.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants