-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Create staging playground deploy workflow and other updates #11079
Conversation
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
…hery into cpepper96/11042-ci
host: ${{ secrets.METAL_SERVER3 }} | ||
username: root | ||
key: ${{ secrets.METAL_SSH_KEY }} | ||
script: | |
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.
It'd be great if we're able to use mesheryctl system update
.
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.
Not a requirement, but ideal to reuse Meshery's tooling.
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.
IMO, using mesheryctl will be ideal and we should start doing that.
It also aligns with the ongoing effort of using Meshery to also provision its remote provider instead of helm.
It also helps identifies where is the pain point or bugs in the flow.
Eventually, we should use mesheryctl for deploying in prod as well.
Signed-off-by: Jerod Culpepper <cpepper96@gmail.com>
Updated to use mesheryctl @leecalcote @MUzairS15 |
Good deal. I think that this command is all that we need. We'll find out shortly. 😉 |
Notes for Reviewers
This PR fixes #11042
secrets.METAL_SERVER1
actions/checkout@v4
in the edge release workflowSigned commits