Skip to content
This repository was archived by the owner on Oct 11, 2023. It is now read-only.

Conversation

@s-tuli
Copy link
Contributor

@s-tuli s-tuli commented Nov 5, 2019

Update bikes.yml by adding a new step that updates the user's branch name such that a valid namespace can be created using that. This also needs an update to the existing add-review-url custom action. There is a new input that is passed in to this action to construct the url which contains the valid child namespace. Note: there is another update needed such that we call the version2 of the custom github action: add-review-url@Releases/v2

…name such that a valid namespace can be created using that. This also needs an update to the existing add-review-url custom action. There is a new input that is passed in to this action to construct the url which contains the valid child namespace. Note: there is another update needed such that we call the version2 of the custom github action: add-review-url@Releases/v2
@s-tuli s-tuli requested a review from stepro November 5, 2019 09:00
…name specs and Release spec for the Helm install command in bikes.yml. Update some of the variable names. Tested E2E. This will need a Release/v2 branch which will contain node_modules as well.
…view-url such that the user specifies http vs https as needed. If not specified the protocol defaults to http. Update bikes.yml to add a step for installing helm for client if client and server version does not match. Add a step to create a valid release-name based on spec for the same by formatting the GITHUB_HEAD_REF environment variable.
…r installing Helm into in case the client version does not match the server version.
…ep: Create child namespaces and add Dev Spaces labels
@s-tuli s-tuli merged commit d4e9c8d into master Nov 8, 2019
@s-tuli s-tuli deleted the s-tuli/childnamespaceUpdate branch February 4, 2020 00:03
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants