Skip to content

frr_exporter: T8290: Enable additional collectors#5002

Draft
peterablehmann wants to merge 2 commits intovyos:currentfrom
peterablehmann:frr_exporter
Draft

frr_exporter: T8290: Enable additional collectors#5002
peterablehmann wants to merge 2 commits intovyos:currentfrom
peterablehmann:frr_exporter

Conversation

@peterablehmann
Copy link
Copy Markdown
Contributor

Change summary

Enable additional collectors

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes)
  • Migration from an old Vyatta component to vyos-1x, please link to related PR inside obsoleted component
  • Other (please describe):

Related Task(s)

Related PR(s)

How to test / Smoketest result

Checklist:

  • I have read the CONTRIBUTING document
  • I have linked this PR to one or more Phabricator Task(s)
  • I have run the components SMOKETESTS if applicable
  • My commit headlines contain a valid Task id
  • My change requires a change to the documentation
  • I have updated the documentation accordingly

@github-actions
Copy link
Copy Markdown

github-actions bot commented Feb 21, 2026

👍
No issues in PR Title / Commit Title

@peterablehmann peterablehmann force-pushed the frr_exporter branch 3 times, most recently from 01ddec1 to a1c33fe Compare February 21, 2026 09:23
@github-actions
Copy link
Copy Markdown

CI integration ❌ failed!

Details

CI logs

  • CLI Smoketests ❌ failed
  • CLI Smoketests (interfaces only) ❌ failed
  • Config tests ❌ failed
  • RAID1 tests ❌ failed
  • CLI Smoketests VPP ⏭️ skipped
  • Config tests VPP ⏭️ skipped
  • TPM tests ⏭️ skipped

Comment on lines +50 to +54
<node name="collectors">
<properties>
<help>Set collectors to be enabled</help>
</properties>
</node>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The node cannot be without a leafNode or tagNode for the configuration.
See example for node-exporter collectors
So the package vyos-1x cannot be built at the moment.

@sever-sever
Copy link
Copy Markdown
Member

@peterablehmann, any progress on this task?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants