/srv/irclogs.ubuntu.com/2014/10/20/#maas.txt

=== alexpilotti_ is now known as alexpilotti
=== CyberJacob|Away is now known as CyberJacob
=== allenap_ is now known as allenap
gmballenap, rvba: https://code.launchpad.net/~gmb/maas/use-node.start-instead-of-start_nodes-bug-1330765/+merge/238758 needs a review when you’ve time08:49
bigjoolsgmb: sorry I should have reviewed that today08:51
gmbbigjools: no worries. Y’all might be getting spammed in a minute anyway; I just tried to create an MP that had that one as a prerequisite, and forgot to add the prerequisite branch. Twice.08:53
gmbNot sure whether LP will kill the notifications for those two deleted MPs.08:53
bigjools\o/08:53
gmballenap, rvba: https://code.launchpad.net/~gmb/maas/use-node.start-instead-of-start_nodes-bug-1330765/+merge/238758 and https://code.launchpad.net/~gmb/maas/use-node.stop-instead-of-stop_nodes-bug-1330765/+merge/238853 are up for review when you’ve time.10:17
dimiternhey guys10:19
dimiterni've just filed bug 138323110:19
ubot5bug 1383231 in MAAS "Can reserve the same static IP twice and cannot release it completely" [Undecided,New] https://launchpad.net/bugs/138323110:19
gmballenap, rvba: Also https://code.launchpad.net/~gmb/maas/remove-stop_nodes-and-start_nodes-bug-1330765/+merge/238857 (these branches are in dependency order, btw)10:27
gmbdimitern: Looking…10:27
allenapgmb: I'll take a look in a bit.10:27
gmballenap: Ta10:27
gmballenap: Realise you’re busy as hell with the transaction stuff right now, so no rush; just putting it out there.10:27
dimiterngmb, I've updated the bug a bit - it's more of a case for better error message actually10:28
gmbokay10:28
dimiternnothing is broken, just confusing10:28
allenapdimitern: There's only one 50.100 reservation.10:30
allenapdimitern: Ah, I see you've updated the bug.10:31
dimiternallenap, yes, I initially though there's a duplicate, but that's not the case; just a better error message is needed10:31
gmbdimitern: Confirmed and triaged.10:31
dimiterncheers!10:32
gmb(Also found another bug)10:32
allenapdimitern: The bug description doesn't appear right still: the 50.100 address isn't in the "But the list still shows it:" list.10:32
dimiternallenap, looking10:33
dimiternallenap, dropped the list section, as it's no longer relevant10:34
allenaprvba: https://code.launchpad.net/~allenap/maas/transactions-redux-the-revenge/+merge/238864 passed the full test suite, but I suspect it might fail in practice.11:03
rvbaallenap: looking11:52
rvbaallenap: why are you using a method to patch things instead of simply configuring Django using the settings?11:57
roaksoaxrvba: this is why we changed the isolation stuff right?12:31
roaksoaxrvba: because the settings were being used in practice12:31
allenaprvba: To make sure it actually happens. A change was previously made to use SERIALIZABLE but it never made it into production because a change was missed to maas_local_settings.py. This way we can't avoid it.12:32
rvbaroaksoax: like allenap said, we never changed the isolation level.  We just made it explicit.12:33
rvbaallenap: I don't get it, why not put this in the settings the user can't modify?12:33
roaksoaxallenap: i do remember the change being made in maas_local_settings.py12:34
allenaprvba: Because they *can* modify maas_local_settings.py.12:34
roaksoaxallenap: otherwise my large maas deployment would have been failing altogether12:34
roaksoaxrvba: so the settings.py get's completely overwritten by what's in maas_local_settings.py ?12:34
rvbaroaksoax: individual settings are overridden by what's in maas_local_settings.py, yes.12:35
roaksoaxrvba: ok, so if settings.py set the ISOLATION to SERIALIZABLE, and not setting it in maas_local_settings.py would mean that the individual setting for that was not being replaced with what we have in maas_local_settings.py12:35
rvbaallenap: well, okay, I see your point.  I think that's completely overkill but okay.12:35
roaksoaxrvba: so that means we were using serializable12:35
rvbaroaksoax: yes12:36
roaksoaxthanks, so 1. we should have never introduced any changes to maas_local_Settings.py 2. we should have not change from serializable to the other read_committed12:36
allenaprvba: It also avoids generating another install-time conflict on maas_local_settings.py12:37
rvbaallenap: my point is that it should be in src/maas/settings.py12:37
roaksoaxnext time, please ask for my review before making changes to config files as the above12:37
allenaproaksoax: No, it was *NOT* using SERIALIZABLE isolation.12:37
rvbaallenap: if that's possibly considering the packaging messes with the config.12:37
allenapThe setting is per-database.12:37
roaksoaxallenap: settings.py was using serializable isolation12:37
allenapmaas_local_settings.py replaces the DATABASES setting.12:38
roaksoaxallenap: did you test that was the case?12:38
rvbaallenap: right, that's unfortunate but that's how it is indeed.12:38
allenapOMFG, read the code!12:38
roaksoaxallenap: did you actually ensured that it was the case? if settings.py was using serializable and maas_local_settings.py wasn't specifying the value, did you ensure that was the case?12:38
roaksoaxallenap: because IIRC< jtv didn't make the config change because he tested that settings.py wasn't being fully overwritten, only what was modified in maas_local_settings.py12:39
rvbaroaksoax: yeah, like I said, individual settings from maas_local_settings.py override what's in settings.py12:40
allenaproaksoax: No, I didn't test. However, the code is almost as simple as:12:41
allenapDATABASES = {... isolation_level: SERIALIZABLE}  # in maas.settings12:41
allenapDATABASES = {... isolation_level not specified}  # in maas_local_settings.py12:41
allenapIOW, the DATABASES value is being completely replaced.12:41
roaksoaxallenap: so then, why do we have a regression where we didn't have it before?12:42
rvbaroaksoax: are we sure it's a regression?12:43
roaksoaxrvba: yup12:43
allenaproaksoax: I'm fairly confident that any regression has been caused by another change. rvba, gmb: Do we ask OIL to try out SERIALIZABLE as a cowboy patch a while back?12:47
allenaps/Do/Did/12:47
roaksoaxallenap: we did12:47
roaksoaxallenap: we did test it12:47
allenaproaksoax: Could the regression come from losing that snippet of configuration in OIL?12:47
allenapAre we sure this is a regression in *trunk*?12:48
roaksoaxallenap: so we have been upgrading from trunk12:48
roaksoaxallenap: ie. we upgraded to 1.5, then various 1.6, then early 1.7, then late 1.712:49
roaksoaxallenap: the issue was seen in late 1.7 (beta4/5?(12:49
rvbaroaksoax: what's the latest version without that pb?12:49
allenaproaksoax: At some point the person upgrading will have been asked about a conflict in maas_local_settings.py. Could it be that the cowboy patch got overwritten at that point?12:49
allenaproaksoax: Sorry for my outburst earlier. I was/am stressed, tired, not very well, but nevertheless it wasn't cool.12:51
=== CyberJacob is now known as CyberJacob|Away
=== CyberJacob|Away is now known as CyberJacob
dimiternany reviewers available?14:22
dimiternI'd like to have this gomaasapi MP reviewed please - https://code.launchpad.net/~dimitern/gomaasapi/static-ipaddresses/+merge/23889314:22
gmbdimitern: I’ll take a look presently.14:23
dimiterngmb, thanks!14:23
=== jfarschman is now known as MilesDenver
gmbdimitern: Approved. I’m not an official reviewer of gomaasapi, but it doesn’t look like LP has actually noticed that :)14:50
dimiterngmb, cheers! :)14:50
dimiterngmb, there's no bot for gomaasapi right? I just need to merge my branch into lp:gomaasapi?14:51
gmbdimitern: I think so.14:51
gmbCan’t remember there being a bot.14:51
dimiterngmb, ok then14:51
=== jfarschman is now known as MilesDenver
=== jfarschman is now known as MilesDenver
=== roadmr is now known as roadmr_afk
th3rtI've got maas setup to be my dns server for newly bootstrapped instances.  Maas is assigning DNS entries based on IP addresses instead of DNS entries based on hostnames.  Can anyone tell me what I misconfigured to cause that to happen?17:06
th3rtFor example,  instead of a dns entry for 7f9wx.maas,  I have one for 10-76-16.1.maas17:07
th3rtwhere 7f9wx is the hostname of the node17:07
th3rtand 10.76.16.1 is the IP17:07
blake_rth3rt: what version of maas?17:08
th3rtdpkg -l output... 1.5.4+bzr2294-017:09
blake_rth3rt: you have ubuntu deployed on the node?17:09
th3rtI do17:09
blake_rth3rt: there is no DNS entry for 7f9wx.maas?17:10
blake_rth3rt: did you enable DNS management on that interface for the cluster?17:10
th3rtNo,  there is no dns entry for 7f9wx.maas and yes,  I did enable DNS (and DHCP) for that interface on the cluster.17:11
blake_rth3rt: any error in region celery.log?17:12
th3rtI have an error from more than 5 hours ago,  but nothing current.17:14
th3rt[2014-10-20 06:40:01,767: ERROR/Worker-2] rndc_command: Command `rndc -c /etc/bind/maas/rndc.conf.maas reload` returned non-zero exit status 1:17:14
blake_rcheck the bind configurations to see if that hostname is in the config17:16
th3rtgrep -R '7f9wx' /etc/bind/maas/ returns nothing17:17
blake_rhmm17:17
th3rtWhen I look at the zone.* files in /etc/bind/maas I have a bunch of DNS entries based on the nodes IP addresses17:17
th3rtSo I can dig 10-76-16-1.maas17:18
th3rtbut not 7f9wx.maas17:18
blake_ryeah you should have a cname record that maps to that17:18
blake_rcheck through the other maas log to see if you see any other errors17:19
th3rtWell,  I don't have any cname records in my zone files17:19
th3rtcelery.log shows17:23
th3rt[2014-10-20 12:20:42,623: INFO/MainProcess] Received task: provisioningserver.tasks.upload_dhcp_leases[83572b37-16af-42ee-b6ba-b6341eb97e24] expires:[2014-10-20 17:21:42.619675+00:00]17:23
th3rt[2014-10-20 12:20:42,628: INFO/Worker-6] The DHCP leases file does not exist.  This is only a problem if this cluster controller is managing its DHCP server.  If that's the case then you need to install the 'maas-dhcp' package on this cluster controller.17:23
th3rtmaas-dhcp is installed and running on the host17:23
th3rt... and handing out IPs17:23
=== roadmr_afk is now known as roadmr
blake_rth3rt: if it cannot parse the leases file then the cname mapping is not made17:30
blake_rth3rt: sudo maas-dhcp-server status17:30
th3rtAlright.  that explains my problem :-)17:31
th3rtservice maas-dhcp-server status17:31
th3rtmaas-dhcp-server start/running, process 2827217:31
th3rt/var/lib/maas/dhcp has no leases file present17:31
th3rtand looks like maybe it should17:31
blake_ryeah it should17:31
th3rt/var/lib/dhcp has a leases file...17:31
blake_rmaybe permissions issue17:31
blake_rthat file is not used17:31
th3rtok17:31
blake_rcheck syslog for dhcp issues17:32
th3rtHah,  found it.  I moved /etc/dhcp and /var/lib/dhcp out of the way and MaaS created it's own leases file in /var/lib/maas/dhcp/17:36
blake_rth3rt: awesome, glad its fixed17:49
th3rtblake_r,  thanks for the help!18:12
blake_rth3rt: np18:12
=== jfarschman is now known as MilesDenver
=== CyberJacob is now known as CyberJacob|Away

Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!