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

release: fix balena pushes with random sleep period #300

Merged
merged 5 commits into from
Dec 7, 2021
Merged

release: fix balena pushes with random sleep period #300

merged 5 commits into from
Dec 7, 2021

Conversation

shawaj
Copy link
Member

@shawaj shawaj commented Dec 7, 2021 •
edited
Loading

Issue

How

Push fix to production

Screenshots

References

#293

Checklist

  • Tests added
  • Cleaned up commit history (rebase!)
  • Documentation added
  • Thought about variable and method names

shawaj added 5 commits December 3, 2021 15:37
@shawaj
feat: add manual draft balena deploy action
5b27542 
to allow us to manually trigger draft builds

Relates-to: #293
@shawaj
feat: add draft deploy action for testnet
859b644 
add draft deploy

Relates-to: #293
@shawaj
Merge pull request #294 from NebraLtd/shawaj/draftdeploy
061b98f 
feat: add manual draft balena deploy action
@shawaj
fix: add random sleep period
04520d8 
Add random sleep period to work around balena API issues
@shawaj
Merge pull request #299 from NebraLtd/shawaj/sleep
024cd80 
fix: add random sleep period
@shawaj shawaj requested review from a team as code owners December 7, 2021 00:14
@shawaj
Copy link
Member Author

shawaj commented Dec 7, 2021

@marvinmarnold @KevinWassermann94 seems to have worked better with the random sleep period

https://github.com/NebraLtd/helium-miner-software/actions/runs/1547183401

@shawaj shawaj merged commit f364dc8 into production Dec 7, 2021
@shawaj
Copy link
Member Author

shawaj commented Dec 7, 2021

fingers crossed 🤞

@shawaj
Copy link
Member Author

shawaj commented Dec 7, 2021

@marvinmarnold @KevinWassermann94 hip hip hooray 🥳

@KevinWassermann94
Copy link
Contributor

Looking great!

shawaj added a commit that referenced this pull request Dec 7, 2021
@shawaj
fix: add multiple API tokens for balena
6e9453e 
add multiple balena API tokens to avoid rate limiting

Relates-to: #293 
Relates-to: #299 
Relates-to: #300
@shawaj shawaj mentioned this pull request Dec 7, 2021
Merged
4 tasks
shawaj added a commit that referenced this pull request Dec 7, 2021
@shawaj
fix: add multiple API tokens for balena
fb80d26 
- add multiple balena API tokens to avoid (or at least reduce the impact of) rate limiting
- add random sleep to testnet pushes

Relates-to: #293
Relates-to: #299
Relates-to: #300
shawaj added a commit that referenced this pull request Dec 7, 2021
@shawaj
fix: add multiple API tokens for balena
baafe64 
- add multiple balena API tokens to avoid (or at least reduce the impact of) rate limiting
- add random sleep to testnet pushes

Relates-to: #293
Relates-to: #299
Relates-to: #300
@shawaj shawaj mentioned this pull request Dec 7, 2021
Closed
@marvinmarnold marvinmarnold mentioned this pull request Dec 8, 2021
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KevinWassermann94 KevinWassermann94 KevinWassermann94 approved these changes

@marvinmarnold marvinmarnold marvinmarnold approved these changes

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

Successfully merging this pull request may close these issues.
3 participants
@shawaj @KevinWassermann94 @marvinmarnold