Wednesday, 2015-11-25

*** tellesnobrega_af is now known as tellesnobrega00:02
*** witlessb has quit IRC00:32
*** egafford has joined #openstack-sahara00:43
*** jinxing has joined #openstack-sahara00:46
*** egafford has quit IRC00:49
*** tellesnobrega is now known as tellesnobrega_af00:49
*** EinstCrazy has joined #openstack-sahara02:08
*** jinxing has quit IRC02:26
*** ViswaV has quit IRC04:11
*** ViswaV has joined #openstack-sahara04:15
*** houming has joined #openstack-sahara04:47
*** hdd has joined #openstack-sahara05:36
*** Poornima has joined #openstack-sahara05:49
*** hdd has quit IRC06:06
*** houming has quit IRC06:07
*** rcernin has joined #openstack-sahara06:19
*** sgotliv has joined #openstack-sahara06:25
*** nkrinner has joined #openstack-sahara06:26
*** houming has joined #openstack-sahara06:26
*** vgridnev has joined #openstack-sahara06:54
*** sgotliv has quit IRC07:27
*** rcernin_ has joined #openstack-sahara07:47
*** tosky has joined #openstack-sahara07:50
*** vgridnev has quit IRC08:26
*** tosky has quit IRC08:35
*** esikachev has joined #openstack-sahara08:42
*** tosky has joined #openstack-sahara08:43
openstackgerritAndrey Pavlov proposed openstack/python-saharaclient: Adding ability to get plugin processes via CLI  https://review.openstack.org/24567409:07
openstackgerritEvgeny Sikachev proposed openstack/sahara: [do not review]test commit  https://review.openstack.org/24928109:07
*** ViswaV_ has joined #openstack-sahara09:08
*** ViswaV has quit IRC09:08
*** witlessb has joined #openstack-sahara09:19
*** vgridnev has joined #openstack-sahara09:31
*** _degorenko|afk is now known as degorenko09:35
openstackgerritEvgeny Sikachev proposed openstack/sahara: [do not review]test commit tempest  https://review.openstack.org/24928109:44
*** bapalm has quit IRC09:53
*** bapalm has joined #openstack-sahara09:55
*** rcernin_ has quit IRC10:06
*** sgotliv has joined #openstack-sahara10:09
*** EinstCrazy has quit IRC10:13
*** AndreyPavlov has joined #openstack-sahara10:15
*** ViswaV_ has quit IRC10:18
*** houming has quit IRC10:29
*** AndreyPavlov has quit IRC11:15
*** AndreyPavlov has joined #openstack-sahara11:20
*** tellesnobrega_af is now known as tellesnobrega11:28
openstackgerritEvgeny Sikachev proposed openstack/sahara: [do not review]test commit tempest  https://review.openstack.org/24928111:30
openstackgerritEvgeny Sikachev proposed openstack/sahara: [do not review]test commit  https://review.openstack.org/24928111:40
*** jamielennox is now known as jamielennox|away11:42
*** jamielennox|away is now known as jamielennox12:01
*** houming has joined #openstack-sahara12:10
*** raildo-afk is now known as raildo12:23
*** Poornima has quit IRC12:43
*** jinxing has joined #openstack-sahara12:57
*** chlong has joined #openstack-sahara13:03
*** david-lyle has quit IRC13:09
openstackgerritEvgeny Sikachev proposed openstack/sahara-ci-config: Update timeout for tempest tests  https://review.openstack.org/24977013:12
*** houming has quit IRC13:12
*** crobertsrh1 has joined #openstack-sahara13:13
*** AndreyPavlov has quit IRC13:14
*** AndreyPavlov has joined #openstack-sahara13:20
openstackgerritAndrey Pavlov proposed openstack/python-saharaclient: Adding indications of results after delete operations  https://review.openstack.org/24978413:36
*** esikachev has quit IRC13:41
openstackgerritAndrey Pavlov proposed openstack/python-saharaclient: Adding indications of results after delete operations  https://review.openstack.org/24978413:42
*** esikachev has joined #openstack-sahara13:47
*** houming has joined #openstack-sahara13:48
*** crobertsrh1 has quit IRC13:57
*** jinxing has quit IRC14:05
*** openstackgerrit has quit IRC14:06
*** jinxing has joined #openstack-sahara14:06
*** openstackgerrit has joined #openstack-sahara14:06
openstackgerritEvgeny Sikachev proposed openstack/sahara: [do not review]test commit tempest  https://review.openstack.org/24928114:15
*** rcernin has quit IRC14:19
openstackgerritAndrey Pavlov proposed openstack/sahara: Launching 1 instance in grenade instead of 2  https://review.openstack.org/24981014:22
*** tmckay has joined #openstack-sahara14:26
*** egafford has joined #openstack-sahara14:30
egaffordSergeyLukjanov: ping14:31
egafford(If still sick, sorry.)14:32
elmikocan't you just let the man rest!14:33
elmiko;)14:33
egaffordelmiko: He said two days, I waited two days.14:34
pino|workso strict14:34
elmikook, then, have at it!14:34
egaffordIf he's still sick, he can rest; if he's back, I ping.14:34
egaffordImages aren't going to generate their own in-service API.14:34
elmikomr. gafford, one ping only...14:34
elmikothis is true, but it would be cool if they did14:35
egaffordelmiko: While you're around, question: have we ever put any thought into using snapshots as our images?14:35
elmikoegafford: iirc, we briefly talked about it, but i don't remember if the talk went anywhere14:35
elmikoit's certainly an interesting idea, from a theoretical standpoint14:36
egaffordI've been bopping around different ways to create a build env and modify an image file within that build env (as that's our current tech,) but doing that in Nova is at least as cumbersome as spawning an image, modifying, snapshotting. I'm worried about Ironic from that standpoint, of course; if we went down that road we might hose ourselves there.14:36
egaffordIt'd also be more cumbersome to generate multiple images in one go with snapshots.14:37
egaffordStill, it's an interesting option, and would remove a lot of complexity around the build env.14:37
*** tellesnobrega is now known as tellesnobrega_af14:38
elmikodoes snapshotting an image produce a sparse qcow2 or something?14:38
egaffordelmiko: I added an "alternative", btw, to the image-elements merge spec suggesting a very different path, in which we never merge SIE, but just do a completely different thing in the SPI r/t idempotent image validation which could grow to replace or complement SIE organically. I actually *prefer* this approach, in many ways; it's sort of a v2 API approach to the problem. :)14:40
egaffordelmiko: Still, there seemed to be some real desire to stick with SIE and elements, and my early experiments seem to bear out that using elements as an interface for a different backend is quite possible with a little work.14:40
egaffordI might write 2 duelling specs and let the team decide between them.14:41
elmikocool, i meant to read it again last night, but failed =(14:42
egaffordelmiko: I respect that.14:42
elmikoi'll get to it today though14:42
egaffordEh, I don't think I'm going to be able to move in earnest on this stuff until chatting with both Mirantis Sergeys.14:42
egaffords/'m going to be able to/should/, really.14:43
openstackgerritEvgeny Sikachev proposed openstack/sahara: Fix skiped tempest test  https://review.openstack.org/24928114:43
elmikoegafford: did you see my note in the review about another possible impl, whereby the plugins just generate some sort of config data that an sie-like tool would consume?14:43
egaffordI can, I just want to get the team together on direction first, because it's important and I like consensus.14:43
elmikoagreed14:44
elmiko;)14:44
egaffordelmiko: Yeah, that's an interesting option. It doesn't quite build toward the API the same way, but I do like it.14:44
*** tellesnobrega_af is now known as tellesnobrega14:44
elmikoit could still sit at an api endpoint though, so you could automate something whereby the tool would get the config data from a running sahara14:45
egaffordIf the idea is that all of these ways of structuring image modification scripts are, in fact, reasonably isomorphic, there's no reason not to have the plugins write their own formatted action set, and generate a translation layer.14:45
egaffordelmiko: Right, my goal with the API though is to have an API that *actually builds your image*.14:45
egaffordThis is harder.14:46
elmikoright, i was just thinking of other options because the whole image gen api might become a tangled web. especially for end user who might try to debug it14:47
elmikoalthough, i do like the possibility of asking sahara to gen an image, and that image gets auto-uploaded to glance/swift/etc...14:47
egaffordelmiko: Right, that's exactly the objective. My thinking in the current spec though is that rather than writing a translation layer from some other format, if our contention is that isomorphism will win in the end (so we might as well write our own language or use another) that by the same token, isomorphism will win in the end, so we might as well use what we already have and move incrementally.14:48
elmikoegafford: yea, it's a good thought14:50
*** tmckay has quit IRC14:50
egaffordelmiko: Well, I do think that in implementation, it's a cumbersome thought.14:50
elmikothat's kinda why i started thinking about a config file for sie14:50
elmikoit would make it easier to manage image gen. imo14:51
pino|work<elmiko> that's kinda why i started thinking about a config file for sie14:51
elmikoand then those "recipes" could be shared or preserved or whatever..14:51
pino|workeeeeeeeeeeeeek14:51
egaffordpino|work: :)14:51
elmikopino|work: why is that bad?14:52
pino|workbtw https://review.openstack.org/23556914:52
egaffordSIE is already a format for instructions to generate images, and a reasonably sparse one.14:52
elmikocool, someone else is already thinking about this lol14:52
pino|work(which imho is just juggling around a buggy tool, and putting lipstick on it)14:52
egaffordIt's weird, and occasionally slightly opinionated, and it has a lot of coupling with its internals where it shouldn't.14:53
egaffordBut I'm largely with Pino that generating a new isomorphic language to generate SIE elements is dancing around the issue.14:53
elmikoboth good points14:53
pino|workyaml is the new xml: praised like the Best Thing Ever™ today, and largely hated in 10 years14:54
elmikohaha!14:54
elmikoin fairness though, yaml is much easier on the eyes than xml14:55
egaffordI think we pull SIE into Sahara and use the elements as the basis for clean, idempotent image generation (which will take new work to clean them up so they work that way, and an engine capable of running them on the post-provisioning side,) or we let SIE be SIE and just build something unrelated on the server-side as an image gen/image validation/clean image provisioning package, and expose it when it's ready.14:55
toskypino|work: yaml AND json14:55
egaffordelmiko: ^14:56
openstackgerritEvgeny Sikachev proposed openstack/sahara: Fix skiped tempest test  https://review.openstack.org/24928114:56
pino|worktosky: json is a dialect of yaml14:56
pino|work:p14:56
*** sgotliv has quit IRC14:56
egaffordpino|work: You're a dialect of yaml.14:56
elmikoegafford: ack, it makes sense. i was just brainstorming other options14:56
egaffordelmiko: Totally.14:56
pino|workegafford: i know you know about what i think, but i'll just mention that "elements" and "clean" cannot go in the same sentence14:56
egaffordpino|work: So, my thinking on that: there's not THAT much wrong with the element format, itself.14:57
egaffordIt has phases, it installs packages, it runs scripts.14:57
pino|workegafford: sir, i beg to differ14:57
egaffordIt does a lot of things that an image gen system ought to do.14:57
elmikoegafford: also, i just have a nagging feeling that if we are going to ask plugin authors to generate a "recipe" for their image generation, then it would be nice to have something more neutral to use as a middle point than a script. but that may not be possible.14:57
egaffordpino|work: What do you see as terribly wrong with elements, themselves? Not DIB, and not the mess of envvars that they throw around at each other like crazy people, but just the element format?14:59
egaffordI mean, they're not my favorite either, but they do have some nice composability.14:59
pino|workegafford: "jack of all spades, and master of none"14:59
pino|workyou can do lots of things with shell scripts... but that's exactly also the biggest problem15:00
pino|workyou have no enforcement of any kind on the operations you want to do15:00
egaffordpino|work: Sure, that's valid.15:01
pino|worksome of the elements are even run on your host, and require root (so put a typo there, and eg your host /dev is gone)15:01
*** tellesnobrega is now known as tellesnobrega_af15:01
elmikothat's one nice thing about hiding image gen behind sahara, we could change to whatever tooling we wanted ;)15:01
pino|workalso, your own script could just throw away what a script of a different element (ran earlier) did15:01
pino|workbut you have no idea of what scripts in elements do exactly, so it's all like walking on a minefield15:02
elmikopino|work: all excellent points15:03
elmikothe root thing is definitely scary15:03
egaffordpino|work: Well, what you're describing on the other side is dependency encapsulation, which is nice. Except the root and guest/host stuff, which is crazy.15:03
pino|workegafford: yes, but encapsulation in this case is hiding for no good15:04
pino|workimage generation should give a well-defined list of operations to do, and you should be able to see each step clearly15:04
elmikopino|work: so, the whole list of operations should be composed into a single master list (or something)?15:05
pino|worksure, you might want to think "ok, i don't want to be bothered with partitioning", and that'd be fine from a certain POV15:05
pino|workbut then you hide more and more stuff, so when things break you have no clue where to start from15:05
pino|work(and i guess on the last sentence you both might agree with me)15:05
elmikoi like the vision you are laying out15:06
egaffordpino|work: I'm largely with you here, yeah.15:06
egaffordI think elements are ok for spinning an image from nothing, but I do agree with you (completely) that sequential operations are more legible when we don't need to fundamentally change everything about the structure of the image.15:08
*** sgotliv has joined #openstack-sahara15:09
pino|workespecially when you want reproduceability (is there such word?) of what you produce15:09
pino|workalso, imho separating the image build in two could help:15:10
pino|work- build the base os image (cloud, vm, baremetal, etc)15:11
pino|work- customize the above with the services/tools needed15:11
*** poseidon1157 has joined #openstack-sahara15:11
poseidon1157Hi all15:11
pino|workusually the 1st is built much less often, so it is a bit easier to build, and to validate that is good15:12
poseidon1157I'm having some issues with the neutron floatingip pool15:12
poseidon1157Where is that actually defined?15:12
poseidon1157The UUID changed necessarily15:12
elmikoposeidon1157: what is the issue you are experiencing?15:13
pino|workposeidon1157: #openstack-neutron maybe?15:13
elmikopino|work: i like the way you break down the issue, i'm also really curious about the idea of some sort of master list that could show everything that will happen15:13
poseidon1157Not a neutron issue.  :)15:14
egaffordpino|work: Yup, with you completely. I'm hoping to just use the product of the first flow you describe.15:14
poseidon1157Second, going to pull out the actual api error15:14
poseidon1157reproducing15:14
elmikoposeidon1157: k, thanks15:14
pino|workonce the former image is build, adding services is much easier to produce and validate (and less demanding, in term of needed build resources)15:14
pino|workegafford: it's just one actually, just split in two phases15:15
poseidon1157"2015-11-25 15:15:31.959 21039 ERROR sahara.utils.api [-] Validation Error occurred: error_code=400, error_message=Floating IP pool 8f2efada-c6c7-40d2-8993-34e590d6de18 not found"15:15
poseidon1157So the UUID, where is that encoded?15:15
poseidon1157That network no longer exists15:16
pino|workespecially that, if you think about it, nowadays the base image is something distros often provide by themselves (hence they ought to take care it is working)15:16
elmikoposeidon1157: when you create node groups, you select a neutron management network, you will most likely need to update your cluster template15:16
poseidon1157elmiko That's what I was trying to do.  floatingips=true in the /etc/sahara conf15:16
elmikoposeidon1157: or check the node group templates that are beeing used15:16
poseidon1157They both say don't use floating ips15:17
egaffordpino|work: Ah, okay, you are agreeing with me, just making an academic (but valid) point about phases.15:17
elmikoposeidon1157: if you have floating_ips=true i don't think you can skip giving those templates floating ip pools15:17
poseidon1157elmiko So I have to designate that in all node templates?15:18
elmikoposeidon1157: yes15:18
poseidon1157elmiko Awesome, tyvm15:18
elmikonp, gl =)15:18
poseidon1157It's silly- Horizon renders an option "Do not assign floating IPs"15:19
poseidon1157Despite the fact that this is not an option :)15:19
elmikoposeidon1157: yea, that's probably something we should discuss about the proper way forward. i *think* there is a way to configure sahara to have floating_ips=true but not use them. crobertsrh may know a little more15:20
pino|workegafford: yeah, i think we're mostly overlapping in ideas15:21
crobertsrhI'm not entirely sure that option is useful15:21
*** tellesnobrega_af is now known as tellesnobrega15:21
elmikocrobertsrh: the floating_ips option?15:21
elmikoor the "Do not assign..."15:22
crobertsrhyeah...we should probably find a way to hide it when it will just break things15:22
elmiko+115:22
poseidon1157Hmm.  I just edited all the node templates.  It's still looking for the old uuid15:22
poseidon1157:/15:22
elmikoposeidon1157: you probably need to recreate the cluster template after updating the node groups, i've run into that before15:23
egaffordpino|work: Okay, writing a competing spec to my own spec.15:23
elmikoegafford: ballsy ;)15:23
pino|worko_O15:23
egaffordI want to make both first steps clear and let people decide.15:24
pino|workand there the one going academic was me :D15:24
elmikohaha15:24
egaffordHa!15:24
poseidon1157elmiko That was it!15:25
elmiko\o/15:25
*** vgridnev has quit IRC15:30
crobertsrhI plan to fix the need to regenerate a cluster template for this release :)15:31
*** chlong has quit IRC15:31
elmikocrobertsrh: awesome, i was thinking about that the other day when i ran into this15:31
crobertsrhI dream of the fix every time I get hosed by it.15:31
elmikolol, i'll bet15:32
openstackgerritEvgeny Sikachev proposed openstack/sahara: Fix skiped tempest test  https://review.openstack.org/24928115:32
*** esikachev has quit IRC15:33
openstackgerritlu huichun proposed openstack/sahara: No need to check if edp_engine is None when run job  https://review.openstack.org/24906815:37
openstackgerritEvgeny Sikachev proposed openstack/sahara: Fix skiped tempest test  https://review.openstack.org/24928115:37
*** tmckay has joined #openstack-sahara15:48
*** houming has quit IRC15:52
poseidon1157Hmm15:54
poseidon1157New issue deploying cloudera15:54
*** houming has joined #openstack-sahara15:55
poseidon1157http://pastebin.com/qanK5nTm15:55
poseidon1157SSL issue15:56
elmikoposeidon1157: interesting, where did you get the cdh image from?15:56
poseidon1157apps.openstack.org I believe15:58
elmikohmm15:59
elmikocould be an error with the images we've generated15:59
pino|workhttp://i.imgur.com/iWKad22.jpg15:59
* elmiko chuckles16:00
poseidon1157Just had a failure with the vanilla image as well.16:00
elmikoposeidon1157: same error?16:00
poseidon1157http://pastebin.com/J5fGXkjW16:00
poseidon1157Different error16:00
crobertsrhHmm, different, but somehow suspiciously similar....missing expected files16:01
poseidon1157Directory in the latter case16:01
elmikoyea16:02
elmikothe cdh error makes me wonder if the online image is not fresh enough, that ssl cert stuff was only added in august of this year16:02
crobertsrhfine idea16:03
elmikoin the latter case, i haven't seen it fail to move the workflow file16:03
elmikoposeidon1157: is that second error reproduceable?16:03
poseidon1157elmiko Let me check16:06
*** jinxing has quit IRC16:06
poseidon1157Yes, reproducable16:13
openstackgerritAndrey Pavlov proposed openstack/sahara: Launching 1 instance in grenade instead of 2  https://review.openstack.org/24981016:13
*** tellesnobrega is now known as tellesnobrega_af16:13
openstackgerritJavier Peña proposed openstack/puppet-sahara: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24526516:14
poseidon1157sahara-kilo-vanilla-1-centos-6.6.qcow2 md5 f7309dcfc6276625f6664dc42bd8b71b16:15
elmikohmm16:15
elmikoposeidon1157: and you are using the vanilla 1.x series plugin?16:16
pino|workisn't that deprecated?16:16
elmikoyea16:16
poseidon11572.616:16
poseidon1157Image marked for 2.6.016:16
elmikook, 2.6 is also deprecated, but not in kilo16:16
elmikoposeidon1157: you probably want to try this image, http://sahara-files.mirantis.com/images/upstream/kilo/sahara-kilo-vanilla-2.6-centos-6.6.qcow216:17
openstackgerritJavier Peña proposed openstack/puppet-sahara: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24526516:19
openstackgerritJavier Peña proposed openstack/puppet-sahara: Support of PyMySQL driver for MySQL backend  https://review.openstack.org/24526516:21
*** AndreyPavlov has quit IRC16:25
*** vgridnev has joined #openstack-sahara16:28
*** houming has quit IRC16:37
openstackgerritMerged openstack/sahara: cleanup sahara commands  https://review.openstack.org/24693916:44
*** ViswaV has joined #openstack-sahara16:48
poseidon1157Reproducible under new image as well16:49
poseidon1157http://pastebin.com/3743MH0c16:49
*** tosky has quit IRC16:51
*** ViswaV has quit IRC16:53
*** ViswaV has joined #openstack-sahara16:53
*** hdd has joined #openstack-sahara16:54
poseidon1157These images are missing keytool?17:02
poseidon1157Hmm- well looks like CentOS has been neglected :)17:06
poseidon1157I'll just use the ubuntu images17:06
*** AndreyPavlov has joined #openstack-sahara17:08
openstackgerritMerged openstack/sahara: Update scenario test readme file  https://review.openstack.org/24884117:09
*** houming has joined #openstack-sahara17:10
*** tellesnobrega_af is now known as tellesnobrega17:11
openstackgerritMerged openstack/python-saharaclient: Adding ability to provide name or ID of the flavor in CLI  https://review.openstack.org/24569217:20
*** david-lyle has joined #openstack-sahara17:30
*** nkrinner has quit IRC17:30
*** raildo is now known as raildo-afk17:38
*** degorenko is now known as _degorenko|afk17:39
*** raildo-afk is now known as raildo17:39
*** david-lyle has quit IRC18:00
*** david-lyle has joined #openstack-sahara18:20
openstackgerritlu huichun proposed openstack/sahara: Check cluster if it is None before run job  https://review.openstack.org/24906818:22
*** hdd has quit IRC18:22
*** hdd has joined #openstack-sahara18:26
*** vgridnev has quit IRC18:29
*** AndreyPavlov has quit IRC18:40
*** houming has quit IRC18:46
*** sgotliv has quit IRC18:49
*** vgridnev has joined #openstack-sahara18:56
*** stanchan has joined #openstack-sahara19:05
*** barra204 is now known as shakamunyi19:12
*** tellesnobrega is now known as tellesnobrega_af19:44
*** crobertsrh has quit IRC20:00
*** crobertsrh has joined #openstack-sahara20:09
elmikoegafford: would you be opposed to adding a cli target to sahara with the sie changes that would still allow for offline creation of images?20:30
egaffordelmiko: The current spec adds the sahara-image-elements CLI to Sahara directly. Are you imagining a separate CLI wrapper?20:32
elmikoegafford: maybe i'm just misunderstanding, would i still be able to invoke diskimage-create or similar?20:34
egaffordWith the spec currently up in sahara-specs, you'd actually be able to invoke the sahara-image-elements scripts exactly as they are now.20:35
elmikook, cool. just my misunderstanding20:35
elmikothanks20:35
egaffordThe spec I put up just moves plugin-specific element code into the plugins themselves, and thus into the sahara repository.20:35
elmikoyea20:35
egaffordAfter that move, we can get fancier about building alternative ways to create images (via an API.)20:36
elmikocool, i re-read it, but it does take some thinking to unpack all the ideas20:36
egaffordOr, as you noted, we can NOT do any of that, and just build a parallel means to pack and verify images into a new API (with some kind of yaml-based definition of how to modify them.)20:38
egaffordI'm actually partial to option 2: it's kind of the v2 API approach (build it in the background, push it when it's done, leave the old stuff until then.)20:38
elmikoi think pino|work convinced me that path is not ideal20:38
elmikowait, option 2 from the spec?20:38
egaffordI list it under "alternatives".20:39
elmikook, yea20:39
egaffordThe spec details only one option.20:39
egaffordBut it does list "don't move anything, build something new into Sahara" as an alternative.20:39
elmikoright20:39
egaffordI posted the spec this way to avoid reinventing the wheel; to take where we are and start moving it toward one solution for offline packing, (new) in-service packing, image validation, and clean image provisioning.20:41
elmikoyea, i just need to think about this more. sorry, but i removed my +1, i like the idea but i want to think about the options more.20:42
egaffordBut I can definitely see arguments against this approach (the main argument being that elements aren't the most readable way to describe a process.) Okay, that's fine.20:42
egaffordWhat are you thinking would be better right now?20:42
elmikoare we talking "pie in the sky whatever mike wants" ?20:43
egaffordNo, sadly we're specifically not. :)20:43
elmikodamnit..20:43
elmikowell, i like the idea of moving this stuff into the plugins. that makes sense to me20:43
egaffordI've spent a bit of time thinking about what The Best Thing would be, and then thinking about How To Get There.20:43
egaffordThe second part is the hardest, by a lot.20:44
elmikoi'm stuck on the idea of moving all the elements into the source tree, it smells slightly off to me20:44
egaffordWell, I think we need to move *something* that describes the image gen process into the source tree.20:44
egaffordIt doesn't have to be elements.20:44
elmikosomething that pino|work said earlier caught my attention too, namely the idea of being able to see all the steps for creating any given image. i'm not sure how we utilize that, but i like the concept.20:45
egaffordBut if it's not elements, we're reinventing our own wheel.20:45
elmikoyea, exactly.20:45
egaffordYeah, that's simple enough. And as noted, I'm happy to reinvent that wheel if we as a team think we should. :)20:45
egaffordI just tend to default to incrementalism and toward forcing change.20:45
egaffordHence this spec.20:46
elmikoit's a very sensible approach20:46
elmikoand i don't disagree with it20:46
egaffordThanks for your thoughts on it; it's really much appreciated.20:46
egaffordWell; I don't know if it's the best approach. I'd be SUPER HAPPY to write some kind of linear manifest: install these packages, run this script, win.20:46
egaffordI think that would be a better approach, in the long run.20:47
elmikoi'm just thinking about us agregating the elements into sahara, and then enabling the plugin authors to use the elements to compose an image (like we do now)20:47
egaffordAs a desirable plan, or as something that will drive sane men mad?20:47
elmikolol20:48
egaffordMy current spec does allow for common, shared elements to be outside of plugins themselves.20:48
elmikoyea, i like that20:48
elmikoi know this isn't in the spec, just curious, do you envision the early version of this just calling out to diskimage-create with whatever options the plugin author desires?20:49
egaffordYup.20:49
egaffordProbably in a nova-spawned build host.20:49
egaffordSpawn host, pull base image from glance, run diskimage-create, rock out.20:50
egaffordUpload to glance before or after rocking out, at plugin author's preference.20:50
elmikoi guess my only qualm is that it seems kinda heavy weight to need an entire stack running to gen. images, but i guess if you are planning on using them you will need a stack anyways...20:50
egaffordRight, the problem right now is that we're thinking "this is heavyweight, you don't need all that to pack an image." True! However, a new POC user coming in will have a stack, by definition.20:51
elmikoright20:52
egaffordAlso, once we have an API that Just Builds Images, then it's an API: it's an interface. If we can abstract away the actual, under-the-hood generation process and engine, then we can swap it out much more at-will.20:52
elmikotrue20:52
*** vgridnev has quit IRC20:52
egaffordAnd right now, image gen is tricky, and often fails because people don't have clean envs.20:52
egaffordIt's much easier for us to spawn people a clean env; that's what OpenStack does.20:53
egaffordAs a note, I want to make it absolutely clear that I understand and agree with your smell concerns.20:53
egaffordI had them too until I poked down a lot of other options, and can be convinced that we should just build something new for this case (which might, in time, remove the need for SIE as a repo.)20:54
elmikoagreed, it's reasonable too20:55
elmikothe whole "build your image in the cloud" is a nice thought too20:55
egaffordYeah, that's the big goal and the big UX win.20:55
elmikook, i'll read it again with a mind towards pointing out areas that could use more definition20:56
elmikotalking it through with you definitely helps sell the case, but i feel that i lost something from just reading the spec20:56
openstackgerritEmilien Macchi proposed openstack/puppet-sahara: release: prepare 7.0.0 (liberty)  https://review.openstack.org/25002120:56
egaffordOnce we have a stable API and SPI for image gen, then plugin authors can use whatever image gen tech they want under the hood, too; wrapping an interface around it gives us a lot of ability to move.20:57
elmikotrue, but it will add complexity to debugging image gen20:57
egaffordWell, part of the problem is that the spec here is only step 1, but I want to talk to Sergey L before posting spec 2, and I don't want to create a strict dependency on spec 2.20:57
elmikounderstandable20:57
egaffordWell, hopefully it'll also make it a bit stabler, but I take your point.20:58
egaffordI don't advocate removing the sahara-image-elements scripts anytime soon.20:58
elmikoagreed, i think it will make it more stable20:58
egafford(Or, maybe, at all; having a CLI is handy.20:58
egafford)20:58
elmikoso, here's my crazy thought (of the moment)20:59
egaffordK20:59
elmikoi can see this being a higher order openstack application20:59
elmikofor example,20:59
egaffordHa!20:59
egaffordYeah, me too.20:59
elmikoyou create an app that uses novaclient and saharaclient to spawn a vm, then gain the image gen specifics from sahara, and finally use the vm to gen the image21:00
egaffordSo my problem with that is that if we wait on an image-packing service as a standalone entity to mature, we'll be here all day.21:00
elmikopossibly including swiftclient or glanceclient to manage the image21:00
elmikoright, but it doesn't need to be part of the sahara service21:00
egaffordYeah, I agree that an OpenStack in-cloud image packer service would be awesome.21:00
elmikoit could just be that when i hit some endpoint in sahara, it returns the current sie script specifically setup to gen the image i'd like, then this image-gen app just uploads that script to the vm and builds the image21:01
egaffordThat's true; I think it should be part of *a* service, though; otherwise we can't wrap it up in a pretty OpenStack API and UI.21:01
elmikoi'm not seeing the win in having it all be contained in sahara21:01
egaffordThe win in having it be in Sahara is that we can put a big old "generate image" button in our UI.21:02
elmikowe could do that regardless of having it in sahara21:02
egaffordIf we're willing to depend on some nascent service that doesn't exist yet, then yes.21:02
elmikoand we could still migrate the image gen bits into the plugins21:03
egaffordTo be clear, I would hope that this piece does eventually become stable enough and cleanly separated enough to spin out.21:03
elmikoi'm not convinced that it would need to be some large service project that exists as a server/db/etc...21:03
egaffordI've mentioned wanting an image generation service for a while really. Well, it needs a server, for sure; it'd likely need a DB to store statuses of ongoing builds, at least.21:04
egaffordAgain, this is the incremental approach.21:04
elmikoit could have all that, but not needed21:04
egaffordThere are problems with the incremental approach, for sure.21:04
*** vgridnev has joined #openstack-sahara21:04
elmikoyea, i'm thinking more of an app that would consume openstack instead of providing a new service21:05
egaffordThen we can't wrap it into the OpenStack user flow, and it has to be a specialist tool. That's not necessarily the end of the world, but it's true.21:05
elmikoi'm not 100% on board with that thought, but i get your point21:06
elmikowith all that said though, i'm ok with the general idea you are proposing21:08
egaffordI think we're haggling between our engineering drive to separate concerns and our desire to give our users a clean and clear experience.21:08
elmikoheh, probably ;)21:08
egaffordIn the end, I *dont* see image packing as foreign to Sahara.21:08
elmikoas long as it ends with a big "gen image" button, the impl can change over time21:08
crobertsrhegafford, my fellow pycharmer.  Have you debugged sahara-all via pycharm recently?21:09
egaffordI think we think it's foreign to Sahara because it always has been. Right, exaclty. :)21:09
crobertsrhIn particular since "Use oslo.service for launching sahara" merged?21:09
elmikoyea, given the tight coupling between plugin and image, then yea it shouldn't be foreign to sahara21:10
egaffordsahara-image-elements is a vital part of the application, unless everyone downloads our pre-packed images and SIE is just an internal RE tool.21:10
egaffordYup, that's this initiative's basic argument. :) And again, your points are sane and valid, though I don't think we should go the separate-application route. We have a separate application, it's called SIE; integration or bust.21:11
egafford:)21:11
egaffordcrobertsrh: I don't think I have. I'm assuming everything is explodey?21:11
*** raildo is now known as raildo-afk21:11
crobertsrhI think it may be.21:12
crobertsrhNot as much explodey, but just unresponsivey21:12
elmikoegafford: fair, and i don't want to stand in the way of that. my knee-jerk reaction is based on the growing complexity of sahara versus my internal desire for smaller simpler tools =)21:12
egaffordSure, sure.21:13
elmikoespecially given that openstack is looking more and more like an operating system21:13
egaffordHappily, I think we can keep this stuff off to the side, or at least fully in its own SPI methods.21:13
elmikoin terms of data separation yes, but once we start handling the other openstack services (eg creating/managing images through nova) it will get messy21:14
egaffordSure, but if we don't do it, then n of our users have to figure it out themselves at the point when they're most likely to stop using our service.21:15
elmikono argument there21:16
elmikoi think we both agree that image gen is a ux weak spot at this point21:16
*** sgotliv has joined #openstack-sahara21:16
egaffordAnyway, please do leave any requests for clarification you like.21:16
elmikoi will, thanks for the extended discussion =)21:16
egaffordAnd understand that all of your criticisms of the plan are valid; I still think this probably is the sanest way forward, but I agree it's a little wonky at first glance.21:17
elmikoyou are most likely correct about it being the sanest way forward, my mind tends to wander when we throw the doors wide open ;)21:18
elmikoand yea, let's get buy-in to work on an image gen service for openstack21:19
egaffordOh, mine too.,21:19
elmikohaha21:19
egaffordHey, it's not a bad plan.21:19
elmikodefinitely not21:19
*** poseidon11571 has joined #openstack-sahara21:20
*** poseidon11571 has quit IRC21:20
*** poseidon1157 has quit IRC21:23
crobertsrhHmm, yep.  Looks like I may need some bonus pycharm love in order to work with the oslo.service launching.  In the meantime, if I undo that patch, things seem to be fine for me again.21:28
crobertsrhegafford:  If pycharm debugging does still magically work for you against master, I'll be interested in your debug config.21:35
*** openstackgerrit has quit IRC21:36
*** openstackgerrit has joined #openstack-sahara21:36
*** vgridnev has quit IRC21:51
egaffordcrobertsrh: I fear that at the moment I don't have a meaningful env for master. Much stable and forked work atm.21:52
crobertsrhNo problem.  More of a heads-up than a plea for help.21:53
egaffordcrobertsrh: Is it not responding AT ALL, like the service isn't even giving you output?21:53
egaffordAppreciated then. :)21:53
crobertsrhIt appears to start-up just fine, but doesn't seem to do anything when I hit it with a request21:54
crobertsrheven if I just curl the base service URL, it normally will reject me because I have no auth, but in the post oslo.service world, it just hangs21:54
elmikoweird21:54
crobertsrhsame for any actual requests that try to hit it21:54
crobertsrhIf I run without debugging turned on, things are fine21:55
elmikomaybe pycharm is preventing some thread from starting?21:55
crobertsrhTotally unsure21:56
crobertsrhMy "fix" is to revert the patch locally for now.21:56
elmikoooph, that stinks21:56
crobertsrhI've also pinged #os-oslo in case there is any sort of known issue at work21:56
elmikogood thinking21:56
crobertsrhOr, I suppose it's possible that I'll be the only special one with an issue21:57
crobertsrhI'm hoping for a mystery config in pycharm, but it will have to wait until after the holiday, I suspect.21:58
crobertsrhHave a fun holiday.  I'm outta here21:58
*** crobertsrh is now known as _crobertsrh21:58
*** tmckay has left #openstack-sahara22:34
*** david-lyle has quit IRC22:34
*** stanchan has quit IRC22:59
*** tellesnobrega_af is now known as tellesnobrega23:05
*** david-lyle has joined #openstack-sahara23:07
*** hdd has quit IRC23:14
*** david-lyle has quit IRC23:24
*** sgotliv has quit IRC23:52

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