-
Notifications
You must be signed in to change notification settings - Fork 19
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
Aws #280
Aws #280
Conversation
Add DEPLOYMENT_DEST configuration variable that specifies the location where the deployment will be generated. This defaults to VERAISON_ROOT (the install location of the deployment), but may be set to a different value, allowing the deployment to be created in an intermediate location before being moved into its final installation destination. This is useful when the native deployment is intended to be used as an input for an alternative deployment mechanism (e.g. a Debian package) rather than being used directly. Signed-off-by: Sergei Trofimov <[email protected]>
Up to now, the version reported by Veraison services was set during build to "commit-$commit" where $commit is the short hash of the HEAD commit during the build. Since 9509e74, we now have regular auto-tagging on the repo in the form "v0.0.yyMM". This should be included in the version. This commit adds scrips/get-veraison-version that generates a version of the form "$version+$commit" where $version is the version from tag sans the leading "v". It also allows overriding this version using VERAISON_BUILD_VERSION environment variable. The override may be useful when git is not available (e.g. when building form a tarball) or when wanting to encode additional information in the version (e.g. when creating a build for a specific demo). This script is then used to populate VERSION_FROM_GIT inside mk/common.mk which is then compiled into the built executables. Signed-off-by: Sergei Trofimov <[email protected]>
Allow installing Veraison services into the system services manager under systemd using -s option (the default is to use the user service manager). Signed-off-by: Sergei Trofimov <[email protected]>
Add "debian" deployment that generates a .deb package that can be installed on Debian and Ubuntu systems. Signed-off-by: Sergei Trofimov <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
An impressive set of commits. 👏
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor questions, but in general looks very good to me!
end-to-end/README.md
Outdated
@@ -4,7 +4,8 @@ All the instructions on this page refer to the Docker-based deployment. To use | |||
the native deployment, just substitute "docker" with "native" in all the | |||
commands given below. For example, `make native-deploy` instead of `make | |||
docker-deploy`, and `./end-to-end-native provision` instead of `./end-to-end-docker | |||
provision` | |||
provision`. The same applies to `./end-to-end-aws` with the proviso that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean the same applies to aws
meaning replace in all scripts with aws instead of using docker? It is not super clear whether the guidance is specific to ./end-to-end command or it follows from the previous text. Could you please clarify the wording?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated.
- Add scripts for deploying Veraison into existing AWS VPC/subnets. - Update end-to-end with aws version. Signed-off-by: Sergei Trofimov <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thanks for addressing all the comments!
AWS deployment.