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.

Sign up for GitHub

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

Jump to bottom

PR broke CI #125

Closed
ryjones opened this issue Apr 7, 2020 · 4 comments
Closed

PR broke CI #125

ryjones opened this issue Apr 7, 2020 · 4 comments
Assignees
@yeasy

Comments

@ryjones
Copy link
Member

ryjones commented Apr 7, 2020

PR #120 was merged, and there are two repos that have not built since then:

https://hub.docker.com/repository/docker/hyperledger/cello-api-engine/builds
https://hub.docker.com/repository/docker/hyperledger/cello-dashboard/builds
@yeasy
Copy link
Member

yeasy commented Apr 17, 2020 •
edited
Loading

@ry, do you know the memory of the building env? It is reported the building will fail with a less-than-1 GB memory server.

@ryjones
Copy link
Member Author

ryjones commented Apr 17, 2020

The following dockerhub accounts have admin permissions:

  • yeasy
  • lordgoodman
  • email4tong
  • hightall

You can take a look in hub.docker.com at the settings and logs.

@yeasy yeasy mentioned this issue Apr 17, 2020
Merged
@ryjones
Copy link
Member Author

ryjones commented Apr 24, 2020

The build for cello-api-engine is fixed. However, cello-dashboard is still failing. Here are the errors right before the build fails. I don't think this is actually a networking issue as it's been a persistent failure.

@ryjones
Copy link
Member Author

ryjones commented Apr 24, 2020

@litong01 If you want to move all of this to Azure pipelines (all of the docker hub native stuff) I'll help

yeasy added a commit that referenced this issue Apr 27, 2020
@yeasy
Merge pull request #134 from litong01/master
32a915a 
Fix issue #125

Update dockerfile of building dashboard
litong01 added a commit that referenced this issue Apr 28, 2020
@litong01
Merge pull request #137 from litong01/master
75bd833 
More fixes to the build broken issue #125
@ryjones ryjones closed this as completed Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yeasy yeasy

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ryjones @yeasy