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

dockerize rover #719

Open
cyberhck opened this issue Aug 6, 2021 · 8 comments
Open

dockerize rover #719

cyberhck opened this issue Aug 6, 2021 · 8 comments
Labels
feature 🎉 new commands, flags, functionality, and improved error messages triage issues and PRs that need to be triaged

Comments

@cyberhck
Copy link

cyberhck commented Aug 6, 2021

Dockerize Rover and publish to docker registry

As the title suggests, it'd be quite nice if apollo team could build official rover docker CLI and publish to docker.

While anyone is able to do this, I think publishing from rover's CI pipeline would mean it'll automatically get updates when there's a new release in rover.

Add details like:

  • On every merge to master it publishes docker with a "master" tag, on actual tag release, it tags the docker image with that tag. In both cases it's available on docker hub.
  • Same as rover CLI
  • I'm sure there are people who dockerized this for their needs.
  • I want to run this on my post-sync hook when my deployment passes. I don't want to do this from CI because there'll be a gap between CI pushing schema and service getting deployed (also what if the deployment fails),
@cyberhck cyberhck added feature 🎉 new commands, flags, functionality, and improved error messages triage issues and PRs that need to be triaged labels Aug 6, 2021
@cyberhck
Copy link
Author

cyberhck commented Aug 6, 2021

we'll probably need different variant:

  • distroless
  • alpine
  • scratch
    being 3 main ones.

@prasek
Copy link

prasek commented Aug 9, 2021

@cyberhck this sounds very similar to apollographql/supergraph-demo-k8s-graph-ops#50

Run auto publish subgraph schema in an Argo Rollout postPromotionAnalysis job that runs rover in a docker container.

Can you provide some more context on this:

I want to run this on my post-sync hook when my deployment passes

Is this https://argoproj.github.io/argo-cd/user-guide/resource_hooks/?

@cyberhck
Copy link
Author

cyberhck commented Aug 9, 2021

I'm just asking for an official docker image, and in case someone is curious why, I've mentioned the reason.

While we can publish docker ourselves, having apollo's ci do that is better

@cyberhck
Copy link
Author

would you be open to a PR? I'd be okay with writing a Dockerfile and a github workflow to dockerize and push to docker hub if you'd prefer.

@cyberhck
Copy link
Author

also, where will it release? docker hub? github registry? I'll need someone to setup secrets

@carlocorradini
Copy link

carlocorradini commented Apr 5, 2022

Any update on this? It could be super useful...

We can dockerize it as Apollo Router and publish it on GitHub Packages.

What do you think? 🤯🥳

@carlocorradini carlocorradini mentioned this issue Apr 6, 2022
3 tasks
@cyberhck
Copy link
Author

cyberhck commented Apr 11, 2022

Will Apollo Router be able to do everything that rover does?

@olivernybroe
Copy link

While waiting for this feature, we created a small wrapper for our needs (really low effort added) 👍

We only use it for publishing the subgraph as a job in k8s.
You can find our docker wrapper here
https://hub.docker.com/repository/docker/worksome/rover/general
https://github.com/worksome/rover

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 🎉 new commands, flags, functionality, and improved error messages triage issues and PRs that need to be triaged
Projects
None yet
Development

No branches or pull requests

4 participants