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

Dynamic service names & domain #387

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Dynamic service names & domain #387

wants to merge 6 commits into from

Conversation

fubarhouse
Copy link
Collaborator

@fubarhouse fubarhouse commented May 30, 2022

I've been asked switch pygmy over to use pygmy.site as a default for users, so that's what this done.

It also contributes to the public strategy amazee.io have previously announced where the amazee.io branding is separated from their products - such as Lagoon.

This PR will switch amazeeio- container names to pygmy- whilst still using existing container names when the network amazeeio-network still exists. Custom container names via the pygmy,name label are still working as expected and are tested.

I've also added a piece to the readme so that users who need to run applications on docker.amazee.io subdomains can do so unhindered - in addition to the dynamic discovery. It would cause me problems as I often use the clean command instead of the down command - which does in fact delete the docker network.

@dasrecht
Copy link

FWIW - If you need a DNS change on pygmy.site feel free to reach out to me :) Just adding that here as I just stumbled over the Domain during our DNS cleanup

@fubarhouse
Copy link
Collaborator Author

Marking this as inactive for now.

$ host -t A pygmy.site
pygmy.site has no A record

@fubarhouse fubarhouse added the inactive No work is currently planned or pursued for this task. label Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive No work is currently planned or pursued for this task.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants