Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Network failure test #468

Merged
merged 1 commit into from Nov 17, 2017
Merged

Conversation

mponomar
Copy link
Contributor

Test Comdb2 with 2 redundant networks where one fails.

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: Success ✓.

Detail | Logs


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

@adizaimi
Copy link

This is great. Can we please have some minimal comments in a README in that directory on what is needed and how to run (given that is different from the other tests).
Thanks.

@riverszhang89
Copy link
Contributor

Will change roborivers accordingly.

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: Success ✓.

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
netloss

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: 2/151 tests failed ⚠.

The first 10 failing tests are:
netloss
sc_blob_update

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: Success ✓.

The first 10 failing tests are:
netloss

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

@riverszhang89
Copy link
Contributor

Keep getting kernel crash from the test.. Per moby/moby#5618, it is a kernel bug and is fixed as of 4.12. Will upgrade roborivers in a little bit.

@riverszhang89
Copy link
Contributor

Okay now docker service won't come up under the new kernel.

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: 1/151 tests failed ⚠.

The first 10 failing tests are:
netloss

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

@riverszhang89
Copy link
Contributor

Finally. moby/moby#15651.

@riverszhang89
Copy link
Contributor

Upgrading kernel to 4.9 seems to fix the crash - I've run the test a dozen times and no crashes so far. However the test doesn't always pass..

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: 2/151 tests failed ⚠.

The first 10 failing tests are:
pmux

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

Copy link

@roborivers roborivers left a comment

Choose a reason for hiding this comment

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

Coding style check: Success ✓.
Smoke testing: Success ✓.
Regression testing: 1/151 tests failed ⚠.

The first 10 failing tests are:
pmux

Detail | Log | Resubmit


Red Hat Enterprise Linux Server release 7.3 (Maipo)
gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-11)

@riverszhang89
Copy link
Contributor

@adizaimi @nirbhayc Thanks for the review!

@riverszhang89 riverszhang89 merged commit 2b017d6 into bloomberg:master Nov 17, 2017
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.

None yet

5 participants