tests: ci: allow parallel npm installs #1596

stephenplusplus · 2016-09-15T02:15:27Z

No description provided.

coveralls · 2016-09-15T04:49:08Z

Coverage remained the same at 100.0% when pulling 6393317 on stephenplusplus:spp--ci-speed-up into 35573c3 on GoogleCloudPlatform:master.

stephenplusplus · 2016-09-15T12:08:49Z

@callmehiphop this has sped up our builds quite a bit!

Previously, each environment would require around ~15-17 minutes just to run npm installs, before running any tests. Now we've got it down to ~8-17 for the tests to complete.

.travis.yml

-  - 4
  - 0.12
+  - 4
+  - 6


coveralls · 2016-09-15T16:18:43Z

Coverage remained the same at 100.0% when pulling f4effa6 on stephenplusplus:spp--ci-speed-up into 35573c3 on GoogleCloudPlatform:master.

.appveyor.yml

+  - packages\resource\node_modules -> packages\resource\package.json
+  - packages\storage\node_modules -> packages\storage\package.json
+  - packages\translate\node_modules -> packages\translate\package.json
+  - packages\vision\node_modules -> packages\vision\package.json


coveralls · 2016-09-15T16:40:28Z

Coverage remained the same at 100.0% when pulling 0c9dd98 on stephenplusplus:spp--ci-speed-up into 35573c3 on GoogleCloudPlatform:master.

coveralls · 2016-09-15T19:28:32Z

Coverage remained the same at 100.0% when pulling 0aeafa6 on stephenplusplus:spp--ci-speed-up into 35573c3 on GoogleCloudPlatform:master.

coveralls · 2016-09-15T19:28:32Z

Coverage remained the same at 100.0% when pulling 0aeafa6 on stephenplusplus:spp--ci-speed-up into 35573c3 on GoogleCloudPlatform:master.

stephenplusplus · 2016-09-15T20:24:56Z

Even though it took 100 commits, this should be an improvement over what we have now. It might take more tweaking, but that's just another 100-commit PR away.

[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [protobufjs](https://protobufjs.github.io/protobuf.js/) ([source](https://togithub.com/protobufjs/protobuf.js)) | [`7.2.6` -> `7.3.0`](https://renovatebot.com/diffs/npm/protobufjs/7.2.6/7.3.0) | [![age](https://developer.mend.io/api/mc/badges/age/npm/protobufjs/7.3.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/protobufjs/7.3.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/protobufjs/7.2.6/7.3.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/protobufjs/7.2.6/7.3.0?slim=true)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>protobufjs/protobuf.js (protobufjs)</summary> ### [`v7.3.0`](https://togithub.com/protobufjs/protobuf.js/blob/HEAD/CHANGELOG.md#730-2024-05-10) [Compare Source](https://togithub.com/protobufjs/protobuf.js/compare/protobufjs-v7.2.6...protobufjs-v7.3.0) ##### Features - add handling for extension range options ([#1990](https://togithub.com/protobufjs/protobuf.js/issues/1990)) ([2d58011](https://togithub.com/protobufjs/protobuf.js/commit/2d58011cc0bc495c68ed70f5aad297deb1722378)) </details> --- ### Configuration 📅 **Schedule**: Branch creation - "after 9am and before 3pm" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR is behind base branch, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about these updates again. --- - [ ] If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/googleapis/gax-nodejs).

## Description This PR achieves the same objectives as https://togithub.com/googleapis/java-bigquery/pull/4010, but for Node as requested in [the planning sheet](https://docs.google.com/spreadsheets/d/1cHAw6cV8k_s40hw28C8STtlymu3MXEvTY6PaKgdgSGE/edit?resourcekey=0-hPyTawvew5ugN037XCfZXg&gid=0#gid=0). The idea is that for reading rows, the users should be able to access high precision values for timestamps if high precision values are being stored on the backend. ## Impact This PR follows a test driven development approach against the getRows method. Tests were written to evaluate what happens when getRows receives various input values for timestampOutputFormat and useInt64Timestamp. These tests revealed that not only are high precision values not being delivered to users for ISO8601_STRING return types, but also other bugs exist like calls hang on getRows calls that fail and conversion logic throughs errors for some calls that fetch rows. This PR fixes all the bugs and ensures the values with the right precision are delivered to users. This chart details the before code changes / after code changes results with impact highlighted in green: <img width="1575" height="383" alt="image" src="https://togithub.com/user-attachments/assets/75c1ff70-073c-477e-9363-8eff5be0aa5f" /> The highlighted green impact shows that conversion logic has been updated to avoid the 'cannot convert' errors and with this new logic changes are also applied to the BigQueryTimestamp class to maintain high precision on timestamp values returned to users. ## Testing New system tests to capture all useInt64Timestamp/timestampOutputFormat combinations for getRows calls.

tests: ci: see if eachLimit stops AppVeyor errors

41e1d5b

stephenplusplus added the testing label Sep 15, 2016

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 15, 2016

stephenplusplus added 5 commits September 14, 2016 22:19

try npm 2

2d49c28

try force

c55603a

try retries

f4e2c4b

up the parallel limit

c4312be

allow more time before restarting

6393317

stephenplusplus changed the title ~~tests: ci: see if eachLimit stops AppVeyor errors~~ tests: ci: allow parallel npm installs Sep 15, 2016

stephenplusplus assigned callmehiphop Sep 15, 2016

stephenplusplus commented Sep 15, 2016

View reviewed changes

.travis.yml

- 4

- 0.12

- 4

- 6

This comment was marked as spam.

Sign in to view

stephenplusplus added 12 commits September 15, 2016 08:43

cache node_modules in travis

ac84890

cache node_modules in appveyor

1c0fa73

fix appveyor formatting

62b7d70

fix appveyor formatting

6b58b93

fix appveyor formatting

9ca6bb2

get packages from manifest.json

8487bdc

limit parallel installs to 5

ab9b4d9

limit install attempts

7b9d1eb

add each modules node_modules directory to cache appveyor

e6a55ed

add specific cache dirs to travis

5ee8233

lower parallel limit

c50de24

learn basic OS differences

069d952

stephenplusplus unassigned callmehiphop Sep 15, 2016

stephenplusplus added the don't merge label Sep 15, 2016

add retries back in

1756aa6

stephenplusplus force-pushed the spp--ci-speed-up branch from d60ff8e to 1756aa6 Compare September 15, 2016 14:51

remove prop

f4effa6

allow more time for expiration date tests

0c9dd98

add cache-min

0aeafa6

callmehiphop reviewed Sep 15, 2016

View reviewed changes

stephenplusplus removed the don't merge label Sep 15, 2016

stephenplusplus assigned callmehiphop Sep 15, 2016

callmehiphop merged commit af8ca8d into googleapis:master Sep 19, 2016

tests: ci: allow parallel npm installs #1596

tests: ci: allow parallel npm installs #1596

Uh oh!

Conversation

stephenplusplus commented Sep 15, 2016

Uh oh!

coveralls commented Sep 15, 2016

Uh oh!

stephenplusplus commented Sep 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Sep 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Sep 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coveralls commented Sep 15, 2016 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

coveralls commented Sep 15, 2016

Uh oh!

stephenplusplus commented Sep 15, 2016

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

stephenplusplus commented Sep 15, 2016 •

edited

Loading

coveralls commented Sep 15, 2016 •

edited

Loading

coveralls commented Sep 15, 2016 •

edited

Loading

coveralls commented Sep 15, 2016 •

edited

Loading