Skip to content

Conversation

rcgoodfellow
Copy link
Contributor

No description provided.

serialize_value_type(&x, &mut data);
keyset_data.extend_from_slice(&data[..2]);
}
ROUTER_V6_RT => {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Huh. How did I get this up on a4x2 if ipv6 routing doesn't work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Things break non-deterministically. See the comments here. Sometimes CI in that PR works, sometimes it does not. I'm not sure if this is the cause yet. Still trying to determine where the inconsistency is creeping in.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is not the cause of the issue in oxidecomputer/maghemite#545. I can reproduce the issue with this patch.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The root cause for the flaky behavior I was seeing in CI has been identified. This was due to restarting dpd while keeping underlying state around in softnpu. The following ticket has been filed for this.

And until that gets done oxidecomputer/maghemite#545 is not restarting dendrite during its test runs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants