Skip to content
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

Crystalin add standalone docker image #48

Merged
merged 5 commits into from
Oct 7, 2020

Conversation

crystalin
Copy link
Collaborator

@crystalin crystalin commented Oct 6, 2020

This PR adds the Docker image for the standalone version of moonbeam.
The parachain has been changed to moonbase-standalone (This will require some upgrade on the documentation @albertov19 )

It also removes the test workflow which is now executed inside the build part of the release workflow. We don't need to build the project 2 times (for release and test) and also it should not release a version if the tests are not passing.

@JoshOrndorff
Copy link
Contributor

I think these are good changes that will make life better for us.

It seems like a lo of code to build and push the docker image. I wonder if we've considered this action https://github.com/docker/build-push-action

I used it successfully in the the cumulus workshop.

@crystalin
Copy link
Collaborator Author

crystalin commented Oct 7, 2020

Isn't build-push-action already what is used in the workflow ?
https://github.com/PureStake/moonbeam/blob/crystalin-release-standalone/.github/workflows/release.yml#L176

@crystalin crystalin merged commit c83c8ce into master Oct 7, 2020
@JoshOrndorff
Copy link
Contributor

Yes, you're right. I got confused by the lines of bash script before it. I guess they are jsut extracting information to use as tags or labels.

@crystalin crystalin deleted the crystalin-release-standalone branch November 19, 2020 15:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants