Bug 1470653

Peter, is this a regression from 6.2?

This sounds like a documentation bug if we could flag it as such to update the docs to exclude the quotes.

Moving to `Documentation`, as `ls "~/.ssh"` doesn't work for ~ extrapolation, we should not expect the installer doing more logic around that. We should not instruct people to use invalid way of passing paths to CLI tools.

Assigning to Charles for review.

(In reply to Brad Buckingham from comment #1)
> Peter, is this a regression from 6.2?

Hey, I don't remember this happening in 6.2, though I might be wrong. I'm ok with passing this to docs, though the displayed message could be improved.

Assigning to Clifton to follow up on any changes required for this bug.

(In reply to Peter Ondrejka from comment #9)

> Clifton, can you please watch
> https://bugzilla.redhat.com/show_bug.cgi?id=1466688 and update docs
> accordingly? There is also a similar problem with satellite-change-hostname
> but I'll notify Russel in the related bug
> https://bugzilla.redhat.com/show_bug.cgi?id=1451564
> 
> Cheers

Hi Peter,

I will follow the linked bug and update the docs accordingly.

Thanks

Hello

Re: the tilde

I think we can just drop ~/ and not replace it with /root as this is the root user's home directory.

If the reader follows the instructions as we write them then it will just work.

(In reply to Stephen Wadeley from comment #11)
> Hello

Hi Stephen,

BZ#1466688 has not been verified or closed yet. Shall I proceed anyway with the current recommendations?

BZ#1470653: Replace tilda with /root/

> /root/mycapsule.example.com-certs.tar instead of
> ~/mycapsule.example.com-certs.tar

BZ#1451564: Replace --capsule-parent-fqdn

> Procedure 7.8. Rename a Satellite Server:
> -- step 5 needs --foreman-proxy-content-parent-fqdn instead of
> --capsule-paretnt-fqdn

Regards,

(In reply to csherrar from comment #13)
> (In reply to Stephen Wadeley from comment #11)
> > Hello
> 
> Hi Stephen,
> 
> BZ#1466688 has not been verified or closed yet. Shall I proceed anyway with
> the current recommendations?
You do not *have* to wait for that one to clear. We can see from Ivan's comments[1][2] that the important thing is for Docs not to have ~/ inside double quotation marks.
We should keep commands as simple as possible and remove "" when they are not required.
If there is a space in some command argument, then you need to quote that.

> 
> BZ#1470653: Replace tilda with /root/
> 
> > /root/mycapsule.example.com-certs.tar instead of
> > ~/mycapsule.example.com-certs.tar

See my comment 12
> 
> BZ#1451564: Replace --capsule-parent-fqdn
> 
> > Procedure 7.8. Rename a Satellite Server:
> > -- step 5 needs --foreman-proxy-content-parent-fqdn instead of
> > --capsule-paretnt-fqdn
> 
Russell raised BZ#1510718 to fix that. I think that is not related to ~/ or "".

I suggest you wait a few more days for some of my bugs to be resolved and make their way to master.
e.g.: BZ#1504280 and BZ#1490043

Most of these issue should be gone then.

Thank you


[1] https://bugzilla.redhat.com/show_bug.cgi?id=1466688#c5
[2] https://bugzilla.redhat.com/show_bug.cgi?id=1470653#c4

(In reply to Stephen Wadeley from comment #17)

> You do not *have* to wait for that one to clear. We can see from Ivan's
> comments[1][2] that the important thing is for Docs not to have ~/ inside
> double quotation marks.
> We should keep commands as simple as possible and remove "" when they are
> not required.
> If there is a space in some command argument, then you need to quote that.

Thanks, I will be more proactive and apply recommendations as tickets develop.

> I suggest you wait a few more days for some of my bugs to be resolved and
> make their way to master.
> e.g.: BZ#1504280 and BZ#1490043
> 
> Most of these issue should be gone then.

I will pull down master as BZ#1504280 is now merged and check/make these changes.

This content is live on the Customer Portal.

Closing.

Updating the component and doc type.