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

Introduce "topology" addresses for guests #2670

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

happz
Copy link
Collaborator

@happz happz commented Feb 7, 2024

Besides a current "guest" attribute - now called "primary address", the patch adds "topology address" for guests to use when talking to each other. As of now, the topology address of a guest is the same as its primary address, but we expected that to change.

Related to #2620

Pull Request Checklist

  • implement the feature

@happz happz added provision Provision step breakage Change will change current behaviour multihost labels Feb 7, 2024
@happz happz added this to the 1.32 milestone Feb 7, 2024
@happz
Copy link
Collaborator Author

happz commented Feb 8, 2024

/packit test

@happz happz force-pushed the guest-primary-and-topology-address branch from 13964fe to de1349a Compare February 12, 2024 12:07
@happz happz marked this pull request as ready for review February 15, 2024 08:05
@thrix thrix added the full test Pull request is ready for the full test execution label Feb 19, 2024
Copy link
Collaborator

@thrix thrix left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@psss psss left a comment

Choose a reason for hiding this comment

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

Looks good, thanks!

@psss
Copy link
Collaborator

psss commented Feb 19, 2024

@happz, do you plan to add some docs?

Besides a current "guest" attribute - now called "primary address", the
patch adds "topology address" for guests to use when talking to each
other. As of now, the topology address of a guest is the same as its
primary address, but we expected that to change.

Related to #2620
@happz happz force-pushed the guest-primary-and-topology-address branch from de1349a to fd80346 Compare February 19, 2024 14:18
@happz
Copy link
Collaborator Author

happz commented Feb 19, 2024

@happz, do you plan to add some docs?

@psss probably not, I suppose we will have tests when the actual topology addresses arrive.

@psss psss self-assigned this Feb 20, 2024
@psss psss merged commit e47ebda into main Feb 20, 2024
22 checks passed
@psss psss deleted the guest-primary-and-topology-address branch February 20, 2024 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breakage Change will change current behaviour full test Pull request is ready for the full test execution multihost provision Provision step
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants