[09:37:28] 10Puppet, 10Infrastructure-Foundations, 10Patch-For-Review, 10User-jbond: puppetdb postgress server: fix dependcey loop - https://phabricator.wikimedia.org/T296550 (10jbond) With filippos latest patch the only outstanding error is ` May 27 08:10:10 filippo-pdb-01 puppet-agent[17242]: (/Stage[main]/Postgr... [11:35:02] I want to migrate a DNS entry managed by netbox to dns repo. I created https://gerrit.wikimedia.org/r/c/operations/dns/+/800709 and checked the wikitech DNS docs. Someone around for a review before I take action? [12:16:41] jelto: looking at it now. [12:17:24] I see IP address 208.80.154.15 has "gitlab-replica-new" pointing to it currently [12:17:45] And there already exists a "gitlab-replica" pointing to 208.80.154.15, the IP in your patch [12:18:22] sry... existing "gitlab-replica" points to 208.80.153.105 [12:18:39] So is it the case that 208.80.154.15 is replacing 208.80.153.105? [12:18:53] And the records linking 208.80.153.105 to "gitlab-replica" should be removed as part of this? [12:19:02] (same is true for the v6 entries btw) [12:20:43] topranks: thanks for looking! No I just want to move gitlab-replica from netbox to dns repo, without changing any record. Migration from -new suffixes will happen later. So I want to manage gitlab-replica in dns repo instead of in netbox. [12:22:03] and I did not delete the dns hostsnames in netbox yet (this is the reason the pipeline still fails). I wanted a review first before I take action. [12:22:18] ok.... but current Netbox created records point to 208.80.153.105 [12:22:36] and your patch has manually added records pointing to 208.80.154.15 [12:22:41] maybe a typo in the patch? [12:23:02] sorry... my bad :D [12:23:14] highly embarrasing but seems I made the typo... your change is good. [12:23:56] I guess you looked at gitlab-replica-new one line below. This is .15. But yeah, I just touched .105 :) [12:24:27] for the 1,000th time this week I confused my own self. yep. [12:27:20] no problem :) Ok then I'll go ahead and make the modifications in netbox and do a recheck [12:27:28] Ok I've updated the two Netbox entries so it won't try to make those records: [12:27:29] https://netbox.wikimedia.org/search/?q=gitlab-replica.wikimedia.org [12:27:49] I'll run the cookbook now, after which CI should be happy with the patch and I'll +1 [12:28:21] Ok great thanks a lot for doing that :) [12:28:28] out of curiosity why the change? There some downside to the Netbox created entries? [12:30:01] we want to do a migration between gitlab hosts somewhere next week and thought it feels better to have full control over the records and TTL. Also for future failover use-cases the dns repo seemed to be the better option. But I'm also open to other thoughts there [12:35:58] Yeah I can see the low-ttl use-case for a migration being important. [12:39:30] jelto: ok all looks good to me +1 if you want to merge the change [12:40:15] topranks: thanks a lot! I'll merge and run authdns-update after that [12:58:09] 10netops, 10Infrastructure-Foundations, 10SRE, 10Patch-For-Review: Finalise design extension of WMCS networks to new cloudsw in Eqiad rows E/F - https://phabricator.wikimedia.org/T304989 (10cmooney) [12:59:05] 10netops, 10Infrastructure-Foundations, 10SRE, 10Patch-For-Review: Configure cloudsw1-e4-eqiad and cloudsw1-f4-eqiad - https://phabricator.wikimedia.org/T304936 (10cmooney) 05Open→03Resolved Work for this is now completed, will update design task once confirmed there are no niggles with reimaging. [13:01:31] topranks: I have a similar change for the non-replica address gitlab.wikimedia.org in https://gerrit.wikimedia.org/r/c/operations/dns/+/800719. Do you want to proceed similar as before? [13:01:31] And sorry for the redundant work but I felt splitting this and starting with the replica would be more comfortable when doing this the first time (at least for me). [13:01:59] yeah no probs... and yes better to split into smaller changes I think. [13:02:02] let me have a look [13:03:49] thanks a lot :) [13:12:28] jelto: ok I think you should be set :) [13:12:50] topranks: thanks a lot for the quick help. I'll proceed with merge and authdns-update