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

Add Cilium version metadata #5408

Merged
merged 4 commits into from
Mar 2, 2020
Merged

Add Cilium version metadata #5408

merged 4 commits into from
Mar 2, 2020

Conversation

ChristineTChen
Copy link
Contributor

What does this PR do?

Upstream PR to cilium has been merged and confirmed

Sample Prometheus output:

# HELP cilium_version Cilium version
# TYPE cilium_version gauge
cilium_version{version="1.6.90"} 0

Motivation

Additional Notes

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • PR title must be written as a CHANGELOG entry (see why)
  • Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
  • PR must have changelog/ and integration/ labels attached

mgarabed
mgarabed previously approved these changes Jan 7, 2020
@ChristineTChen
Copy link
Contributor Author

HOLD until feature is released in a stable build. Currently available to test in cilium v.1.7.0-rc2

@stale
Copy link

stale bot commented Feb 13, 2020

This pull request has been automatically marked as stale because it has not had activity in the last 30 days. Note that this will not be automatically closed, but the notification will remind us to investigate why there's been inactivity.

@stale stale bot added the stale label Feb 13, 2020
@ChristineTChen
Copy link
Contributor Author

/azp run

@stale stale bot removed the stale label Mar 2, 2020
@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@ofek ofek merged commit 1d260b4 into master Mar 2, 2020
@ofek ofek deleted the cc/cilium-version branch March 2, 2020 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants