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

add initial pass at infra diagram #2635

Merged
merged 1 commit into from
Aug 30, 2021
Merged

Conversation

spiffxp
Copy link
Member

@spiffxp spiffxp commented Aug 30, 2021

It's likely incomplete in a number of places, some of which I marked
with TODO. This is loosely supposed to represent all of the
infrastructure we manage, and the infrastructure we have yet to migrate.

This was generated from hand written source, but using a google-internal
tool, so I'm not sure if I can share the source from which it was generated

The connections are laid out poorly by the tool, so I wouldn't pay a ton
of attention to them. Roughly speaking solid lines are the path code
flows as it turns into a kubernetes/kubernetes release artifact. The
dotted lines are asynchronous or polling checks. The sample use case I had
in my head was the first cut of an in-development release off of a release
branch, thus including some testgrid dashboard changes.

One thing missing off the top of my head is things that run as Prow Jobs,
so some detail are missing like:

  • how k8s-triage is populated
  • how k8s-metrics is populated
  • how artifact promotion happens
  • github automation like label_sync or peribolos

My preference would be to merge and iterate with this, because this is about
as far as I have time to push this for now. I'm open to feedback though,
and interested most specifically in things that are missing from this
diagram, including external services not directly managed by us.

It's likely incomplete in a number of places, some of which I marked
with TODO. This is loosely supposed to represent all of the
infrastructure we manage, and the infrastructure we have yet to migrate.

This was generated from hand written source, but using a google-internal
tool, so I'm not sure if I can share the source from which it was generated

The connections are laid out poorly by the tool, so I wouldn't pay a ton
of attention to them. Roughly speaking solid lines are the path code
flows as it turns into a kubernetes/kubernetes release artifact. The
dotted lines are asynchronous or polling checks. The sample use case I had
in my head was the first cut of an in-development release off of a release
branch, thus including some testgrid dashboard changes.

One thing missing off the top of my head is things that run as Prow Jobs,
so some detail are missing like:
- how k8s-triage is populated
- how k8s-metrics is populated
- how artifact promotion happens
- github automation like label_sync or peribolos

My preference would be to merge and iterate with this, because this is about
as far as I have time to push this for now. I'm open to feedback though,
and interested most specifically in things that are missing from this
diagram, including external services not directly managed by us.
@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Aug 30, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: spiffxp

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@k8s-ci-robot k8s-ci-robot added wg/k8s-infra approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Aug 30, 2021
@spiffxp
Copy link
Member Author

spiffxp commented Aug 30, 2021

Rendered as a PNG for ease of review
k8s-infra-as-of-2021-q3

@spiffxp
Copy link
Member Author

spiffxp commented Aug 30, 2021

/kind documentation

@k8s-ci-robot k8s-ci-robot added the kind/documentation Categorizes issue or PR as related to documentation. label Aug 30, 2021
@spiffxp
Copy link
Member Author

spiffxp commented Aug 30, 2021

/cc @dims @thockin

@k8s-ci-robot k8s-ci-robot requested a review from dims August 30, 2021 20:20
@spiffxp
Copy link
Member Author

spiffxp commented Aug 30, 2021

/cc @listx

FYI @kubernetes/release-engineering I didn't get around to enumerating all of the releng-related projects in this pass, I'll cc in more folks specifically the next time I've got bandwidth to dig into that

@k8s-ci-robot k8s-ci-robot requested a review from listx August 30, 2021 20:21
@spiffxp
Copy link
Member Author

spiffxp commented Aug 30, 2021

/cc @ameukam

@k8s-ci-robot k8s-ci-robot requested a review from ameukam August 30, 2021 20:35
@ameukam
Copy link
Member

ameukam commented Aug 30, 2021

This is an amazing representation of what we are doing right now. We still discovering resources that need to be migrated (e.g. #1469 (comment)).

/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Aug 30, 2021
@k8s-ci-robot k8s-ci-robot merged commit 49417c3 into kubernetes:main Aug 30, 2021
@k8s-ci-robot k8s-ci-robot added this to the v1.23 milestone Aug 30, 2021
@spiffxp spiffxp deleted the diagram branch August 30, 2021 21:13
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

nice
late /lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. kind/documentation Categorizes issue or PR as related to documentation. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants