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

[VanFlow]: Include a "version" configuration field for site records #1650

Open
c-kruse opened this issue Oct 24, 2024 · 2 comments
Open

[VanFlow]: Include a "version" configuration field for site records #1650

c-kruse opened this issue Oct 24, 2024 · 2 comments

Comments

@c-kruse
Copy link

c-kruse commented Oct 24, 2024

The control plane and console presently rely on a version string set on the SITE vanflow records to communicate skupper controller versions of connected sites (distinct from router version.)

Could we add a configuration option so that the "controller" (in this case the non-kube bootstrap process) could expose its version?

This was likely missed on the first go-around as the vanflow spec was missing this attribute: skupperproject/vanflow-specification@260504a

@ssorj
Copy link
Member

ssorj commented Dec 6, 2024

I'm thinking the router should take a --site site.json argument. For now that file would be:

{
  "version": "9.9.9"
}

But in future, if there's anything else we want in the vanflow site record, we could inject it here. For example, SiteRecord currently has Provider and Namespace, which we could also set. I guess that raises a question: how is the SiteRecord.Name currently determined?

@c-kruse
Copy link
Author

c-kruse commented Dec 6, 2024

@ssorj catching you up.

As of pretty recently we added configuration for this in the regular body of router config. #1543 https://github.com/skupperproject/skupper-router/blob/main/python/skupper_router/management/skrouter.json#L637-L666 and just missed a version property - probably because the field was missing from vanflow spec documentation.

In the nonkube bootstrap we set site name to match the site record name: https://github.com/skupperproject/skupper/blob/v2/pkg/nonkube/api/site_state.go#L271-L276

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

No branches or pull requests

2 participants