teleport icon indicating copy to clipboard operation
teleport copied to clipboard

Make the Fluentd guide more usable

Open ptgott opened this issue 2 years ago • 5 comments

  • Fix a mismatch between archive names in the two commands within the "MacOS" tab in Step 1
  • Fix the name of the identity file resulting from tctl auth sign in the accompanying text.

ptgott avatar Jul 13 '22 22:07 ptgott

@zmb3 I'm not sure what problem I ran into last week, but I was able to run the event handler plugin just fine on my M1 Mac today after downloading and unpacking teleport-event-handler-v9.0.4-darwin-amd64-bin.tar.gz. I've deleted the Apple silicon instructions and the "Build from Source" tab, since these aren't necessary.

ptgott avatar Jul 18 '22 15:07 ptgott

I was able to run the event handler plugin just fine on my M1 Mac today after downloading and unpacking teleport-event-handler-v9.0.4-darwin-amd64-bin.tar.gz.

Also works for me.

vitorenesduarte avatar Jul 21 '22 10:07 vitorenesduarte

Another issue we may want to fix is that most steps have "Self-Hosted | Teleport Cloud", while step 6 is the other way around:

Screenshot 2022-07-21 at 11 53 22 Screenshot 2022-07-21 at 11 53 30

vitorenesduarte avatar Jul 21 '22 12:07 vitorenesduarte

@ptgott sounds like you have Rosetta installed then. You still downloaded an amd64 build. Users who don't have Rosetta installed will see an error until we provide arm64 builds.

zmb3 avatar Jul 29 '22 20:07 zmb3

@zmb3 Just following up to see if the latest edits address your feedback.

ptgott avatar Aug 09 '22 17:08 ptgott

@ptgott I've noticed this https://github.com/gravitational/teleport-plugins/issues/631#issuecomment-1214732081 the other day.

vitorenesduarte avatar Aug 16 '22 14:08 vitorenesduarte

@vitorenesduarte Thanks! I've created an issue (#15661) to give the Fluentd guide a comprehensive refresh for the next docs sprint. I can do this while I work on guides to using Teleport with popular log management tools. In the meantime, I'd rather not expand the scope of this PR too much, as it's already received one approval.

ptgott avatar Aug 18 '22 19:08 ptgott

@ptgott See the table below for backport results.

Branch Result
branch/v10 Failed
branch/v8 Failed
branch/v9 Failed

github-actions[bot] avatar Aug 29 '22 20:08 github-actions[bot]