Closed benburkert closed 8 months ago
Can confirm, we just ran into the same issue earlier.
I can confirm, we have the same issue too
I don't see any recent commits to the action that might explain this so this is most likely a breaking change in the API ?
The action is setup to float to the latest version of the newrelic/cli container and that is now on v0.74 of the cli.
Unfortunately the latest cli version is expecting a property from their graphql api (NerdGraph), which isn't live yet. Either they need to deploy the updated graphql api or roll back the cli to not ask for the missing property.
Not much you can do about this since they deployed a change in their client before the server, and allow the action to float to latest client.
yeah, this PR was merged in 4 hours ago, causing this issue 🤦
PR opened to fix this by pinning the cli. I'd argue it should always be pinned and handled like the dependency it is.
Sorry about that everyone. A version of the CLI went out today with an API that is not yet GA. I merged @barryhagan's PR to pin the CLI to the previous version hopefully unblocking you all.
@mbazhlekova i think this will still be an issue until there is a new release of this repo. And for anyone who has pinned to the current point release, wont it require a new release of the cli docker container?
@benburkert - yeah, it is still an issue if you are using the versioned action like this:
uses: newrelic/deployment-marker-action@v2.4.0
There needs to be a new release/tag to get an action version that is pinned to the working cli. You should be able to workaround this immediately by specifying the main branch until there is a tagged release after my PR. (a temporary fix, not recommended long term):
uses: newrelic/deployment-marker-action@main
Alternatively, just reference the commit hash directly:
uses: newrelic/deployment-marker-action@cd8b5c28275fac9b05f66df6f025090072612bb8
Actually we're going to release a new version of the CLI without the commit that's causing the problem to get around having to update the deployment marker version.
You should still release a v2.5.0 of this with the pinned CLI version. I appreciate that the CLI rollback will fix the 2.4.0 version that everyone is using for now, but that doesn't address the floating dependency problem.
Every single time someone runs this action, they are taking the latest release of the CLI container at that moment. As we found today, that is not always a good thing.
This is a stable solution that I will use until you tag a new release:
@barryhagan v2.5.0 is out with your change to pin the version
I'm using
newrelic/deployment-marker-action@v2.4.0
and earlier today action runs started failing with the messageError: level=fatal msg="Cannot query field \"customAttributes\" on type \"ChangeTrackingDeployment\"."
. Here is the log for the step:And here is what my action yml config looks like:
The config hasn't changed recently and, as far as I know, the credentials are still valid.