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

feat: allow to set hostAliases for traefik pod #1094

Merged
merged 7 commits into from
Jun 20, 2024

Conversation

x0ddf
Copy link
Contributor

@x0ddf x0ddf commented Jun 17, 2024

What does this PR do?

Allows to set host aliases in the traefik pod.

More

  • Yes, I ran make test and all the tests passed

Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
Copy link
Contributor

@darkweaver87 darkweaver87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@x0ddf x0ddf requested a review from mloiseleur June 18, 2024 10:17
@x0ddf
Copy link
Contributor Author

x0ddf commented Jun 18, 2024

@youkoulayley, please merge it if there are no other remarks/improvements that need to be implemented.

@x0ddf
Copy link
Contributor Author

x0ddf commented Jun 19, 2024

@mloiseleur @darkweaver87, no pressure, but when can this PR be expected to be merged(I'm relying on this change due to migration to traefik v3)?

traefik/values.yaml Outdated Show resolved Hide resolved
EXAMPLES.md Outdated Show resolved Hide resolved
EXAMPLES.md Outdated Show resolved Hide resolved
@mloiseleur
Copy link
Contributor

Thanks 👍 .
I think we are good to go. See my last minor suggestion.

x0ddf and others added 3 commits June 20, 2024 01:06
Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
Co-authored-by: Michel Loiseleur <97035654+mloiseleur@users.noreply.github.com>
@x0ddf
Copy link
Contributor Author

x0ddf commented Jun 19, 2024

@mloiseleur fixed

@x0ddf x0ddf requested a review from mloiseleur June 20, 2024 19:09
@traefiker traefiker merged commit 42e5745 into traefik:master Jun 20, 2024
1 check passed
@x0ddf x0ddf deleted the feature/host-aliases branch June 21, 2024 05:06
@x0ddf
Copy link
Contributor Author

x0ddf commented Jun 21, 2024

@mloiseleur, I didn't find any information about releasing new versions with chart improvements, only updates related to the latest version of Traefik (#625 ). How frequently are you delivering new artifacts to the registry with minor improvements/fixes, like this one and #1093, for example?

@mloiseleur
Copy link
Contributor

@x0ddf It depends. There is no hard rule. It's usually after a new release of Traefik Proxy or Traefik Hub.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants