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.

Sign up for GitHub

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

Jump to bottom

Emit iterations metric as part of netext.NetTrail #1203

Merged
merged 5 commits into from
Nov 4, 2019
Merged

Emit iterations metric as part of netext.NetTrail #1203

merged 5 commits into from
Nov 4, 2019

Commits on Nov 4, 2019

  1. feat(cloud): emit iterations metric as part of netext.NetTrail 

    Closes: #1189
    Ivan Mirić committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    5de1b50 View commit details
    Browse the repository at this point in the history

  2. wip(js): emit iterations metric only for default function

    Ivan Mirić committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    34a7278 View commit details
    Browse the repository at this point in the history

  3. test(core): improve robustness of TestMinIterationDuration 

    Resolves: #1203 (comment)

Because the iterations metric is now emitted before the minIterationDuration
sleep is done--possibly what should've been the correct behavior--it
changes these test expectations. And since the previous test wasn't
checking for more than one minIterationDuration loop, it makes sense to
tweak the settings a bit to gather more data.
    Ivan Mirić committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    1fab0f1 View commit details
    Browse the repository at this point in the history

  4. test(core): add metrics emission test without minIterationDuration 

    Resolves: #1203 (comment)
    Ivan Mirić committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    5fae26c View commit details
    Browse the repository at this point in the history

  5. test(core): disable parallel TestMetricsEmission on Windows 

    Resolves: #1203 (comment)
    Ivan Mirić committed Nov 4, 2019
    Configuration menu
    Copy the full SHA
    9cd26e4 View commit details
    Browse the repository at this point in the history