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

feat: allow module-qualified labels in generator.yml #1333

Merged

Conversation

astralblue
Copy link
Contributor

Currently, some labels such as ifName and ifDescr are defined in multiple modules and must be specified using OID. This patch avails such OIDs under both its bare label and module-qualified labels, e.g. both ifName and IF-MIB::ifName.

@bastischubert could you please take a look?

@astralblue astralblue force-pushed the allow_module_name_in_generator_yml branch from 8a8a94f to 4b95b09 Compare January 7, 2025 04:19
Currently, some labels such as ifName and ifDescr are defined in
multiple modules and must be specified using OID.  This patch avails
such OIDs under both its bare label and module-qualified labels,
e.g. both "ifName" and "IF-MIB::ifName".

Signed-off-by: Eugene Kim <[email protected]>
Signed-off-by: Eugene Kim <[email protected]>
@astralblue astralblue force-pushed the allow_module_name_in_generator_yml branch from 4b95b09 to ba7028d Compare January 7, 2025 04:24
@astralblue astralblue changed the title feat: allow module name in generator.yml feat: allow module-qualified labels in generator.yml Jan 7, 2025
Copy link
Member

@SuperQ SuperQ left a comment

Choose a reason for hiding this comment

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

I love it.

Would this also work for walk?

modules:
  if_mib:
    walk:
      - IF-MIB::interfaces
      - ...

@SuperQ SuperQ requested a review from bastischubert January 7, 2025 08:34
@SuperQ SuperQ merged commit f5d509d into prometheus:main Jan 8, 2025
6 checks passed
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.

3 participants