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.

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

sandbox-docker-compose broken by upstream changes #508

Merged
merged 1 commit into from
Nov 1, 2021
Merged

sandbox-docker-compose broken by upstream changes #508

merged 1 commit into from
Nov 1, 2021

Conversation

Kiser360
Copy link
Contributor

While upgrading to the latest Algod 3.x.x, Docker was unable to build a new algod image using the provided make sandbox-up. After investigation I found that a recent change on algorand/sandbox c3fa3acce89747b8a864ceb32890cca22a319b4d broke compatibility with algo-builder's provided scripts. This pull request will resolve the issue by removing the no longer necessary /tmp portion of the 'TEMPLATE' variable's path

Proposed Changes

Potential followups

Copy link
Member

@ratik21 ratik21 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR, i will add sandbox-setup to our workflows as well.

@robert-zaremba robert-zaremba merged commit c0df5a4 into scale-it:master Nov 1, 2021
ratik21 pushed a commit that referenced this pull request Dec 23, 2021
* /tmp no longer necessary, added to the upstream algorand/sandbox (#508)

* Bump nokogiri from 1.11.2 to 1.12.5 in /docs

Bumps [nokogiri](https://github.com/sparklemotion/nokogiri) from 1.11.2 to 1.12.5.
- [Release notes](https://github.com/sparklemotion/nokogiri/releases)
- [Changelog](https://github.com/sparklemotion/nokogiri/blob/main/CHANGELOG.md)
- [Commits](sparklemotion/nokogiri@v1.11.2...v1.12.5)

---
updated-dependencies:
- dependency-name: nokogiri
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: Devin Kiser <[email protected]>
Co-authored-by: Robert Zaremba <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
ratik21 pushed a commit that referenced this pull request Dec 23, 2021
* /tmp no longer necessary, added to the upstream algorand/sandbox (#508)

* Bump rexml from 3.2.4 to 3.2.5 in /docs

Bumps [rexml](https://github.com/ruby/rexml) from 3.2.4 to 3.2.5.
- [Release notes](https://github.com/ruby/rexml/releases)
- [Changelog](https://github.com/ruby/rexml/blob/master/NEWS.md)
- [Commits](ruby/rexml@v3.2.4...v3.2.5)

---
updated-dependencies:
- dependency-name: rexml
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: Devin Kiser <[email protected]>
Co-authored-by: Robert Zaremba <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
ratik21 pushed a commit that referenced this pull request Dec 23, 2021
* /tmp no longer necessary, added to the upstream algorand/sandbox (#508)

* Bump addressable from 2.7.0 to 2.8.0 in /docs

Bumps [addressable](https://github.com/sporkmonger/addressable) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/sporkmonger/addressable/releases)
- [Changelog](https://github.com/sporkmonger/addressable/blob/main/CHANGELOG.md)
- [Commits](sporkmonger/addressable@addressable-2.7.0...addressable-2.8.0)

---
updated-dependencies:
- dependency-name: addressable
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: Devin Kiser <[email protected]>
Co-authored-by: Robert Zaremba <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants