| === CyberJacob is now known as CyberJacob|Away | ||
| jtv | bigjools: looks like "make lint" is broken in a really weird way... tries to run "tail" on lp:~maas-maintainers/maas/packaging/debian/changelog (not even head, which would seem to make more sense) | 02:14 |
|---|---|---|
| bigjools | jtv: whut | 02:16 |
| bigjools | jtv: oh ARGH | 02:17 |
| bigjools | my change has a boo boo | 02:17 |
| * bigjools fixes | 02:17 | |
| jtv | Ah, it's all "make" attempts that get this — it's just that the message is sometimes drowned out by other output. | 02:23 |
| bigjools | jtv: yes, I was convinced by Gavin to use the lp branch but forgot it needs checking out somewhere ... | 02:24 |
| bigjools | so I've gone back to the old way for now until I figure out a sensible approach | 02:24 |
| bigjools | jtv: just got a random test failure | 02:35 |
| bigjools | ERROR: maasserver.models.tests.test_node.NodeManagerTest.test_node_can_be_multiple_networks | 02:36 |
| bigjools | ValidationError: {'vlan_tag': [u'Network with this Vlan tag already exists.']} | 02:36 |
| jtv | lemme look | 02:36 |
| jtv | Aigh. | 02:37 |
| jtv | Nastiness. | 02:37 |
| jtv | Not much we can do about that one I suspect. | 02:38 |
| jtv | Unless... | 02:38 |
| jtv | We could start writing "plural" factory methods. | 02:38 |
| jtv | "Make me <n> networks." | 02:38 |
| jtv | Which would then be responsible for preventing these clashes. | 02:38 |
| bigjools | jtv: do you know how to conditionally set a variable in Makefile? It evaluates everything on startup, even stuff inside a target | 02:48 |
| bigjools | I tried some shell but the value gets thrown away after... | 02:48 |
| jtv | No, haven't had time to mess with that stuff for years... Gavin's the one to ask. | 02:48 |
| * jtv used to be knee-deep in GNU Make. | 02:49 | |
| bigjools | me too | 02:49 |
| bigjools | not done one from scratch for 8 yuears | 02:49 |
| jtv | The friend and former colleague who showed me how to set up all the really hard stuff isn't even alive any more... | 02:50 |
| jtv | Didn't quite get to see his 20th anniversary with Debian. :( | 02:51 |
| jtv | I don't suppose we care if our Makefile is GNU-specific. | 02:52 |
| bigjools | victory is mine | 03:18 |
| jtv | No! I saw it first! | 03:19 |
| jtv | Er, what? | 03:19 |
| bigjools | makefile victory | 03:31 |
| bigjools | it's looking nice now | 03:31 |
| bigjools | branch forthcoming | 03:31 |
| jtv | But does it work? | 03:31 |
| bigjools | yes! | 03:31 |
| jtv | Review also forthcoming. | 03:31 |
| jtv | Once you let me know the MP is ready, of course. | 03:31 |
| === freeflying is now known as freeflying_away | ||
| === mwhudson is now known as zz_mwhudson | ||
| === freeflying_away is now known as freeflying | ||
| bigjools | jtv: https://code.launchpad.net/~julian-edwards/maas/makefile-packaging-target/+merge/207348 | 03:39 |
| jtv | Coming right up. | 03:40 |
| bigjools | the only thing I know will fail is if you have no-trees on by default (like me) when branching | 03:43 |
| bigjools | but there's no option that's the opposite of --no-tree on branch :( | 03:43 |
| bigjools | jtv: what is that migration actually changing in PG? | 06:10 |
| bigjools | I can't see any change | 06:10 |
| bigjools | other than the django field name | 06:10 |
| jtv | It might change it from a fixed-length varchar to a regular varchar. | 06:11 |
| jtv | But yes, this is more of a UI change. Did anyone mention lately that the layers get a little muddled in Django? | 06:11 |
| bigjools | jtv: django does not do varchar | 06:12 |
| bigjools | everything is fixed | 06:13 |
| jtv | Well... it creates varchar fields. | 06:13 |
| jtv | But of fixed lenghts. | 06:13 |
| bigjools | look at the migration, nothing changed! | 06:13 |
| jtv | It's called "varchar" because nul-terminating strings was a novel thing at the time. | 06:13 |
| bigjools | heh | 06:13 |
| jtv | Space-padding used to be quite normal and socially acceptable. | 06:13 |
| jtv | Now, I hesitate to mention it in public. | 06:13 |
| jtv | Argh! Yup, it's still 255 characters wide. Damn. | 06:14 |
| bigjools | I think the field is only used as a hint for the UI | 06:15 |
| jtv | The difference in field width is, yes. | 06:15 |
| jtv | I mean | 06:15 |
| jtv | sorry | 06:15 |
| jtv | The difference in field _type_. | 06:16 |
| bigjools | yes | 06:16 |
| jtv | Want me to bump up the inanely fixed maximum size? | 06:16 |
| jtv | Oh wait, it looks like you _can_ remove that size on a TextField, and get a proper varchar! | 06:17 |
| bigjools | ! | 06:27 |
| jtv | Unstoppable progress. Plunging head-first into the 1990s. | 06:33 |
| bigjools | Django - unchained | 06:40 |
| jtv | bigjools: I updated my multiline-network-description branch with the unlimited description size. Could you give it another go? | 06:56 |
| bigjools | jtv: done | 06:57 |
| jtv | Thanks. | 06:57 |
| bigjools | jtv: make sure input is sanitised before display | 06:57 |
| bigjools | don't want another xss | 06:57 |
| jtv | That's what I added that test for. | 06:58 |
| jtv | Don't know why that wasn't tested in the original Vlan class, but the <pre> doesn't really change anything there. | 06:58 |
| bigjools | ok | 07:01 |
| jtv | rvba: better get landing. :) | 07:09 |
| rvba | jtv: landing my branches now. | 07:11 |
| jtv | \o/ | 07:11 |
| rvba | jtv: I've got another one up for review: https://code.launchpad.net/~rvb/maas/network-mac-link-net-page/+merge/207210 | 07:11 |
| jtv | On it. | 07:11 |
| bigjools | hey rvba | 07:12 |
| rvba | Thanks. | 07:12 |
| rvba | \o bigjools | 07:12 |
| === CyberJacob|Away is now known as CyberJacob | ||
| rvba | jtv: rarg, you landed a migration. It's conflicting with my branch… I'm fixing my branch now… | 07:56 |
| jtv | Oh! Yes... I didn't realise we'd be clashing for a migration number. Very sorry. | 07:57 |
| jtv | bigjools: any chance you could look at this MP? https://code.launchpad.net/~jtv/maas/enlist-restore-avahi-code/+merge/207362 | 07:58 |
| bigjools | jtv: sorry dinner time, later if nobody beat me to it | 08:11 |
| bigjools | and I was OTP before | 08:11 |
| rvba | jtv: I'll have a look in just a sec. | 08:11 |
| rvba | Fixing the migration first. | 08:11 |
| rvba | (Was OTP before too) | 08:11 |
| === CyberJacob is now known as CyberJacob|Away | ||
| rvba | jtv: conflict in https://code.launchpad.net/~jtv/maas/api-connect_macs/+merge/207374 | 10:18 |
| bigjools | from maasserver import messages | 10:23 |
| bigjools | ImportError: cannot import name messages | 10:23 |
| bigjools | loads of that when I run tests on trusty | 10:23 |
| jtv | Grrr | 10:25 |
| jtv | bigjools: got the packaged apiclient installed? | 10:25 |
| jtv | Need to remove that. | 10:25 |
| bigjools | oh problaby.... meh | 10:26 |
| jtv | Sometimes I think we need not just a file with requirements but one with antirequirements as well. | 10:26 |
| jtv | "If you have any of these installed, things will break." | 10:26 |
| bigjools | I wonder if we can fix this one day | 10:27 |
| bigjools | it's useful to have packages installed as well | 10:27 |
| jtv | Yes... I wonder why the branch versions don't simply take precedence over installed packages. | 10:28 |
| rvba | jtv: I guess a cleanup branch is in order to use the new make_networks() factory method in all the code I just landed. | 10:33 |
| jtv | rvba: always nice! | 10:33 |
| jtv | rvba: want me to do it? | 10:35 |
| rvba | jtv: if you don't mind. I'd like to work on the "Disallow single MAC on multiple non-VLAN networks" task. | 10:36 |
| rvba | It's not trivial. Requires the introducing of form-level validation where we don't even use forms. | 10:37 |
| jtv | :-/ | 10:40 |
| arges | hi. is there some docs on xinstall_preseed with curtin? i'd like to be able to use the fastpath installer and then install an hwe kernel on a maas node. Thanks | 14:21 |
| arges | http://astokes.org/customizing-fastpath-curtin-installations/ answers it | 14:35 |
| === freeflying is now known as freeflying_away | ||
| === chuck_ is now known as zul | ||
| evilnickveitch | hello. does the tftp server for images live on the cluster controller or the region controller? | 19:01 |
| roadmr | evilnickveitch: methinks it lives on the cluster controller, but I'm just speculating | 19:02 |
| evilnickveitch | the docs tend to suggest it does, but if i configure rgion controller to a different address, the tftp server seems to disappear too | 19:03 |
| evilnickveitch | but thanks for your speculation roadmr :) | 19:03 |
| roadmr | oops :( that I have no idea, I have both on the same hardware | 19:04 |
| === CyberJacob|Away is now known as CyberJacob | ||
| === zz_mwhudson is now known as mwhudson | ||
| ccope | does anyone know if Canonical sells support for MAAS? | 19:22 |
| ccope | nvm found it in a pdf, the answer is yes | 19:46 |
| === mwhudson is now known as zz_mwhudson | ||
| === zz_mwhudson is now known as mwhudson | ||
| === kevin is now known as Guest24427 | ||
| === CyberJacob_ is now known as cyberJacob_ | ||
| === cyberJacob_ is now known as CyberJacob_ | ||
| === CyberJacob_ is now known as cyberjacob_ | ||
| === cyberjacob_ is now known as CyberJacob | ||
| === CyberJacob is now known as CyberJacob_ | ||
| === CyberJacob_ is now known as CyberJacob | ||
| === tooth_ is now known as tooth | ||
| === ppetraki_ is now known as ppetraki | ||
| === freeflying_away is now known as freeflying | ||
| bigjools | evilnickveitch: yes the images all live on the cluster | 23:49 |
| bigjools | but the cluster needs to talk to the region to ascertain the kernel params and whatnot | 23:50 |
| === CyberJacob is now known as CyberJacob|Away | ||
Generated by irclog2html.py 2.7 by Marius Gedminas - find it at mg.pov.lt!