search

kubernetes-retired / spartakus

[EOL] Anonymous Usage Collector
Apache License 2.0
74 stars 30 forks source link

README: make uuidgen OSX friendly #2

Closed philips closed 7 years ago

philips commented 7 years ago

On OSX uuidgen doesn't have -r and on linux -r is default. Just remove the flag.

philips commented 7 years ago

ping @thockin

thockin commented 7 years ago

ACK.

On Wed, Sep 28, 2016 at 1:00 PM, Brandon Philips notifications@github.com wrote:

@philips commented on this pull request.

In README.md https://github.com/kubernetes-incubator/spartakus/pull/2:

@@ -128,7 +128,7 @@ The simplest way is to use kubectl run. kubectl run spartakus \ --image=gcr.io/google_containers/spartakus-amd64:v1.0.0 \ -- \

  • volunteer --cluster-id=$(uuidgen -r)
  • volunteer --cluster-id=$(uuidgen)

On OSX uuidgen doesn't have -r and on linux -r is default.

https://linux.die.net/man/1/uuidgen "By default uuidgen will generate a random-based UUID if a high-quality random number generator is present."

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/kubernetes-incubator/spartakus/pull/2, or mute the thread https://github.com/notifications/unsubscribe-auth/AFVgVCDKDNENnGVsTyhj8Sa0MrfJ7yUsks5qusdWgaJpZM4KCaTl .