[13:09] anyone able to help me validate if my constraints tag is proper? juju-gui doesn’t seem to be respecting my constraints [13:09] in a v4 bundle [13:10] firl: what's the constraint? [13:10] rick_h_ http://pastebin.com/ULBjUtiy [13:10] constraints: tags=compute [13:11] maas tag specifically [13:11] it fails completely when I try to put it on the service layer [13:11] firl: looking sec [13:11] ty [13:12] firl: did you try putting " " around it? [13:12] I did, but I can try again [13:12] firl: see https://jujucharms.com/docs/stable/charms-bundles#local-deploy-via-command-line [13:12] yeah the service constraints didn’t work at all [13:13] let me rebootstrap and try with quotes around it [13:13] firl: I was looking at the machine one vs service ones [13:13] firl: can have Makyo look when he's in, next 20ish [13:14] cool; yeah just trying to create a repeatable way to just tag systems and deploy to them in the lab [13:15] firl: oh, but is that just the part of the file? [13:15] what? [13:15] firl: never mind, ignore me [13:15] haha ok [13:16] I don’t see the tags constraint in the juju-gui, but I assume it’s supposed to still respect it in the bundle file right? [13:16] firl: checking [13:17] firl: yea, it's there https://github.com/juju/juju-bundlelib/blob/master/jujubundlelib/validation.py#L27 [13:17] cool [13:17] firl: ok so how are you deploying the bundle? [13:17] firl: juju-quickstart "filename"? [13:18] no, drag and drop [13:18] ah ok [13:18] i can try the juju-quickstart file name [13:18] no, that's ok. [13:18] kk [13:18] ahhhhhhhhh [13:18] oh no...you're dragging/dropping in a real enviornment right? [13:18] yeah [13:18] juju bootstrap --to machine.name [13:19] ok cool [13:19] juju deploy juju-gui --to 0 && juju expose juju-gui [13:20] cool, yea that should all work. [13:20] kk [13:21] ( I had an issue with quickstart originally I think, don’t remember why, that’s why I went the juju-gui route ) [13:21] firl: ah but they both use the same path [13:21] juju-guickstart just passes it to the gui [13:22] that’s what I figured; but still for some reason didn’t like it [13:22] I can try to replicate via juju quickstart [13:22] that's ok, what's the exact error you get? [13:22] * rick_h_ can search for it [13:22] 1s let me replicate [13:24] frankban: http://pastebin.com/ULBjUtiy [13:24] rick_h_: An error occurred while deploying the bundle: json: cannot unmarshal string into Go value of type []string [13:24] drag and drop into the UI works I think [13:25] rick_h_: http://pastebin.com/5McramHp [13:26] same error with juju-deployer, dragging and dropping seems to “work" [13:26] but ignores constraints [13:35] rick_h_: even with “ ’s around it, no dice [13:42] firl: thansk for trying. Makyo and frankban are looking into duplicating it and chasing it down [13:42] cool, if you want me to try any specific versions / patches let me know [13:43] firl: will do, <3 [13:43] I can open a bug also if it warrants it and I am not just using it wrong [13:43] frankban: app.db.machines.item(4) [13:44] afk for a bit [13:45] frankban_: app.env.get('ecs').changeSet === alexpilotti_ is now known as alexpilotti [14:28] uiteam, Makyo: https://code.launchpad.net/~frankban/juju-deployer/fix-tags-constraint/+merge/270070 [14:28] tvansteenburgh: could you please take a look? [14:28] ^ [14:58] firl: hey we investigated and found two bugs in how the tags constraints are provided to juju, on ein the GUI and one in the deployer. The fix for the deployer already landed on trunk, and we are working on the GUI side [14:59] tvansteenburgh: thanks for the review! [14:59] frankban: np, let me know if/when you need a new release === alexpilotti_ is now known as alexpilotti [16:47] frankban: woohoo! thanks [16:47] frankban: how can i test it on the deployer side? === rogpeppe2 is now known as rogpeppe [20:03] marcoceppi: you ever used mojo? [20:46] rick_h_ , frankban , Makyo : I just tried the latest deployer, it deployed, but I think it didn’t respect the tags again. Should I also have a different version of the juju to go with the new deployer? [21:02] firl: no, just the deployer, but not through the GUI [21:03] rick_h_: Yeah I used juju-deployer, and I no longer get the error, however doesn’t respect the tags [23:18] rick_h_: I know enough about it but I haven't actually used it much in depth