googleforgames/open-match-docs

Install with Helm is confusing

sisso opened this issue · 1 comments

sisso commented

The issues I found trying to run

  1. No mention that you need to manually create namespaces. I suggest adding the add namespace command before it.
  2. After install, the note that it will be broken is below the command itself. I was debugging for a while why was not working until giving up to read the next step and found it was expected. I suggest but the note before or just remove it completely from gettings started (see 3)
  3. The install of the default evaluator can not be executed because you already have a released installed. "Default Evaluator" looks like just an extra component, not that you are reinstalling the full stack but now working with some defaults. I suggest just replace this command with a previously broken install and comment that you are installing with default values. Or replace the evaluator install by "helm upgrade" and describe that you are now upgrading it with default configurations.
syntxerror commented

@sisso Hi thanks for raising this. We had an PR that was meant to fix this but should've been merged. See here #197 . Does this seem to be a bit consistent with what you're experiencing?

  • 👍1