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

Possibility to define Service name #101

Open
ayozemr opened this issue Oct 18, 2017 · 7 comments
Open

Possibility to define Service name #101

ayozemr opened this issue Oct 18, 2017 · 7 comments

Comments

@ayozemr
Copy link

ayozemr commented Oct 18, 2017

Actually I see that Service name is calculated with application + target.name, so if I want to use this tool I have to remake my clusters and services to match this naming protocol? Would be reasonable to be able to specify service_name the same way I specify cluster, so I can tell the app "Deploy the new version of the service X on cluster Y"

Maybe I have understood something wrong.

Thanks for your time!

@camertron
Copy link
Contributor

Hey @ayozemr, apologies for not responding in a timely manner. Yes, broadside asks that you use the application + target.name convention, but as I recall that was an arbitrary choice on our part and not an ECS requirement. It is absolutely reasonable to be able to provide a custom service name that doesn't hold with the convention, and we'd welcome a pull request to that effect.

@slpsys slpsys changed the title Posibility to define Service name Possibility to define Service name Nov 1, 2017
@slpsys
Copy link
Contributor

slpsys commented Nov 1, 2017

@ayozemr to back up @camertron, I think we'd definitely be OK with this change. Internally, we're moving many things from ECS to self-managed Kubernetes, and it's unclear how much of a role Broadside will play in that ecosystem. We have a number of applications under active development whose only deploy environment is ECS via Broadside, but this isn't a particular feature we've needed in the past (though I agree, this would be especially useful for e.g. staging vs. production).

We'd definitely welcome a patch via Pull Request, and would be happy to review.

@ayozemr
Copy link
Author

ayozemr commented Nov 3, 2017

Hello! Thanks for your time. I will check it with my team, but we work with Python and JS and we may do a bad job with Ruby :-(

If we finally code it, I will tell you for review :-)

Thanks again and good job with the tool.

@slpsys
Copy link
Contributor

slpsys commented Nov 5, 2017

👍

@apurvis
Copy link
Contributor

apurvis commented Nov 6, 2017

just a bit of background - the main reason the service naming works like this is to ensure that services (and task definitions) are easily recognizable by their naming, and you don't have to juggle a bunch of different configurations that are not obviously linked to each other (which often happens when developers are lazy about naming their stuff consistently).

so to answer the original question, yes, you are encouraged to hand over the naming of your services and task definitions to broadside. while it is probably annoying to transition over the naming the first time, i suspect it will save you a fair amount of headache in the medium to long term.

though I agree, this would be especially useful for e.g. staging vs. production

@slpsys this is why you can have different targets for staging and production... if you name your target staging_web then the service and task definition names will reflect that.

@slpsys
Copy link
Contributor

slpsys commented Nov 7, 2017

Yeah, I'll be honest, I hadn't really looked back on the code when I wrote that, and the last time I did it was before a refactor--it's not as trivial as I'd assumed it'd be based on the request.

@apurvis
Copy link
Contributor

apurvis commented Nov 7, 2017

if I want to use this tool I have to remake my clusters and services to match this naming protocol?

@ayozemr you shouldn't have to remake any clusters... it's just services (and task definitions) that broadside controls the naming of.

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

No branches or pull requests

4 participants