Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

fix: update alert policies one at a time because (concurrent updates are not supported) #507

Merged
merged 5 commits into from
Dec 22, 2020
Merged

Conversation

syedashrafulla
Copy link
Contributor

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • [N/A] Appropriate docs were updated (if necessary)

Fixes #506 🦕

@syedashrafulla syedashrafulla requested a review from a team as a code owner December 1, 2020 04:07
@product-auto-label product-auto-label bot added the api: monitoring Issues related to the googleapis/nodejs-monitoring API. label Dec 1, 2020
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Dec 1, 2020
@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #507 (ee32860) into master (afa651f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #507   +/-   ##
=======================================
  Coverage   98.51%   98.51%           
=======================================
  Files          10       10           
  Lines       16338    16338           
  Branches      353      359    +6     
=======================================
  Hits        16096    16096           
  Misses        236      236           
  Partials        6        6           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update afa651f...ee32860. Read the comment docs.

@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 1, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 1, 2020
@bcoe
Copy link
Contributor

bcoe commented Dec 1, 2020

@syedashrafulla it looks like there are a couple linting issues to address, I kicked off the integration tests so you can see if things are working.

@syedashrafulla
Copy link
Contributor Author

@bcoe apologies about that, lint should now be cleaned.

@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 2, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 2, 2020
@syedashrafulla
Copy link
Contributor Author

I think all that's missing is now a review, but let me know if I should run a test.

@bcoe bcoe changed the title fix: update alert policies one at a time because concurrent updates are nots upported fix: update alert policies one at a time because (concurrent updates are not supported) Dec 22, 2020
@bcoe bcoe added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 22, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Dec 22, 2020
@bcoe
Copy link
Contributor

bcoe commented Dec 22, 2020

@syedashrafulla thank you for the contribution.

@bcoe bcoe merged commit 2bf75a2 into googleapis:master Dec 22, 2020
gcf-merge-on-green bot pushed a commit that referenced this pull request Jan 7, 2021
🤖 I have created a release \*beep\* \*boop\* 
---
### [2.1.5](https://www.github.com/googleapis/nodejs-monitoring/compare/v2.1.4...v2.1.5) (2021-01-07)


### Bug Fixes

* update alert policies one at a time because (concurrent updates are not supported) ([#507](https://www.github.com/googleapis/nodejs-monitoring/issues/507)) ([2bf75a2](https://www.github.com/googleapis/nodejs-monitoring/commit/2bf75a2294f82d3b75e0ee3d44c91b0b8e5ec358))
---


This PR was generated with [Release Please](https://github.com/googleapis/release-please).
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: monitoring Issues related to the googleapis/nodejs-monitoring API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

The test for samples/alerts.js fails due to "too many concurrent updates."
3 participants