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

Add bikes pr workflow and bikesharing workflow for GitHub Actions #165

Merged
merged 3 commits into from
Oct 3, 2019

Conversation

Greenie0506
Copy link
Contributor

@philon-msft Adding the two workflows.

@Greenie0506 Greenie0506 changed the title Add bikes pr workflow and biksharing workflow for GitHub Actions Add bikes pr workflow and bikesharing workflow for GitHub Actions Oct 2, 2019
container-registry-password: ${{ secrets.REGISTRY_PASSWORD }}
secret-name: ${{ secrets.IMAGE_PULL_SECRET }}

- name: Helm Instal PR
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Install" spelling

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oof, thanks

cd samples/BikeSharingApp/charts
helm init --client-only
helm dependency update
helm upgrade --install --namespace ${{ secrets.PARENT_SPACE }} bikesharing . --set "bikes.image.repository=${{ secrets.CONTAINER_REGISTRY }}/bikes, bikesharingweb.image.repository=${{ secrets.CONTAINER_REGISTRY }}/bikesharingweb, billing.image.repository=${{ secrets.CONTAINER_REGISTRY }}/billing, gateway.image.repository=${{ secrets.CONTAINER_REGISTRY }}/gateway, reservation.image.repository=${{ secrets.CONTAINER_REGISTRY }}/reservation, reservationengine.image.repository=${{ secrets.CONTAINER_REGISTRY }}/reservationengine, users.image.repository=${{ secrets.CONTAINER_REGISTRY }}/users, image.tag=$GITHUB_SHA"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would secrets.ROOT_SPACE be a better name than secrets.PARENT_SPACE?

@philon-msft philon-msft merged commit 5ce9013 into master Oct 3, 2019
@Greenie0506 Greenie0506 deleted the add_github_actions branch October 10, 2019 21:37
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.

2 participants