[Feedback requested] Dropping redirectors and updating text output for sosreport about sos HOT 5 OPEN

Cc @jcastill and @mhradile at least.

AFAIK original plans were to keep the shortcut/redirectors for the sos-4.* timeframe. While the planned change itself is sound, my gut feeling is users are not yet prepared, still thinking about "sosreport (tarball) is taken by sosreport (command)". We do have a warning text Please note the 'sosreport' command has been deprecated but it is hard to say how many people read the leading text (or realize its implication). People usually run sosreport, wait for the outcome and copy the generated tarball. So maybe we shall put such string (also) at the end of the stdout where it gets more attention as a heads up, now?

Also, what would be the justification for the redirectors drop - just "we have the legacy redirectors for a while"? Isn't better message "here we add another sos subcommand (e.g. sos upload) under the sos umbrella and it is the good time to drop the redirectors"?

from sos.

from sos.

AFAIK original plans were to keep the shortcut/redirectors for the sos-4.* timeframe.

I mean, we only went to 4.x instead of continuing the 3.x line because of the new binary. I don't see anything on the horizon that would want us to move to a 5.x versioning scheme (also, just going to sidestep the entire versioning scheme debate minefield here). Software evolves and we should be mindful of backwards compatibility for sure, but we shouldn't be shackled to maintaining old entrypoints.

While the planned change itself is sound, my gut feeling is users are not yet prepared, still thinking about "sosreport (tarball) is taken by sosreport (command)".

It's been over 3 years at this point; how much preparation is realistically needed?

Beyond that we should want to encourage an understanding that sos does more than "just" the report function. One of the easiest ways to do this is to have users notice "hey, report is a subcommand, what else can this thing do?"

We do have a warning text Please note the 'sosreport' command has been deprecated but it is hard to say how many people read the leading text (or realize its implication). People usually run sosreport, wait for the outcome and copy the generated tarball. So maybe we shall put such string (also) at the end of the stdout where it gets more attention as a heads up, now?

I'd definitely be open to adding a footer message for a minor version cycle before we drop the redirectors.

Also, what would be the justification for the redirectors drop - just "we have the legacy redirectors for a while"? Isn't better message "here we add another sos subcommand (e.g. sos upload) under the sos umbrella and it is the good time to drop the redirectors"?

As I recall, sos upload was shot down when I brought it up previously in an SST meeting - but less pedantically to your point of tying it to another subcommand being added...it'd be nice but I don't think we should marry the two. By that same token, if there are subcommands you/anyone think would be a useful addition let's get a discussion going on that separately from this one.

Our (old and nowhere near migrated) tests are definitely not ready.

I'm going to be politically nice here and not comment on this point.

from sos.

What about adding the text "Please note the 'sosreport' command has been deprecated" in 4.7, leave it for 4.8 (and perhaps expand it with a 'will be completely dropped in the next minor release', and drop redirectors in 4.9 (or even 4.10 if we consider that there will be 4.xx) ?

We have this documented in many places already, and two minor releases may give us enough time to finish whatever else is needed regarding communication.

Our (old and nowhere near migrated) tests are definitely not ready.

@mhradile we can discuss this, but hopefully two minor releases may give us enough time to plan for the change?

from sos.

I am ok having that explicit statement "sosreport command will be removed in the (next | 4.X) release" around the "sosreport has been generated at" text, and react accordingly :)

from sos.