newrelic / newrelic-cli

The New Relic Command Line Interface
https://developer.newrelic.com/explore-docs/newrelic-cli
Apache License 2.0
125 stars 70 forks source link

fix(infra): Remove Test endpoints to fix issue with infra-alert.service #1609

Closed Nandu-pns closed 1 month ago

Nandu-pns commented 1 month ago

JIRA - https://new-relic.atlassian.net/browse/NR-270753

This PR aims to address the below timeout issue for few customers as it appears CLI is performing few test calls to verify if the New Relic endpoints are reachable and this infra-alert.service is being invoked during this specific call.

FATAL Get "https://login.newrelic.com/login?return_to=https%3A%2F%2Finfrastructure-alert.service.newrelic.com%2Fv2": dial tcp 18.188.220.94:443: i/o timeout

Nandu-pns commented 1 month ago

Closing this PR as we plan to discard the specific infra test API call which is causing the issue instead of removing TestEndpoints function call. Changes are in the client-go PR