Skip to content

buildomat: delete interop jobs - #882

Merged
taspelund merged 1 commit into
mainfrom
trey/delete-build-interop
Aug 28, 2026
Merged

buildomat: delete interop jobs#882
taspelund merged 1 commit into
mainfrom
trey/delete-build-interop

Conversation

@taspelund

Copy link
Copy Markdown
Contributor

These jobs were used as the basis of automated interop testing for mgd. However, since falcon-lab was introduced we've stopped using them. Let's delete these jobs in favor of just going with falcon-lab.

These jobs were used as the basis of automated interop testing for mgd.
However, since falcon-lab was introduced we've stopped using them.
Let's delete these jobs in favor of just going with falcon-lab.

Signed-off-by: Trey Aspelund <trey@oxidecomputer.com>
@taspelund
taspelund force-pushed the trey/delete-build-interop branch from 3ce8af6 to 7671381 Compare August 17, 2026 22:42

@nicolaskagami nicolaskagami left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems fine to me but I do wonder:

  • Are they becoming stale?
  • How does falcon-lab supplant them?

@taspelund

Copy link
Copy Markdown
Contributor Author

Seems fine to me but I do wonder:

* Are they becoming stale?

I'm not understanding what you're asking. Can you rephrase?

* How does falcon-lab supplant them?

falcon-lab has topologies for BGP unnumbered and BFD w/ static routing, both of which have accompanying tests. These are run by CI as part of the falcon job (the actual test is in .github/buildomat/jobs/falcon-lab.sh).

Prior to falcon-lab, we used the interop crate in the testbed repo for these kind of interop tests, and those network topologies were copied over to falcon-lab as an in-repo replacement

@nicolaskagami nicolaskagami left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm not understanding what you're asking. Can you rephrase?

I was mostly trying to understand if the test had diverged from the code or its intended purpose.

@taspelund

Copy link
Copy Markdown
Contributor Author

Ah, I see. I wouldn't say it's diverged so much as it's stopped being maintained. I don't think it's kept up with changes to RSS or API methods, particularly because it's annoying to maintain it as part of a separate repo

@taspelund
taspelund merged commit aa9632b into main Aug 28, 2026
20 checks passed
@taspelund
taspelund deleted the trey/delete-build-interop branch August 28, 2026 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants