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

[tests] lower flakiness of test_no_parallelism #2690

Merged
merged 1 commit into from
Jul 20, 2016

Conversation

degemer
Copy link
Member

@degemer degemer commented Jul 18, 2016

No description provided.

@masci
Copy link
Contributor

masci commented Jul 18, 2016

Can we try to re-enable the test under Win?

@degemer degemer force-pushed the quentin/flaky-no-paralellism branch from 04f9a0b to 77bc340 Compare July 18, 2016 18:14
@degemer
Copy link
Member Author

degemer commented Jul 18, 2016

@masci Trying it now. It is still relying on time.sleep though.

@degemer
Copy link
Member Author

degemer commented Jul 18, 2016

I restarted it twice on AppVeyor (so in total 6 runs) and it didn't fail on Windows, so I think we can let it enabled.

@degemer degemer merged commit c76ab39 into master Jul 20, 2016
@degemer degemer deleted the quentin/flaky-no-paralellism branch July 20, 2016 15:43
truthbk pushed a commit that referenced this pull request Jul 20, 2016
[tests] fixing for new dd_integrations path, instead of 3rd-party.
degemer added a commit that referenced this pull request Jul 28, 2016
* master: (119 commits)
  [core] remove noisy logs (#2715)
  run multiple instances of pylint (#2716)
  [packaging] Release 5.8.5 (#2712)
  [changelog][5.8.5] Add notes on packaging changes (#2710)
  [changelog] Update 5.8.5 with python upgrade
  be more specific when logging ssh errors (#2708)
  [marathon] allow base_url with path (#2620)
  [changelog] Update 5.8.5
  add issue and pr templates
  [sdk] minor tweaks - sdk env detection, check location (#2694)
  [http_check] log exceptions 🔊
  use 0.0.0.0 as server address when non_local_traffic is passed (#2691)
  [elastic] tag stats metric with the node name 🏷 (#2696)
  [openstack] moving proxy logic to AgentCheck, for maintainability. Fixing typos.
  [changelog] 5.8.5 draft
  [tests] lower flakiness of test_no_parallelism (#2690)
  [core] don't use docker hostname if it's a EC2 one (#2661)
  [haproxy] Fix `KeyError` when an unknown status is found (#2681)
  [changelog] Fix md links
  [jenkins] Deprecate check (#2688)
  ...
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.

None yet

2 participants