[01:00] jtv: quick pre-imp? === CyberJacob is now known as CyberJacob|Away [07:13] jtv: gmb: The following (taken from the most recent failure in the test lab) looks like a fallout from the recent 'label' work: http://paste.ubuntu.com/7118396/ [07:14] rvba: Damn. [07:15] rvba: I need to step out for a little while right now but I'll glom onto this as soon as I get back. Best to email it to jtv too as he knows more about the label stuff than I. [07:16] gmb: okay. I'll have a look now; let's talk about it when you're back. [07:17] rvba: Cool, thanks. [07:17] * gmb -> back in a wee while === CyberJacob|Away is now known as CyberJacob [07:33] gmb: Could you please have a look at: https://code.launchpad.net/~rvb/maas/bug-import-eph-crash/+merge/211673 [08:16] rvba: TypeError: install_image_from_simplestreams() takes at least 4 arguments (4 given) [08:16] lol? [08:16] :) [08:20] rvba: want me to review that? [08:21] bigjools: sure [08:22] bigjools: that's if you rather do that than talk to me :) [08:22] rvba: ok let's try and see if 3g holds up [08:22] I'll call you [08:34] * gmb returns [08:34] Thanks for fixing that rvba. Sorry I had to be afk for the duration. [08:34] * gmb didn't time it that way. Promise. [08:37] gmb: no worries [08:39] gmb: building a package with the latest trunk now [08:44] rvba: Is there any card from the new import script work that you'd like me to pick up in particular, or should I just grab something at random? === CyberJacob is now known as CyberJacob|Away [09:07] rvba: did my change fix ipmi in the lab? [09:09] bigjools: I was precisely in the process of testing this when I came across bug 1294516. [09:09] bug 1294516 in MAAS "maas-import-ephemerals crashes" [Critical,Fix committed] https://launchpad.net/bugs/1294516 [09:09] bigjools: now that it's fixed, I'm testing the package again. [09:10] rvba: cool. One more thing, unrelated, but when there's no boot images the architecture drop down is empty and there's no message saying why [09:11] remember the work we did for the power types? [09:11] that is missing in the arch dropdown [09:11] I added a card as this is a little more serious [09:11] as it will trap many people [09:11] I have to travel now, I will be late for the call later [09:11] Okay. [09:12] probably at the top of the hour instead [09:12] hope you don;t mind waiting for me [09:12] :) [09:12] :) [09:53] bigjools: the int. tests passed with the new package. Starting a new test run just for safety… [10:28] \o/ [10:28] Why is "maas-region-admin dbshell" broken again? I thought I'd fixed that. [10:28] jtv: yeah me too, I saw that earlier :/ [10:29] bigjools: Second run is not over yet but it's passed the point where it failed before. I think we can consider the bug fixed. [10:29] gmb: I'm going to work on the boot images stuff… so you're can grab the "Architecture drop-down empty and no message why (see description)" task if you want :) [10:30] rvba: WFM. [10:30] bigjools: Ahh, it's the --installed option. [10:32] jtv: whaaaa? [10:33] bigjools: run dbshell with the --installed option. [10:33] We still need a user-friendly error, don't we? [10:37] Do you guys think it makes sense to deal with boot images in the API globally on only on a cluster per cluster basis? [10:37] I mean, is it useful to have a way to list all the boot images? Or do we want the API to operate on a nodegroup object? [10:39] In one case, listing bootimages is done with /api/boot-images/op=list [10:39] In the other case, it's /api/nodegroups//boot-images/op=list [10:42] Think in batches. [10:42] I'd just make nodegroup a constraint on a global nodegroups listing. [10:42] Although... this isn't very scale-sensitive. [10:42] Never mind me. [10:43] Not really sure which option is best. [10:44] Nodes objects are defined "globally" (although they are related to nodegroups) because it was important to be able to manage them as a set. [10:44] In terms of the functionality we have in mind, having a separate list per nodegroup seems the most appropriate. [10:44] Interfaces are defined on nodegroups. [10:46] I think it all depends on how likely we are to need a cross-nodegroup listing. [10:47] For the uses we have, we plan to move away from that. [10:47] Yeah, defining this globally only gives us that: a cross-nodegroup listing. [10:48] Or rather "gives us only that" [10:48] If we need it in rare cases, I don't think we need to worry about a client having a few extra round trips to make. [10:48] Either "only" is correct. [10:50] All right, I'll define this only the clusters objects. It's definitely a bit cleaner to do it this way. === marcoceppi is now known as marcoceppi-mobil [14:24] HI, is anyone familiar with preseed settings for MaaS? I want to partition a disk while commissioning === marcoceppi-mobil is now known as marcoceppi === marcoceppi is now known as marco-traveling [15:18] hello fine people, is anyone available to review a maas related juju-core change? https://codereview.appspot.com/77850043 there are a couple of lines with doubts and you might have the answer [15:22] rvba: ^ pls :) [15:23] perrito666: sure, I'll have a look in a sec. [15:23] rvba: thank a lot [15:23] *thanks === cmagina-away is now known as cmagina [15:51] perrito666: I think I replied to your question, see the MP. [15:52] rvba: thank you,checking [16:16] * gmb -> afk for a while [16:47] rvba: your changes got applied, thanks for the input [16:48] np [16:58] hi rvba, is there a way to add and remove tags via the UI? [17:01] tych0: no, you have to use the API/CLI. [17:01] ok, cool === cmagina is now known as cmagina-away === cmagina-away is now known as cmagina === cmagina is now known as cmagina-away === cmagina-away is now known as cmagina === CyberJacob|Away is now known as CyberJacob === cmagina is now known as cmagina-away === cmagina-away is now known as cmagina === cmagina is now known as cmagina-away