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

AccountId() NRQL function (not yet ready for publish) #6854

Conversation

mlobiondo-newrelic
Copy link
Contributor

@mlobiondo-newrelic mlobiondo-newrelic commented Mar 30, 2022

This is waiting to be published until some time in April. This is documenting a NRQL function accountId() that allows you to return the accounts associated with the data queried. This is currently only for nerdgraph; query builder ability will come later (currently you can only run queries in UI query builder that are constrained by the account you're in but with nerdgraph you can cross account boundaries).

…-website/commit/567d3b1ea5975e0e12fcdfb7c002087166bfe1c7

Please follow conventional commit standards
in your commit messages and pull request title.

Give us some context

  • What problems does this PR solve?
  • Add any context that will help us review your changes such as testing notes,
    links to related docs, screenshots, etc.
  • If your issue relates to an existing GitHub issue, please link to it.

@CLAassistant
Copy link

CLAassistant commented Mar 30, 2022

CLA assistant check
All committers have signed the CLA.

@github-actions
Copy link

Hi @mlobiondo-newrelic 👋
Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes).

@mlobiondo-newrelic
Copy link
Contributor Author

Hi @bradleycamacho, this is the new version of the accountId() PR (the changes are the same). This still should not be merged yet.

@gatsby-cloud
Copy link

gatsby-cloud bot commented Mar 30, 2022

Gatsby Cloud Build Report

docs-website-develop

🎉 Your build was successful! See the Deploy preview here.

Build Details

View the build logs here.

🕐 Build time: 20m

@zuluecho9 zuluecho9 changed the title Document accountId() function AccountId() NRQL function (not yet ready for publish) Mar 30, 2022
@zuluecho9 zuluecho9 self-assigned this Mar 30, 2022
@zuluecho9 zuluecho9 added content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers) labels Mar 30, 2022
@zuluecho9
Copy link
Contributor

For other tech writers: I did a review of this, for what it's worth. When this goes live, if it's not yet available to do this in query builder UI, I think we should add a note that this is not available in the UI, but only available when using NerdGraph queries. (As Mlobiondo said "you can use cross-account queries in dashboards via NerdGraph, JSON or Terraform, though not the Query Builder", and we maybe need to dig into that a little more.

@zuluecho9 zuluecho9 self-requested a review March 30, 2022 21:43
@zuluecho9 zuluecho9 moved this from Hero to triage to Scheduled work in Docs PRs and Issues Mar 30, 2022
@zuluecho9 zuluecho9 moved this from Scheduled work to Drafts in Docs PRs and Issues Mar 30, 2022
@mlobiondo-newrelic
Copy link
Contributor Author

For other tech writers: I did a review of this, for what it's worth. When this goes live, if it's not yet available to do this in query builder UI, I think we should add a note that this is not available in the UI, but only available when using NerdGraph queries. (As Mlobiondo said "you can use cross-account queries in dashboards via NerdGraph, JSON or Terraform, though not the Query Builder", and we maybe need to dig into that a little more.

@zuluecho9, the accountId() NRQL function described in this docs update is available via the Query Builder like any other NRQL function. What's not available via the Query Builder is Cross-Account Queries, a related feature that will have its own documentation when it's GAed. We don't want to mention cross-account queries in the public docs for accountId() since cross-account queries are only in beta at this point.

@zuluecho9
Copy link
Contributor

@mlobiondo-newrelic Is there much practical use of the accountId() function if they don't have the cross-account functionality? I guess that's what I'm not getting. Is there still a practical use to using it? I would think it would always be returning the same account IDs if they don't have cross-account ability. if that makes sense.

@mlobiondo-newrelic
Copy link
Contributor Author

@zuluecho9 It's not particularly useful when not used in a cross-account query. It will always return the same account ID on a query run against a single account. It's generally available now, though, while cross-account queries are not quite ready.

zuluecho9
zuluecho9 previously approved these changes Apr 5, 2022
@zuluecho9 zuluecho9 merged commit abd75da into newrelic:develop Apr 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content requests related to docs site content from_internal Identifies issues/PRs from Relics (except writers)
Projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants