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

STAC-22163: print platform version on connection success #93

Merged

Conversation

deontaljaard
Copy link
Contributor

@deontaljaard deontaljaard commented Feb 14, 2025

As part of an initiative to align version numbers across SUSE Observability components, it was decided to display the platform version instead of the app version (docker image tag) to reduce potential confusion and simplify support.

Changes overview:

  • Generated latest openapi definitions
  • Updated context validation to print the platform version, if contained in the server info response
  • Updated gomnd and exportloofref linters to mnd and copyloopvar

Output when server returns platform version
Screenshot 2025-02-25 at 14 22 35

Output when server doesn't return platform version (when a new/updated client interacts with an older version of the server) - backwards compatibility
Screenshot 2025-02-25 at 14 23 32

@deontaljaard deontaljaard force-pushed the stac-22163-embed-platform-info-in-connection-validation-log branch from 4ce5412 to d0d5062 Compare February 25, 2025 12:42
@deontaljaard deontaljaard marked this pull request as ready for review February 25, 2025 13:15
@deontaljaard deontaljaard merged commit 218df2f into main Feb 25, 2025
5 checks passed
@deontaljaard deontaljaard deleted the stac-22163-embed-platform-info-in-connection-validation-log branch February 25, 2025 15:11
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.

2 participants