-
Notifications
You must be signed in to change notification settings - Fork 9.7k
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
*: add "--unsupported-arch" flag to remove confusing error message #9089
Conversation
a237048
to
814da31
Compare
321dc7c
to
2a1fb13
Compare
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
419a9d4
to
e8aa9a8
Compare
Signed-off-by: Gyuho Lee <gyuhox@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #9089 +/- ##
=========================================
Coverage ? 75.96%
=========================================
Files ? 359
Lines ? 29953
Branches ? 0
=========================================
Hits ? 22754
Misses ? 5608
Partials ? 1591
Continue to review full report at Codecov.
|
return cfg.Validate() | ||
} | ||
|
||
func (cfg *Config) Validate() error { | ||
// TODO qualify arm64 | ||
goArch := runtime.GOARCH |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
still keep this in a func?
@@ -32,7 +32,7 @@ For etcd to officially support a new platform as stable, a few requirements are | |||
|
|||
etcd has known issues on 32-bit systems due to a bug in the Go runtime. See the [Go issue][go-issue] and [atomic package][go-atomic] for more information. | |||
|
|||
To avoid inadvertently running a possibly unstable etcd server, `etcd` on unstable or unsupported architectures will print a warning message and immediately exit if the environment variable `ETCD_UNSUPPORTED_ARCH` is not set to the target architecture. | |||
To avoid inadvertently running a possibly unstable etcd server, `etcd` on unstable or unsupported architectures will print a warning message and immediately exit if flag `--unsupported-arch` or environment variable `ETCD_UNSUPPORTED_ARCH` is not set to the target architecture. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
there is no flag for this feature since we do not really want to promote the usage of running etcd on unsupported platform, and do not want to make it a feature flag. env is much more easier to deprecated for example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then should we just have special handling for ETCD_UNSUPPORTED_ARCH
? Otherwise, we will keep warning
pkg/flags: unrecognized environment variable ETCD_UNSUPPORTED_ARCH=arm64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes possiblely.
Fix #9077.
The confusing warning disappears with this.