newrelic / nri-varnish

New Relic Varnish Integration
MIT License
3 stars 9 forks source link

No params file with Varnish 6? #39

Closed jorgb90 closed 2 years ago

jorgb90 commented 2 years ago

How should we enable this when there is no params file anymore? Seems to be missing in documentation.

gsanchezgavier commented 2 years ago

Hi @jorgb90 could you explain further the issue you are having? Here are the docs to configure the integration.

jorgb90 commented 2 years ago

@gsanchezgavier I think its pretty clear? The module needs the .params file in the config, but Varnish 6 doesnt have a .params file anymore. Or at least on Ubuntu there is no .params file.

Schermafbeelding 2021-11-17 om 09 42 15

Should we instead use the .service file? "/etc/systemd/system/varnish.service"

gsanchezgavier commented 2 years ago

Hi @jorgb90 thanks for the details. As you mention, Varnish 6 removes the varnish.params file. This was used on the integration to collect this data as Inventory, you can still collect metrics by adding the METRICS envar and set it to true

integrations:
  - name: nri-varnish
    env:
      INSTANCE_NAME: new_relic
      METRICS: true
    interval: 15s
    labels:
      env: production
      role: varnish

I will reopen this issue to update the docs.

jorgb90 commented 2 years ago

Thanks @gsanchezgavier !