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

Added Cluster Queue/Jobs example to GraphQL Cookbook #2224

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

james2791
Copy link
Contributor

Added two examples for Clusters with GraphQL: inclusive of fetching Cluster queues for a given organisation, and also the current running Jobs for a given Cluster Queue (ID).

pages/apis/graphql/graphql_cookbook.md Outdated Show resolved Hide resolved

### List cluster queues

List the first 20 cluster queues' key and ID for the first 10 clusters in the organisation
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this could be a bit simpler/clearer. Maybe:

Suggested change
List the first 20 cluster queues' key and ID for the first 10 clusters in the organisation
For the first 10 clusters in an organization, list the key and ID of the first 20 cluster queues.

What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the latest commit 👍, TY @mbelton-buildkite

}
```

### List jobs as part of a specific cluster queue
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this still keep the right meaning?

Suggested change
### List jobs as part of a specific cluster queue
### List jobs in a cluster queue

Just looking if there's a sensible way to shorten it a bit.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the latest commit 👍, TY @mbelton-buildkite

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants