Add ability to run shell before and after deployment - #391
Merged
nikita-petko merged 17 commits intoAug 13, 2026
Conversation
…ment deploy.yml: ~ Bump CFA to v10 ~ Add step to read deployment scripts either as abs or relative paths or raw inline scripts. ~ Run scripts with deployment phase where before scripts will fail the deployment on error, and after scripts won't
#!skip-build!# Add missing js-yaml!
#!skip-build!# ~ Instead of returning file paths (uhhh because job A and B not on same machine :/) use the raw script ~ Write script to file output ~ Use correct component name
#388(@nikita-petko): RSISRIOFAS #!components: grid-bot-recovery Update build.yml: ~ Add lib/* Update .vscode: ~ Add taks and launch configurations for grid-bot-recovery Update grid-bot component: ~ Add gRPC service endpoint. Update grid-bot-recovery: ~ Move shard count to a once flag, to run initialization once on startup. Please Note: Only interactions will actually respond here for some reason.
~ Allow updates to happen manually instead of always occuring whenever Set() is called.
#!components: grid-bot, grid-bot-recovery ~ Make CheckHealth use google.protobuf.Empty instead ~ Add SetStatus RPC
#!components: grid-bot, grid-bot-recovery deploy.yml: ~ Pass env to deploy-nomad-jobs step just in case it needs it grid-bot component: ~ Add set-maintenance-status before script that will set the maintenance status ~ Update gRPC host.
#!components: grid-bot-recovery workflows: ~ Update build.yml to fully allow multiple components ~ Pass env to read-deployment-scripts grid-bot component: ~ Fix syntax with pre-env expression.
#!components: grid-bot grpc: ~ Add correct settings to service provider. utility: ~ Update MSLogger to also log exception if not null. grid-bot component: ~ Pre-script error out when either grpcurl not downloaded, or proto-url
…oyment Signed-off-by: Nikita Petko <petko@vmminfra.net>
~ Add import-path ~ Set maintenance_message ~ Sleep for 5 seconds to ensure propagation
Sync get-component-names with rbxinfra/bedev2-service-template
nikita-petko
added
kind: feature
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
~ Bump CFA to v10
~ Add step to read deployment scripts either as abs or relative paths or raw inline scripts. ~ Run scripts with deployment phase where before scripts will fail the deployment on error, and after scripts won't