Add Go and Rust client bindings #2389
Comments
|
On the Rust side we could also use it here https://github.com/ostreedev/ostree/blob/5551c54c6e6eba8145b95bd3b28223f1941a9e8d/tests/inst/src/rpmostree.rs#L16 |
That's the route we took with go-systemd and at least on the Go side it is probably a better idea. On the Rust side, Zincati has a bunch of CLI-parsing methods here, but that's probably good enough for now as there isn't yet a good non-blocking DBus library. |
cgwalters
added a commit
to cgwalters/rpm-ostree
that referenced
this issue
Feb 12, 2021
This is intended to be published to https://crates.io/crates/rpmostree-client Part of coreos#2389 This directly imports the code from https://github.com/ostreedev/ostree/blob/5551c54c6e6eba8145b95bd3b28223f1941a9e8d/tests/inst/src/rpmostree.rs Once merged and released I'll try converting the ostree test suite over as well as Zincati. Internally add a testutils helper to validate it works.
cgwalters
added a commit
to cgwalters/rpm-ostree
that referenced
this issue
Feb 16, 2021
This is intended to be published to https://crates.io/crates/rpmostree-client Part of coreos#2389 This directly imports the code from https://github.com/ostreedev/ostree/blob/5551c54c6e6eba8145b95bd3b28223f1941a9e8d/tests/inst/src/rpmostree.rs Once merged and released I'll try converting the ostree test suite over as well as Zincati. Internally add a testutils helper to validate it works.
cgwalters
added a commit
to cgwalters/rpm-ostree
that referenced
this issue
Feb 16, 2021
This is intended to be published to https://crates.io/crates/rpmostree-client Part of coreos#2389 This directly imports the code from https://github.com/ostreedev/ostree/blob/5551c54c6e6eba8145b95bd3b28223f1941a9e8d/tests/inst/src/rpmostree.rs Once merged and released I'll try converting the ostree test suite over as well as Zincati. Internally add a testutils helper to validate it works.
openshift-merge-robot
pushed a commit
that referenced
this issue
Feb 17, 2021
This is intended to be published to https://crates.io/crates/rpmostree-client Part of #2389 This directly imports the code from https://github.com/ostreedev/ostree/blob/5551c54c6e6eba8145b95bd3b28223f1941a9e8d/tests/inst/src/rpmostree.rs Once merged and released I'll try converting the ostree test suite over as well as Zincati. Internally add a testutils helper to validate it works.
|
I've created https://github.com/cgwalters/rpmostree-client-go |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Motivated by openshift/machine-config-operator#2302
and the zincati bit in coreos/coreos-assembler#1964
Basically let's create e.g.
github.com/coreos/rpm-ostree-client-goandgithub.com/coreos/rpm-ostree-client-rustbindings that mostly have:status --jsoninto strongly typed structs (as both the MCD and Zincati do)rpm-ostree ...with arguments but also kind of requires them to set theRPMOSTREE_CLIENT_IDenv var (maybe via aContextstruct or so?)Of course these bindings could directly invoke via DBus too but that greatly increases the scope.
The text was updated successfully, but these errors were encountered: