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

go, api: Create a structure API #2338

Closed
wants to merge 1 commit into from

Conversation

qinqon
Copy link
Member

@qinqon qinqon commented May 8, 2023

At some scenarios is useful to have a "structured" version of the
network_state, this change creates a golang api tested toward examples
and stated used at integration tests it also add the kubebuilder tags to
be k8s CRD friendly. The golang code is generated with a rust tool that traverse the AST of specific rust source files.

@kubevirt-bot
Copy link
Collaborator

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@qinqon qinqon force-pushed the generate-golang-api branch 18 times, most recently from 070254a to 47c365e Compare May 11, 2023 11:18
@stale
Copy link

stale bot commented Jun 18, 2023

Thank you for your contribution! There was no activity in this pull request recently. To avoid pull requests to pile up, an automated process marked this pull request as stale. It will close this pull request if no further activity occurs. The current policy is available at: https://github.com/nmstate/nmstate/blob/base/.github/stale.yml

@stale stale bot added the stale There was no progress for some time and the PR will be close if this does not change label Jun 18, 2023
@stale stale bot removed the stale There was no progress for some time and the PR will be close if this does not change label Jun 29, 2023
@qinqon qinqon force-pushed the generate-golang-api branch 7 times, most recently from 28b0288 to 432fca0 Compare July 5, 2023 13:01
@qinqon qinqon force-pushed the generate-golang-api branch 8 times, most recently from e6965f2 to 3b6a2eb Compare January 18, 2024 08:45
@qinqon qinqon force-pushed the generate-golang-api branch 4 times, most recently from 6c169cd to ee8922d Compare January 22, 2024 14:32
@qinqon qinqon marked this pull request as ready for review January 22, 2024 14:33
@qinqon qinqon requested review from tyll and cathay4t January 22, 2024 14:33
@qinqon qinqon force-pushed the generate-golang-api branch 5 times, most recently from 0ae14ed to 0d8d555 Compare January 25, 2024 08:44
@qinqon qinqon force-pushed the generate-golang-api branch 2 times, most recently from 617d07a to 5063b42 Compare February 15, 2024 06:49
@qinqon qinqon force-pushed the generate-golang-api branch 2 times, most recently from 3e5f72b to f44ee0a Compare February 19, 2024 06:46
At some scenarios is useful to have a "structured" version of the
network_state, this change creates a golang api tested toward examples
and stated used at integration tests it also add the kubebuilder tags to
be k8s CRD friendly, the api is created using a code generator writen in
rust that use the crate "syn" to traverse the rust ast of specific files
that contains the nmstate rust interface.

Signed-off-by: Enrique Llorente <ellorent@redhat.com>
@qinqon
Copy link
Member Author

qinqon commented Feb 27, 2024

/hold

We are going to move this to https://github.com/nmstate/kubernetes-api

@qinqon qinqon closed this May 16, 2024
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.

None yet

4 participants