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

Studio Agent #66

Closed
4 of 5 tasks
abernix opened this issue Jul 12, 2021 · 7 comments
Closed
4 of 5 tasks

Studio Agent #66

abernix opened this issue Jul 12, 2021 · 7 comments
Assignees
Labels

Comments

@abernix
Copy link
Member

abernix commented Jul 12, 2021

Requirements

TBD. An agent which communicates with Studio and acts as the foundation for operation (signature-based), field (shape) and latency (trace) stats to power Apollo Studio functionality.

Relevant Subtasks / Links

These are likely components/candidates of this agent!

@o0Ignition0o o0Ignition0o transferred this issue from another repository Nov 4, 2021
@Geal
Copy link
Contributor

Geal commented Dec 9, 2021

relevant links for how data must be filtered for privacy reasons:

@garypen garypen self-assigned this Jan 6, 2022
@garypen
Copy link
Contributor

garypen commented Jan 10, 2022

I'm working on three components of this at the moment:

  • Adding a basic infrastructure to allow usage data to be transferred from the router to studio usage ingress
  • Normalising queries so that statistics are reported accurately
  • Figuring out which statistics to collect and how to present them in the required format

The second task is mainly relying on work which needs to be done in apollo-rs. @lrlna Is there a ticket for the semantic validation work that I can reference here?

@lrlna
Copy link
Member

lrlna commented Jan 20, 2022

@garypen yes, the overview ticket is apollographql/apollo-rs#144

@abernix
Copy link
Member Author

abernix commented Jan 26, 2022

@garypen Administratively: #67 (from the original post above) is currently marked as Deferred, though I think it's wound up / inclusive of the stuff you're working on. Is that right? (If yes, would it make sense to mark that more granular issue as "In Progress", update #309 to indicate that it closes that issue and keep this as a higher-level longer-term coordination issue?).

@garypen
Copy link
Contributor

garypen commented Jan 31, 2022

I'm doing the work described as "Phase 1" in the quip document proposal from David. That doesn't include any field usage stuff. To be honest this has evolved fairly dynamically since the original issues were filed. Maybe we need to spend some time "issue wrangling"?

@abernix
Copy link
Member Author

abernix commented Feb 2, 2022

@garypen Ok, that's fair — I agree some issue grooming is in order. I definitely understood that it was just Phase 1 that you were doing — it's absolutely true that phase doesn't include field usage reporting, I just forgot that. In that regard, I think the answer to my question above is merely "No" (and, in retrospect, I could have latched onto that — I was merely doing some cleaning up to make sure that PRs were closing issues.)

Looking at this particular issue, it's possible it's not the right match for what I was looking for, which is "the issue that #309 should close".

With that in mind, I think we're missing an issue for the Phase 1 work. What do you think about:

  1. Opening an issue that is just for the operation-based reporting / Phase 1 stuff, documenting what's in the doc. We should communicate (publicly) what the scope of work is.
  2. Add it to the board and add appropriate GH Project metadata to it — iteration, size, etc.
  3. Edit initial studio agent, client awareness and operation reporting #309 to indicate that it closes that new issue. (It'll be necessary for the review to make sure it ticks the boxes, ya?)
  4. Consider leaving this issue open as the umbrella issue that I think it is today — update it to convey the expected follow-up stuff (e.g., phase 2, etc.). Alternatively, consider editing this to just cover the "Agent" part of it, if that's even interesting on its own.

Thoughts? (If you agree, feel free to go for it?)

@garypen garypen reopened this Feb 8, 2022
abernix added a commit that referenced this issue Feb 17, 2022
Follows up: #445
Follows up: #309

Relates to: #66
Relates to: #444
abernix added a commit that referenced this issue Feb 17, 2022
Follows up: #445
Follows up: #309

Relates to: #66
Relates to: #444
@abernix
Copy link
Member Author

abernix commented May 25, 2022

Most of the items on this parent issue are closed, so I'll close this and leave the child issues to hold their weight in terms of follow-up items.

@abernix abernix closed this as completed May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants