GSA / data.gov

Main repository for the data.gov service
https://data.gov
Other
618 stars 98 forks source link

Remove decommissioned data.gov records from 18f DNS #4233

Closed FuhuXia closed 1 year ago

FuhuXia commented 1 year ago

This inventory list should be kept up to date for any subdomain changes.

data.gov subdomains to be removed:

Please click any records that you are sure can be safely removed.

The following are CNAME records still pointing to BSP resources.

robert-bryson commented 1 year ago

Updated list above for removals/couple of missing records. See PR here. As per Fuhu's edits, I just copied over the terraform file to my fork. His work is still captured. I can't create another fork if I already have one, apparently.

robert-bryson commented 1 year ago

Moving to blocked, waiting on 18f PR and Touchpoints account reactivated.

robert-bryson commented 1 year ago

Just got merged!

FuhuXia commented 1 year ago

federation.data.gov reported down. We should not delete it.

robert-bryson commented 1 year ago

Need to be a site point of contact to be able to clean up the list on Touchpoints. I reached out to Ryan Wold to get myself added.

robert-bryson commented 1 year ago

@hkdctol, Ryan Wold set me as the Point of Contact for those sites so I am able to manage them myself. I'm communicating with him as per what I should be setting them to and also what the Point of Contact should be long term.

robert-bryson commented 1 year ago

I have gone through the list on Touchpoints and there are a number of differences between the list above and Touchpoints. A number of subdomains are not on the list in what would be the sketch for this story. Below are subdomains that are missing.

Subdomains on touchpoints that are not in 18f/dns terraform (have DNS_PROBE_FINISHED_NXDOMAIN error):

Subdomains on touchpoints that are in 18f/dns terraform (404s on data.gov side):

It is not clear to me either the origin of the list provided in the ticket or why these differences exist. I believe we can remove the former list from Touchpoints with the web ui and the latter with that and a PR to 18f/dns as well. Please let me know if otherwise, @FuhuXia & @hkdctol. If that sounds good, I will make the PR/decommission/archive them.

Lastly, I decommissioned the records listed in the ticket. I also clicked the archive button (as recommended by Ryan Wold), but I don't think they are indeed archived; the UI returns a message saying 'Authorization is Required' and the button is labelled 'For Registry Managers'.

robert-bryson commented 1 year ago

@hkdctol to find the details on Touchpoints go here (and filter for data.gov to make it a bit more readable).

When I say in 18f/dns terraform, I am referring to this file that is the actual definition of resources in code.

hkdctol commented 1 year ago

On climate, we did retain those old posts, could it redirect to https://data.gov/climate/1/? All the rest in that list of 6--I don't think we need.

robert-bryson commented 1 year ago

Ok, I will change that redirect and remove the rest. I assume that I'm free to go ahead and decommission/archive the first list?

hkdctol commented 1 year ago

let me ask ryan what that means

hkdctol commented 1 year ago

@robert-bryson we can proceed--thanks

FuhuXia commented 1 year ago

On climate, we did retain those old posts, could it redirect to https://data.gov/climate/1/? All the rest in that list of 6--I don't think we need.

This is done already in the code here. I just ran cf command cf map-route www-redirects climate.data.gov and now it is live.

robert-bryson commented 1 year ago

Looks like this PR was just approved (and hopefully merged soon!). Once in, this ticket can be closed.

robert-bryson commented 1 year ago

18f/dns PR merged!

Would be closed, but @FuhuXia caught that I missed cleaning up the redirects for these. See that PR here.