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
ec2system: change handling of default ec2boot binaries #18
Conversation
Instead of hardcoding ec2boot in the config (which materializes to, e.g., $HOME/.bigslice/config), we set the default to an empty string, and fill in the default binary associated with the current release. We also rewrite binaries with the official prefix to match the current version.
Co-Authored-By: jcharum <jcharumilind@grailbio.com>
Co-Authored-By: jcharum <jcharumilind@grailbio.com>
Any idea if/how the build failure is related? |
No, don't think so: this error seems to pop up regularly on Travis-CI. |
@jcharum: Somehow these changes do not appear to made it into |
@jcharum: figured it out. since this was based on |
Instead of hardcoding ec2boot in the config (which materializes to, e.g., $HOME/.bigslice/config), we set the default to an empty string, and fill in the default binary associated with the current release. We also rewrite binaries with the official prefix to match the current version.
Repaired by #19. |
Instead of hardcoding ec2boot in the config (which materializes to, e.g., $HOME/.bigslice/config), we set the default to an empty string, and fill in the default binary associated with the current release. We also rewrite binaries with the official prefix to match the current version.
Sorry I missed that. I'll make sure to double-check that next time. Were we supposed to have rebased #18 onto master? It seems nonobvious in the UI how to do that, but I'll work it out next time. |
Instead of hardcoding ec2boot in the config (which materializes to, e.g., $HOME/.bigslice/config), we set the default to an empty string, and fill in the default binary associated with the current release.
We also rewrite binaries with the official prefix to match the current version.