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.

Sign up for GitHub

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

Jump to bottom

remove names.NewRandomK8() completely from the project #573

Open
Tracked by #488
mojtaba-esk opened this issue Oct 15, 2024 · 0 comments · May be fixed by #580 or #581
Open
Tracked by #488

remove names.NewRandomK8() completely from the project #573

mojtaba-esk opened this issue Oct 15, 2024 · 0 comments · May be fixed by #580 or #581
Assignees
@mojtaba-esk
Labels
knuu refactor Refactoring
Milestone
v0.17.0

Comments

@mojtaba-esk
Copy link
Member

The names.NewRandomK8() function which introduces randomness in the k8s resource names, are still being used in some parts of the project such as Traefik reverse proxy pkg.

In order to have long running tests, we need to remove them completely.
@mojtaba-esk mojtaba-esk added the refactor Refactoring label Oct 15, 2024
@mojtaba-esk mojtaba-esk added this to the v0.17.0 milestone Oct 15, 2024
@mojtaba-esk mojtaba-esk self-assigned this Oct 15, 2024
@smuu smuu mentioned this issue Oct 16, 2024
Open
@smuu smuu added the knuu label Oct 16, 2024
@notlelouch notlelouch linked a pull request Nov 14, 2024 that will close this issue
Open
@mojtaba-esk mojtaba-esk linked a pull request Nov 15, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mojtaba-esk mojtaba-esk

Labels
knuu refactor Refactoring
Projects
None yet
Milestone
v0.17.0
2 participants
@mojtaba-esk @smuu