-
Notifications
You must be signed in to change notification settings - Fork 13
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
Deploy core services from tar #725
Conversation
…e-service-from-tar
Did you manage to setup a dev environment for deploying service from local computer? |
not yet created the tool for dev to deploy locally, this will be done in another PR |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
manual tests are ok
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not complete at all and doing in fast.
- what the problem dose it solve?
Second in order to deploy service from URL we need to have mesg.yaml then we donwload second mesg.yaml
and which one should we take?
It introduces egg and chicken problem and no solution to it. What if mesg.yaml will have url and inside new service it will be diffretn url etc....
For me this PR need to rethink more corner cases
|
||
return grpc.New(c.Server.Address, a), nil | ||
} | ||
|
||
func deployCoreServices(c *config.Config, api *api.API) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This logic shouldn't be in main.go
the url point directly to the source of the service, not to a mesg.yml |
But the source will have proabably mesg.yml. If not then how we want to keep files:
So this simple change will have effect on how the developers distrubute services and sources. |
I don't understand your point, why do we need the yaml and the endpoint ? the yaml is inside the tarbal that will be downloaded with the tarball. Here we download the full tar (source + definitions) and deploy it when the core starts like that these services are available for the core and the core can use them. For now there is the |
@krhubert are you ok with this one, can we merge or is there still some part to clarify ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can merge this PR but another PR needs to be done to save the deployed services' sid and return it in a new API info
.
This PR is done her #737. |
This one depends on #724
Here we add services directly in the configs with a way to deploy them and when the core starts then all the services are automatically downloaded/deployed/started in order for the core to use them.