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

pkger enforce resource metadata.name field is dns compliant #17940

Closed
3 tasks done
jsteenb2 opened this issue May 2, 2020 · 0 comments · Fixed by #17941
Closed
3 tasks done

pkger enforce resource metadata.name field is dns compliant #17940

jsteenb2 opened this issue May 2, 2020 · 0 comments · Fixed by #17941

Comments

@jsteenb2
Copy link
Contributor

jsteenb2 commented May 2, 2020

Pkger resources metadata.name field should be DNS compliant

  • resource names must be between 3 and 63 characters long.
  • resource names can consist only of lowercase letters, numbers, dots (.), and hyphens (-).
  • resource names must not be formatted as an IP address (for example, 192.168.5.4).

useful link from std lib:
isDomainName: https://golang.org/src/net/dnsclient.go

jsteenb2 added a commit that referenced this issue May 4, 2020
this PR includes a lot of small changes to names in existing test pkgs.
the tests are updated to follow suit.

closes: #17940
jsteenb2 added a commit that referenced this issue May 4, 2020
this PR includes a lot of small changes to names in existing test pkgs.
the tests are updated to follow suit.

closes: #17940
ethanyzhang pushed a commit that referenced this issue May 5, 2020
this PR includes a lot of small changes to names in existing test pkgs.
the tests are updated to follow suit.

closes: #17940
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant