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

Use Hostname As ID #205

Closed
wants to merge 1 commit into from
Closed

Conversation

lkiesow
Copy link
Contributor

@lkiesow lkiesow commented May 13, 2020

When managing a larger cluster and wanting to take down specific hosts
or chunks, is it quite inconvenient to find out the specific IDs handed
out to servers in scalelite. Being able to use the domain names instead
would be much nicer.

While it is conceivable that multiple servers share the same domain
name, this is actually pretty unlikely. Usually, the domain name should
be unique and work just fine as identifier.

This patch switches to using the domain name as an identifier by default.
If required for any reason, people can still switch to using UUIDs
though. Old IDs will, of course, stay valid.


Example:

% ./bin/rake 'servers' | grep -A1 id:

id: bbb03.lkiesow.io
        url: https://bbb03.lkiesow.io/
--
id: bbb02.lkiesow.io
        url: https://bbb02.lkiesow.io/
--
id: bbb01.lkiesow.io
        url: https://bbb01.lkiesow.io/

When managing a larger cluster and wanting to take down specific hosts
or chunks, is it quite inconvenient to find out the specific IDs handed
out to servers in scalelite. Being able to use the domain names instead
would be much nicer.

While it is conceivable that multiple servers share the same domain
name, this is actually pretty unlikely. Usually, the domain name should
be unique and work just fine as identifier.

This patch switches to using the domain name as identifier by default.
If required for any reason, people can still switch to using UUIDs
though. Old IDs will, of course, stay valid.
lkiesow added a commit to lkiesow/scalelite that referenced this pull request Jul 26, 2020
This patch allows all commands to accept not only an exact ID match but
also search patterns for matching URLs. This allows for a much easier
control of groups of servers instead of having to find out the servers
identifier every time and then handle each server indivifually.

For example, you can run the following command to enbale multiple
servers:

    ./bin/rake 'servers:enable[bbb0\[1-5\].lkiesow.io]'

You can still use exact identifier matches though and the identifiers
will always be checked before checking the URL. Hence, you still can use
a command like:

    ./bin/rake 'servers:disable[a18201ec-f090-4bff-aaac-d0f5ab003f42]'

This closes blindsidenetworks#205 which was trying to solve the same problem by using the
host manes as server identifier instead. That shouldn't be necessary any
longer when this patch is applied.
lkiesow added a commit to lkiesow/scalelite that referenced this pull request Jul 26, 2020
This patch allows all commands to accept not only an exact ID match but
also search patterns for matching URLs. This allows for a much easier
control of groups of servers instead of having to find out the servers
identifier every time and then handle each server indivifually.

For example, you can run the following command to enbale multiple
servers:

    ./bin/rake 'servers:enable[bbb0\[1-5\].lkiesow.io]'

You can still use exact identifier matches though and the identifiers
will always be checked before checking the URL. Hence, you still can use
a command like:

    ./bin/rake 'servers:disable[a18201ec-f090-4bff-aaac-d0f5ab003f42]'

This closes blindsidenetworks#205 which was trying to solve the same problem by using the
host manes as server identifier instead. That shouldn't be necessary any
longer when this patch is applied.
@jfederico jfederico added the outdated Branch is outdated and locked for updates label Feb 1, 2021
@kepstin
Copy link
Member

kepstin commented Mar 1, 2021

Closing this in favour of #470

@kepstin kepstin closed this Mar 1, 2021
lkiesow added a commit to lkiesow/scalelite that referenced this pull request Mar 1, 2021
This patch allows all commands to accept not only an exact ID match but
also search patterns for matching URLs. This allows for a much easier
control of groups of servers instead of having to find out the servers
identifier every time and then handle each server indivifually.

For example, you can run the following command to enbale multiple
servers:

    ./bin/rake 'servers:enable[bbb0\[1-5\].lkiesow.io]'

You can still use exact identifier matches though and the identifiers
will always be checked before checking the URL. Hence, you still can use
a command like:

    ./bin/rake 'servers:disable[a18201ec-f090-4bff-aaac-d0f5ab003f42]'

This closes blindsidenetworks#205 which was trying to solve the same problem by using the
host manes as server identifier instead. That shouldn't be necessary any
longer when this patch is applied.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
outdated Branch is outdated and locked for updates
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants