Add DNS configs #139

thockin · 2018-09-28T18:07:52Z

This is carried over from the work-in-progress repo in prep to go live.

@hh @kubernetes/k8s-infra-team

This is carried over from the work-in-progress repo in prep to go live.

ixdy · 2018-09-28T18:26:27Z

dns/zone-configs/k8s.io.yaml

+  type: CNAME
+  value: netlifyglobalcdn.com.
+
+# Not sure what these control, no prior notes.


This was to verify k8s.io and kubernetes.io as being the official URLs for these orgs, per https://blog.github.com/changelog/2018-08-07-domain-verification/. I worked with @cblecker on this, I can't remember if there was a tracking issue.

kubernetes/org#4

This falls under github management which is a sig-contributor-experience subproject, @cblecker is a fine PoC, or @kubernetes/owners if you want to de-SPOF

ixdy · 2018-09-28T18:27:23Z

dns/zone-configs/k8s.io.yaml

+www:
+  type: CNAME
+  value: k8s.io.
+_acme-challenge:


this was probably from a LetsEncrypt run that wasn't properly deleted; I think we can remove it from this file.

ixdy · 2018-09-28T18:29:23Z

dns/zone-configs/k8s.io.yaml

+  type: CNAME
+  value: redirect.k8s.io.
+# TODO: Where does this live and who owns it?
+cherrypick:


was this the cherrypick dashboard, part of mungegithub, which I think is now dead? @spiffxp @cjwagner

yes, kill it with fire

It seems dead. Removing it.

ixdy · 2018-09-28T18:32:07Z

dns/zone-configs/k8s.io.yaml

+node-perf-dash:
+  type: A
+  value: 130.211.155.47
+# Who owns this and where is it running?


This is https://github.com/kubernetes/perf-tests/tree/master/perfdash, and looks like @krzysied is the best contact.

sig scalability

ixdy · 2018-09-28T18:32:54Z

dns/zone-configs/k8s.io.yaml

+pr-test:
+  type: CNAME
+  value: redirect.k8s.io.
+# Who owns this and where is it running?


currently the Google EngProd team, presently running in GKE project k8s-prow.

ixdy · 2018-09-28T18:36:45Z

dns/zone-configs/k8s.io.yaml

+sigs:
+  type: CNAME
+  value: redirect.k8s.io.
+# Who owns this and where did we get that IP?


This is a docker container running on a GCE VM. Originally set up by @goltermann, I think @grodrigues3 and @roberthbailey are the current owners.

ixdy · 2018-09-28T18:37:16Z

dns/zone-configs/k8s.io.yaml

+submit-queue:
+  type: CNAME
+  value: redirect.k8s.io.
+# Who owns this and do we need it?


I think this is for aws kops testing, right @justinsb ?

cblecker · 2018-09-28T18:53:07Z

/assign
Planning to review :)

spiffxp

Incomplete review. Before I do more ownership comments, I guess I should ask what you're looking for here? person? subproject? sig? all? etc.

spiffxp · 2018-09-28T19:04:45Z

dns/zone-configs/k8s.io.yaml

+  type: CNAME
+  value: redirect.k8s.io.
+# TODO: Where does this live and who owns it?
+cherrypick:


yes, kill it with fire

spiffxp · 2018-09-28T19:05:42Z

dns/zone-configs/k8s.io.yaml

+devstats:
+  type: CNAME
+  value: k8s.devstats.cncf.io.
+# TODO: jorge and paris and which sig?


devstats and discourse are both sig-contributor-experience

spiffxp · 2018-09-28T19:07:05Z

dns/zone-configs/k8s.io.yaml

+  type: CNAME
+  value: redirect.k8s.io.
+# Web frontend for unauthenticated GCS access.  Running in GKE (@thockin).
+gcsweb:


The code for this lives in k/test-infra at the moment, I was unaware you were its steward @thockin should we get you in an OWNERS file for it?

