Skip to content

ideally, omicron-omdb should not depend on omicron-nexus #10852

Description

@iliana

#10447 added a dependency from the omicron-omdb crate to omicron-nexus in order to access omicron_nexus::app::crucible::crucible_pantry_client_volume_health. This function is not used by Nexus itself and appears to exist in Nexus to rely on some of the types defined in the same module.

While I have not landed this change yet, I've had plans for a while to split up the builds during cargo xtask releng to allow the packages included on the host OS to be built separately from the zones that are distributed on their own in the TUF repo, with the goal of parallelizing the two longest tasks: building omicron-nexus and building the host OS image. omdb is in the switch zone and thus the host image, and adding this edge to the dependency graph eliminates this optimization.

(Note: the dev-dependency from omicron-omdb to omicron-nexus is fine.)

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions