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

Add a workflow for CI #1695

Merged
merged 1 commit into from
Apr 7, 2022
Merged

Add a workflow for CI #1695

merged 1 commit into from
Apr 7, 2022

Conversation

mscherer
Copy link
Contributor

@mscherer mscherer commented Apr 5, 2022

The workflow try to build the docker image on PR, and push
a image on each commit, and stable version when there is a release

Signed-off-by: Michael Scherer misc@redhat.com

@mscherer
Copy link
Contributor Author

mscherer commented Apr 5, 2022

It seems that the CI fail because the build fail (a issue on npm that I am not sure to be able to debug).

@@ -0,0 +1,43 @@
name: Build Docker images
Copy link
Member

Choose a reason for hiding this comment

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

@mscherer : The Docker build in the test is failing. Which may mean something or it may not.

The command '/bin/sh -c npm run build' returned a non-zero code: 1
```

It looks like its being built with React? I know we previously were using NPM, setting the version to the latest of 12.22.x .. 

Copy link
Member

Choose a reason for hiding this comment

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

@mscherer : Maybe I should've seen you already noticed that.

Copy link
Member

Choose a reason for hiding this comment

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

OK... I think I see what is happening.

@mscherer
Copy link
Contributor Author

mscherer commented Apr 7, 2022

I fixed a bug on the naming of the container, and I ignore the frontend for now

@mscherer mscherer force-pushed the add_ci branch 2 times, most recently from 1922aa7 to 6cd5f03 Compare April 7, 2022 09:15
The workflow try to build the docker image on PR, and push
a image on each commit, and stable version when there is a release

Signed-off-by: Michael Scherer <misc@redhat.com>

Ignore the frontend for now
@sgoggins sgoggins merged commit d45e2a5 into chaoss:main Apr 7, 2022
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