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

ASG Suspend #37

Open
wants to merge 155 commits into
base: master
Choose a base branch
from
Open

ASG Suspend #37

wants to merge 155 commits into from

Conversation

dyaxley
Copy link

@dyaxley dyaxley commented Nov 20, 2019 •
edited by petercrotty
Loading

Added features:

  • ASGs tagged with hammertime:asgsuspend will have all processes suspended and ec2 instances shutdown instead of terminated. For Kaos SORGU and SORAU, this tag is applied via a daily scheduled buildkite build here https://github.com/nib-group/rqp-whics-managed-scripts/blob/master/.buildkite/scripts/tag-asgs.sh. If other accounts do not have this tagging in place then they should be unaffected by this feature.
  • Upgrade core components, including node 12 ---> 14
  • Improved re-tries to prevent throttling errors

This PR has been tested successfully in the Kaos SORGU and SORAU accounts for some time.

dyaxley and others added 30 commits September 26, 2019 22:13
@dyaxley
first cut
d8a0a5a
@dyaxley
first cut
9282775
@dyaxley
Test out asg suspend
f61fd37
@dyaxley
Test out asg suspend
582baa2
@dyaxley
rejig functions
5106c11
@dyaxley
update func name
871fd8d
@dyaxley
dryrun
cf937b2
@dyaxley
test suspending
c43fb90
@dyaxley
added suspend and resume to iam
37ca8fb
@dyaxley
instances as array
a4e2625
@dyaxley
update array
c3b1c78
@dyaxley
fix the resume
26da142
@dyaxley
resume?
b832da4
@dyaxley
log inst
d107aaf
@dyaxley
fix resume issue in start.js
79f8203
@dyaxley
logg everything
7145d6d
@dyaxley
update iam to include kms create grant to allow lamda to start encryp…
d55fa42 
…ted ec2
@dyaxley
minor update
896dfec
@dyaxley
Update to suspend all processes
9ce3044
@dyaxley
Added weekend off param
cf3781d
@dyaxley
default to false
796418d
@dyaxley
default to false
4f0e406
@dyaxley
hmmm
4b15335
@dyaxley
clean up function
10e07bd
@dyaxley
fix cron days
3049a2d
@dyaxley
temp fix
7d93b0c
@dyaxley
add comment
0be98ec
@dyaxley
update function
dc3d302
@dyaxley
update tests
7a7f111
@dyaxley
updates to weekendsOff logic to support timezones & updates to npm pa…
8bc8690 
…ckages
katevucmd and others added 30 commits July 10, 2023 23:36
@katevucmd
test
58c3d01
@katevucmd
promise
df28c41
@katevucmd
promiseeeee
801f8cd
@katevucmd
yay it works
584ee9a
@katevucmd
update some comment
debee1a
@katevucmd
clean
93e894c
@katevucmd
clean
c623ab8
@katevucmd
clean
3f1b03a
@katevucmd
clean
e7d8647
@katevucmd
Merge pull request #69 from nib-health-funds/W2C-546
087a709 
W2C-546 Fix loop for resume and untag asg
@katevucmd
add retry
c4e6145
@katevucmd
set up for testing
2a4b2f4
@katevucmd
reduce waiting time
d6e1a5d
@katevucmd
import retry
7e6b1f2
@katevucmd
fix path for import
2308ffd
@katevucmd
import retry
38cfa33
@katevucmd
test promise again
78c7ea5
@katevucmd
fix startInstances?
37ac38b
@katevucmd
add return
4a65aef
@katevucmd
remove wait
8227939
@katevucmd
get list of instances for asg
50eb5a7
@katevucmd
get list of instances for asg
63855c6
@katevucmd
add some check?
fa53102
@katevucmd
start everything
b2085da
@katevucmd
mock value for testing
7d1525f
@katevucmd
remove some tag
511aec8
@katevucmd
remove mock test
1175c8c
@katevucmd
reset waiting 4minutes
570097c
@katevucmd
Merge pull request #70 from nib-health-funds/W2C-680
1bdeaad 
W2C-680 Throttling for startInstances API
@petercrotty
Merge branch 'master' into asgSuspend
7a33352
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ramesius ramesius ramesius left review comments

@krutisfood krutisfood krutisfood left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@dyaxley @codecov-io @codecov-commenter @ramesius @krutisfood @petercrotty @katevucmd