I wrote it, but I haven't touched it in a long time (mostly doesn't need touching), so sure if you want... :)

spiffxp · 2018-09-28T19:10:01Z

dns/zone-configs/k8s.io.yaml

+  type: CNAME
+  value: netlifyglobalcdn.com.
+
+# Not sure what these control, no prior notes.


kubernetes/org#4

This falls under github management which is a sig-contributor-experience subproject, @cblecker is a fine PoC, or @kubernetes/owners if you want to de-SPOF

spiffxp · 2018-09-28T19:12:30Z

dns/zone-configs/k8s.io.yaml

+node-perf-dash:
+  type: A
+  value: 130.211.155.47
+# Who owns this and where is it running?


sig scalability

spiffxp · 2018-09-28T19:14:01Z

dns/zone-configs/k8s.io.yaml

+testgrid:
+  type: CNAME
+  value: redirect.k8s.io.
+# Project metrics.  Running in GKE (@apelisse, @jgrafton)


At this point I would swap in myself or @cjwagner for this

spiffxp · 2018-09-28T19:15:43Z

dns/zone-configs/k8s.io.yaml

+contributor:
+  type: CNAME
+  value: kubernetes-contributor.netlify.com.
+cs:


@dims what sig?

closest is probably sig-testing.

thockin · 2018-09-28T22:37:15Z

Added a bunch of names (yay) and re-pushed. Once this merges, we can take PRs to update the comments.

cblecker · 2018-09-28T22:44:54Z

/lgtm
👍

spiffxp · 2018-09-28T22:56:44Z

/lgtm
enough to iterate on

thockin · 2018-09-28T23:10:50Z

/approve

k8s-ci-robot · 2018-09-28T23:10:55Z

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: thockin

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:

~~OWNERS~~ [thockin]

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

BenTheElder · 2018-09-29T02:37:48Z

dns/zone-configs/kubernetes.io.yaml

+  value: 104.197.79.9
+spartakus:
+  type: CNAME
+  value: spartakus.k8s.io.


does this exist? spartakus.k8s.io seems dead

BenTheElder · 2018-09-29T02:41:15Z

dns/zone-configs/k8s.io.yaml

+  value: redirect.k8s.io.
+gubernator:
+  type: CNAME
+  value: redirect.k8s.io.


this seems to go to kubernetes.io now, it should probably go to https://k8s-gubernator.appspot.com/?

Add DNS configs

853ec9d

This is carried over from the work-in-progress repo in prep to go live.

thockin assigned dims Sep 28, 2018

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 Sep 28, 2018

k8s-ci-robot requested review from ixdy and rmmh September 28, 2018 18:08

ixdy reviewed Sep 28, 2018

View reviewed changes

k8s-ci-robot assigned cblecker Sep 28, 2018

spiffxp reviewed Sep 28, 2018

View reviewed changes

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 28, 2018

Document a bunch of records

5944e49

thockin force-pushed the master branch from bb917ca to 5944e49 Compare September 28, 2018 22:45

k8s-ci-robot removed the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 28, 2018

k8s-ci-robot assigned spiffxp Sep 28, 2018

k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Sep 28, 2018

k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 28, 2018

k8s-ci-robot merged commit 5880c6b into kubernetes:master Sep 28, 2018

BenTheElder reviewed Sep 29, 2018

View reviewed changes

dims mentioned this pull request Nov 8, 2018

[Umbrella issue] Automate DNS for Kubernetes assets #151

Closed

Add DNS configs #139

Add DNS configs #139

Conversation

thockin commented Sep 28, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

cblecker commented Sep 28, 2018

spiffxp left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

thockin commented Sep 28, 2018

cblecker commented Sep 28, 2018

spiffxp commented Sep 28, 2018

thockin commented Sep 28, 2018

k8s-ci-robot commented Sep 28, 2018

BenTheElder Sep 29, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

BenTheElder Sep 29, 2018 •

edited

Loading