Skip to content
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

Added note about receptor not listening #2514

Merged
merged 2 commits into from
Nov 12, 2024
Merged

Conversation

ianf77
Copy link
Contributor

@ianf77 ianf77 commented Nov 5, 2024

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036

@ianf77 ianf77 added documentation Improvements or additions to documentation Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch 2.4 Content applies to AAP 2.4 2.5 Content applies to AAP 2.5 Needs backport to 2.5 Changes still need to be applied to the AAP 2.5 branch Ready for technical review Content is ready for technical reviews labels Nov 5, 2024
@ianf77 ianf77 changed the title Added not about receptor not listening Added note about receptor not listening Nov 5, 2024
+
`receptor_listener=True`
+
However, a connection Controller B --> A is likely to be rejected as that connection already exists.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don’t love saying is likely here. I’m unsure of the installer behavior but this is something we can test manually.

im fine if we leave it like this and circle back at a later time to address this issue

@ianf77 ianf77 added the Technical review complete Content has been reviewed for technical accuracy and completeness label Nov 12, 2024
Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
Copy link
Contributor

@EMcWhinn EMcWhinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One suggestion, rest looks good.

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
Copy link
Contributor

@EMcWhinn EMcWhinn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@EMcWhinn EMcWhinn merged commit ba03784 into ansible:main Nov 12, 2024
ianf77 added a commit to ianf77/aap-docs that referenced this pull request Nov 12, 2024
* Added not about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036

* Added note about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
EMcWhinn pushed a commit that referenced this pull request Nov 12, 2024
* Added not about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036

* Added note about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
ianf77 added a commit to ianf77/aap-docs that referenced this pull request Nov 12, 2024
* Added not about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036

* Added note about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
EMcWhinn pushed a commit that referenced this pull request Nov 12, 2024
* Added not about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036

* Added note about receptor not listening

Explain in documentation that receptor is not listening on first controller

https://issues.redhat.com/browse/AAP-23036
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.4 Content applies to AAP 2.4 2.5 Content applies to AAP 2.5 documentation Improvements or additions to documentation Needs backport to 2.4 Changes still need to be applied to the AAP 2.4 branch Needs backport to 2.5 Changes still need to be applied to the AAP 2.5 branch Ready for technical review Content is ready for technical reviews Technical review complete Content has been reviewed for technical accuracy and completeness
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants