| *** japplewhite has joined #openstack-sahara | 00:11 | |
| *** witlessb has quit IRC | 00:12 | |
| *** macjack has quit IRC | 00:15 | |
| *** saneax has quit IRC | 01:21 | |
| *** jamielennox is now known as jamielennox|away | 02:20 | |
| *** jamielennox|away is now known as jamielennox | 02:32 | |
| openstackgerrit | Ken Chen proposed openstack/sahara: Enable YARN ResourceManager HA in CDH plugin https://review.openstack.org/213000 | 02:38 |
|---|---|---|
| openstackgerrit | Ken Chen proposed openstack/sahara: Add CDH HDFS HA part in the user doc https://review.openstack.org/212552 | 02:48 |
| *** macjack has joined #openstack-sahara | 03:06 | |
| *** japplewhite has quit IRC | 03:19 | |
| *** japplewhite has joined #openstack-sahara | 03:19 | |
| *** japplewhite has quit IRC | 03:23 | |
| *** japplewhite has joined #openstack-sahara | 03:25 | |
| *** japplewhite has quit IRC | 03:28 | |
| *** openstackgerrit has quit IRC | 03:31 | |
| *** openstackgerrit has joined #openstack-sahara | 03:31 | |
| *** japplewhite has joined #openstack-sahara | 03:42 | |
| *** japplewhite has quit IRC | 04:01 | |
| *** Nikolay_St has quit IRC | 04:03 | |
| *** coolsvap|away is now known as coolsvap | 04:06 | |
| *** Poornima has joined #openstack-sahara | 04:23 | |
| *** Poornima has quit IRC | 04:35 | |
| *** sgotliv has joined #openstack-sahara | 04:44 | |
| *** sgotliv has quit IRC | 04:52 | |
| *** saneax has joined #openstack-sahara | 05:06 | |
| *** Poornima has joined #openstack-sahara | 05:17 | |
| *** hdd has joined #openstack-sahara | 05:20 | |
| *** nkrinner has joined #openstack-sahara | 05:36 | |
| *** jamielennox is now known as jamielennox|away | 05:47 | |
| *** pino|work has joined #openstack-sahara | 05:57 | |
| *** sgotliv has joined #openstack-sahara | 06:01 | |
| *** macjack has quit IRC | 06:02 | |
| *** macjack has joined #openstack-sahara | 06:04 | |
| *** hdd has quit IRC | 06:11 | |
| *** sgotliv has quit IRC | 06:16 | |
| *** saneax has quit IRC | 06:17 | |
| *** saneax has joined #openstack-sahara | 06:26 | |
| *** vgridnev has joined #openstack-sahara | 06:47 | |
| *** macjack has quit IRC | 07:13 | |
| *** macjack has joined #openstack-sahara | 07:14 | |
| *** macjack has quit IRC | 07:14 | |
| *** macjack has joined #openstack-sahara | 07:16 | |
| *** macjack has quit IRC | 07:16 | |
| *** Nikolay_St has joined #openstack-sahara | 07:18 | |
| *** vgridnev has quit IRC | 08:23 | |
| *** witlessb has joined #openstack-sahara | 08:34 | |
| *** tosky has joined #openstack-sahara | 08:46 | |
| *** tosky has quit IRC | 08:46 | |
| *** sanjayu_ has joined #openstack-sahara | 08:47 | |
| *** tosky has joined #openstack-sahara | 08:48 | |
| *** sanjayu has quit IRC | 08:50 | |
| *** tosky has quit IRC | 08:51 | |
| *** sanjayu_ has quit IRC | 08:52 | |
| *** sanjayu has joined #openstack-sahara | 08:52 | |
| *** tosky has joined #openstack-sahara | 08:53 | |
| *** vgridnev has joined #openstack-sahara | 09:12 | |
| *** DWfuturetec has joined #openstack-sahara | 09:21 | |
| *** macjack has joined #openstack-sahara | 09:30 | |
| *** macjack has quit IRC | 09:35 | |
| *** macjack has joined #openstack-sahara | 09:35 | |
| *** degorenko has joined #openstack-sahara | 09:36 | |
| *** Poornima has quit IRC | 09:52 | |
| *** Poornima has joined #openstack-sahara | 09:57 | |
| *** macjack has quit IRC | 10:00 | |
| *** vgridnev has quit IRC | 10:08 | |
| openstackgerrit | Merged openstack/sahara: Remove unneeded starting ntp https://review.openstack.org/215537 | 10:08 |
| *** DWfuturetec has quit IRC | 10:11 | |
| openstackgerrit | Merged openstack/sahara: Updated from global requirements https://review.openstack.org/215959 | 10:20 |
| openstackgerrit | Merged openstack/sahara: Updated from global requirements https://review.openstack.org/215989 | 10:20 |
| *** ekarlso has quit IRC | 10:32 | |
| *** ekarlso has joined #openstack-sahara | 10:32 | |
| *** DWfuturetec has joined #openstack-sahara | 10:35 | |
| *** jamielennox|away has quit IRC | 10:50 | |
| *** sgotliv has joined #openstack-sahara | 11:03 | |
| *** jamielennox|away has joined #openstack-sahara | 11:04 | |
| *** jamielennox|away is now known as jamielennox | 11:04 | |
| *** vgridnev has joined #openstack-sahara | 11:05 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Add open ports for fake plugin https://review.openstack.org/216052 | 11:17 |
| *** sgotliv has quit IRC | 11:21 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Add open ports for fake plugin https://review.openstack.org/216052 | 11:32 |
| *** Poornima has quit IRC | 11:48 | |
| *** Poornima has joined #openstack-sahara | 11:50 | |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Add open ports for fake plugin https://review.openstack.org/216052 | 12:15 |
| *** sgotliv has joined #openstack-sahara | 12:25 | |
| *** DWfuturetec has quit IRC | 12:33 | |
| *** hdd has joined #openstack-sahara | 12:56 | |
| *** Poornima has quit IRC | 13:03 | |
| *** tmckay has joined #openstack-sahara | 13:13 | |
| *** japplewhite has joined #openstack-sahara | 13:25 | |
| *** jamielennox has quit IRC | 13:26 | |
| *** _crobertsrh is now known as crobertsrh | 13:28 | |
| *** links has joined #openstack-sahara | 13:31 | |
| *** egafford has joined #openstack-sahara | 13:37 | |
| *** japplewhite has quit IRC | 13:38 | |
| *** hdd has quit IRC | 13:41 | |
| egafford | Hello all! Please see https://etherpad.openstack.org/p/sahara-liberty-bug-fix-day for bugs. If we possibly can, let's hit high-prios first. | 13:42 |
| *** jamielennox|away has joined #openstack-sahara | 13:43 | |
| egafford | (Not sure how many of us are about yet.) | 13:43 |
| *** jamielennox|away is now known as jamielennox | 13:43 | |
| * elmiko salutes egafford | 13:46 | |
| elmiko | sir, yes sir! | 13:46 |
| egafford | elmiko: Having trouble finding an appropriate smiley to express the mix of sincere amusement and :/ that deserves. | 13:47 |
| *** saneax has quit IRC | 13:47 | |
| elmiko | haha | 13:47 |
| egafford | That absurd happy fistbump emote from a few weeks ago would work probably. | 13:47 |
| egafford | elmiko: This one is interesting: https://bugs.launchpad.net/sahara/+bug/1486653 | 13:49 |
| openstack | Launchpad bug 1486653 in Sahara "Create big cluster failed because token expired" [High,In progress] - Assigned to Sergey Reshetnyak (sreshetniak) | 13:49 |
| egafford | (And very related to work we've both done.) | 13:50 |
| elmiko | yea, it's probably the type of thing where we should attempt to re-acquire the token when we see a failure | 13:50 |
| tmckay | working on the swift/spark integration bug, like I was on Friday | 13:52 |
| egafford | tmckay: Woot. | 13:52 |
| tmckay | current problem is CDH cluster install fails | 13:52 |
| tmckay | tells me it can't install the Oozie sharelib | 13:52 |
| egafford | tmckay: That is a significant problem, indeed. | 13:52 |
| tmckay | any ideas? anyone ever seen this? | 13:52 |
| egafford | Don't think so, no. tmckay: what's the bug # on that? I want to make sure I get it on the sheet early. | 13:53 |
| elmiko | i've only seen that during image creation, not cluster creation | 13:53 |
| tmckay | https://bugs.launchpad.net/sahara/+bug/1486544 | 13:54 |
| openstack | Launchpad bug 1486544 in Sahara "[Spark1.3] Spark jobs doesn't work with Swift" [Critical,Triaged] - Assigned to Trevor McKay (tmckay) | 13:54 |
| tmckay | so the spark standalone stuff is fixed, but yarn scheduler is still broken | 13:54 |
| elmiko | tmckay: so actually i was wrong about the cdh stuff. i was able to get a 2-node cluster working last week with the spark scenario tests | 13:55 |
| vgridnev | tmckay, have you tried to rebuild image? | 13:55 |
| tmckay | just made a new image on Friday | 13:55 |
| crobertsrh | egafford: Oh...thanks for the bug fix day reminder :) | 13:56 |
| tmckay | vgridnev, elmiko, do you know of a known good image somewhere that I can pull down? Just to make sure? | 13:57 |
| egafford | crobertsrh: No problem. | 13:57 |
| elmiko | tmckay: i would share mine, but the internet here is very slow | 13:57 |
| vgridnev | tmckay, do you use cents image? | 13:58 |
| vgridnev | centos* | 13:58 |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Set $JAVA_HOME/bin directory to PATH https://review.openstack.org/216265 | 14:01 |
| tmckay | vgridnev, yes | 14:02 |
| SergeyLukjanov | egafford, thx for organising the bug fix day ;) | 14:02 |
| egafford | SergeyLukjanov: NP. | 14:02 |
| *** jamielennox has quit IRC | 14:08 | |
| *** jamielennox|away has joined #openstack-sahara | 14:10 | |
| *** jamielennox|away is now known as jamielennox | 14:11 | |
| openstackgerrit | Sergey Reshetnyak proposed openstack/sahara-image-elements: Use JRE directory for JRE_HOME for RHEL-based os https://review.openstack.org/216272 | 14:11 |
| *** DWfuturetec has joined #openstack-sahara | 14:13 | |
| elmiko | egafford: fyi, i'm picking up https://bugs.launchpad.net/sahara/+bug/1486493 | 14:21 |
| openstack | Launchpad bug 1486493 in Sahara "Tests return 0 exit code after fail" [High,Triaged] - Assigned to Michael McCune (mimccune) | 14:21 |
| elmiko | i marked the pad | 14:21 |
| egafford | elmiko: I noticed! | 14:22 |
| * egafford gives elmiko the "first non-egafford pad update' badge. | 14:23 | |
| egafford | Highly coveted. | 14:23 |
| elmiko | haha | 14:23 |
| elmiko | nice, just wanted to keep you in the loop ;) | 14:23 |
| egafford | elmiko: Thanks very much; more communication is better. | 14:24 |
| vgridnev | egafford, SergeyLukjanov was first | 14:27 |
| egafford | vgridnev: :0 | 14:27 |
| * elmiko hands his badge back to egafford | 14:27 | |
| vgridnev | egafford, see line 223 | 14:28 |
| * egafford hands the badge to SergeyLukjanov, with all due apologies. | 14:28 | |
| egafford | The committee for distribution of bugfix-day-related badges sincerely regrets this error, and reaffirms its commitment to transparency and justice. | 14:29 |
| elmiko | lol | 14:29 |
| SergeyLukjanov | :) | 14:30 |
| *** macjack has joined #openstack-sahara | 14:35 | |
| egafford | SergeyLukjanov: Where does sreshetnyak work (in terms of TZ)? | 14:35 |
| openstackgerrit | Merged openstack/sahara-image-elements: Update CentOS-6.6 image version https://review.openstack.org/215686 | 14:35 |
| egafford | vgridnev: ^ You will probably also know this. | 14:38 |
| vgridnev | egafford, right now all mirantis-sahara folks are in saratov | 14:39 |
| egafford | sreshetnyak has (reasonably!) signed up for a bunch of reasonably high-prio bugs before bugfix week; I want to get an idea of what he thinks he'll get done and which bugs he wouldn't mind someone else grabbing. | 14:39 |
| egafford | vgridnev: Ooh, ouch. | 14:39 |
| egafford | Thought you were largely after us (in SF) rather than before (in Saratov). My apologies. | 14:40 |
| sreshetnyak | egafford: I'm working on #1488101 and #1486653 | 14:43 |
| sreshetnyak | and #1478059 on review | 14:44 |
| openstackgerrit | Merged openstack/sahara-image-elements: Fix centos7 building image https://review.openstack.org/209951 | 14:44 |
| openstackgerrit | Merged openstack/sahara-image-elements: Added support of MapR 5.0.0 to sahara-image-elements https://review.openstack.org/207865 | 14:45 |
| egafford | sreshetnyak: What about https://bugs.launchpad.net/sahara/+bug/1478059 and https://bugs.launchpad.net/sahara/+bug/1475263? | 14:46 |
| openstack | Launchpad bug 1478059 in Sahara "Sahara uses publicURL by default" [High,In progress] - Assigned to Sergey Reshetnyak (sreshetniak) | 14:46 |
| openstack | Launchpad bug 1475263 in Sahara kilo "Borders in requirements.txt not sorted as in global" [High,In progress] - Assigned to Sergey Reshetnyak (sreshetniak) | 14:46 |
| openstackgerrit | Merged openstack/sahara-image-elements: Remove libevent installation from MapR plugin https://review.openstack.org/214526 | 14:46 |
| *** bapalm- has joined #openstack-sahara | 14:46 | |
| sreshetnyak | ttps://bugs.launchpad.net/sahara/+bug/1478059 fix on review https://review.openstack.org/#/c/206017/ | 14:47 |
| egafford | sreshetnyak: Good then; in terms of 1475263, do you want to keep that in addition to the two you're currently working on, or release it back into the pool? | 14:50 |
| sreshetnyak | I think need to move this bug to invalid or incomplete because it's a problem in ci-build-package system. | 14:57 |
| egafford | sreshetnyak: Reasonable. | 15:02 |
| *** Erming_ has quit IRC | 15:03 | |
| *** nkrinner has quit IRC | 15:14 | |
| *** Nikolay_St has quit IRC | 15:35 | |
| *** tellesnobrega has quit IRC | 15:37 | |
| *** IBerezovskiy has joined #openstack-sahara | 15:37 | |
| *** egafford has quit IRC | 15:40 | |
| *** egafford has joined #openstack-sahara | 15:41 | |
| *** tellesnobrega has joined #openstack-sahara | 15:43 | |
| *** vgridnev has quit IRC | 16:02 | |
| SergeyLukjanov | elmiko, I've just release python-barbicanclient 3.3.0 | 16:04 |
| SergeyLukjanov | AFAIR you've been waiting for it :) | 16:04 |
| *** links has quit IRC | 16:08 | |
| *** hdd has joined #openstack-sahara | 16:18 | |
| *** hdd has quit IRC | 16:27 | |
| *** hdd has joined #openstack-sahara | 16:37 | |
| *** IBerezovskiy has quit IRC | 16:39 | |
| elmiko | SergeyLukjanov: yea, that was the precursor to castellan release, thanks! | 16:42 |
| *** Nikolay_St has joined #openstack-sahara | 16:44 | |
| pino|work | elmiko: of course, first a barbican, then a castellan around it, makes sense | 16:47 |
| elmiko | pino|work: lol, yes ;) | 16:47 |
| *** sgotliv has quit IRC | 16:47 | |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Add workaround for POST image updating https://review.openstack.org/216336 | 16:50 |
| *** vgridnev has joined #openstack-sahara | 16:50 | |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Add workaround for POST image updating https://review.openstack.org/216336 | 16:51 |
| openstackgerrit | Merged openstack/sahara: Add support of SSL in scenario tests https://review.openstack.org/212476 | 16:51 |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Add workaround for POST image updating https://review.openstack.org/216336 | 16:52 |
| openstackgerrit | Denis Egorenko proposed stackforge/sahara-ci-config: Add workaround for POST image updating https://review.openstack.org/216336 | 16:56 |
| SergeyLukjanov | elmiko, pino|work, ok cool | 16:56 |
| pino|work | SergeyLukjanov: mine was just a joke... :) | 16:56 |
| elmiko | SergeyLukjanov: for me, it means i will probably start pushing the improved secret storage reviews soon™ | 16:57 |
| *** melaks has joined #openstack-sahara | 17:19 | |
| egafford | Picking up https://bugs.launchpad.net/sahara/+bug/1436425. | 17:19 |
| openstack | Launchpad bug 1436425 in Sahara "[CDH] Too many connection" [High,Confirmed] - Assigned to Ethan Gafford (egafford) | 17:19 |
| elmiko | good luck egafford, that one looked interesting... | 17:19 |
| egafford | elmiko: Yeah, looks like a probable SQL misconfig, but it might be deeper. | 17:20 |
| elmiko | that would be cool if it's just sql config | 17:20 |
| egafford | Yeah, either that or we're doing something really quite awful with connection pooling. Or CDH is. | 17:21 |
| egafford | elmiko: How are our overconfident tests coming along? | 17:22 |
| egafford | Manage to repro? | 17:22 |
| elmiko | yes, i can repro. i'm digging through the testr and script stuff now | 17:22 |
| elmiko | just trying to figure out what return code is not being picked up | 17:23 |
| egafford | Cool. Glad that got picked up early; very worrisome really. | 17:23 |
| elmiko | i actually noticed this last week when i was running the scenario tests, mainly why i picked this one up | 17:23 |
| openstackgerrit | Merged stackforge/sahara-ci-config: Add workaround for POST image updating https://review.openstack.org/216336 | 17:30 |
| *** vgridnev has quit IRC | 18:10 | |
| *** bapalm- is now known as bapalm_ | 18:19 | |
| *** vgridnev has joined #openstack-sahara | 18:22 | |
| elmiko | egafford: so, this is weird with #1486493 | 18:24 |
| elmiko | i created a special fail test, that does an assertEqual(True, False) | 18:24 |
| elmiko | if i run that through the scenario tests, then i get a fail on the command line | 18:24 |
| elmiko | but if i run an actual scenario test, that fails, i get success on the cli | 18:25 |
| * elmiko scratches head | 18:25 | |
| egafford | elmiko: Yeah, I'm absolutely seeing it too (cluster bombs out with resource error, scenario tests give me a thumbs-up.) | 18:27 |
| elmiko | yea, same | 18:27 |
| elmiko | i'm digging in a little more here, i do see a difference in the pretty_tox.sh return | 18:28 |
| elmiko | this is so weird | 18:28 |
| egafford | Wonder if we're catching a specific exception type, logging it, and not reraising? | 18:28 |
| elmiko | i think that might be possible | 18:28 |
| egafford | Your assertion exception got through, but. | 18:28 |
| elmiko | well, if i do the assertEqual, then yea, i get a true failure from testr | 18:29 |
| elmiko | and even if i raise and exception i'm seeing a true fail | 18:29 |
| elmiko | but, the scenario test i'm running, for spark, raises an assertion, and it says i have 1 failed test, but tests says "yay you win!" | 18:30 |
| elmiko | oh well, back to the drawing board | 18:30 |
| *** macjack has quit IRC | 18:42 | |
| openstackgerrit | Merged openstack/sahara: Remove libevent installation from MapR plugin https://review.openstack.org/214520 | 18:49 |
| egafford | elmiko: In scenario.base.BaseTestCase.tearDown, I added "self.fail("Scenario tests failed")" before the check for whether any(c['status'] == CHECK_FAILED_STATUS), and the tests passed. I also tried raising an exception there. | 18:50 |
| egafford | elmiko: I'm uncertain that the tearDown method is being called. | 18:50 |
| elmiko | egafford: good call, i can confirm that a self.fail will produce an error on the cli | 18:50 |
| elmiko | i'm starting to be curious about the fixtures that we use in the _poll_job_status | 18:51 |
| elmiko | i wonder if it's doing something, as my jobs are the ones that are failing | 18:51 |
| egafford | elmiko: So you're seeing self.fail fail the tests? Curious. | 18:51 |
| egafford | Where did you inject self.fail to test? | 18:52 |
| elmiko | egafford: yea, i have a scenario harness setup, and i just call self.fail('oops') and i see it happen | 18:52 |
| elmiko | currently, i'm tracing through the scenario base class to see if i can figure out why it's eating the self.fail calls, because i can see those happen in the test that doesn't report a failure | 18:53 |
| openstackgerrit | Merged openstack/sahara: Actually install Heat for the Hest-based jobs https://review.openstack.org/215952 | 19:01 |
| openstackgerrit | Merged openstack/sahara: Fail if FAILED in the scenario tests run log https://review.openstack.org/215962 | 19:01 |
| elmiko | egafford: wonder if we need to do that ^ | 19:03 |
| elmiko | SergeyLukjanov: on https://review.openstack.org/215962 did you look into why the scenario tests don't report properly to the shell? | 19:03 |
| elmiko | egafford: ok, i can fully reproduce this with my custom code now. i am calling self.fail and i get success on the cli | 19:05 |
| openstackgerrit | Merged openstack/sahara: Implement ability of creating flavor for scenario tests https://review.openstack.org/203567 | 19:05 |
| openstackgerrit | Merged openstack/sahara: Update plugin version for transient tests to vanilla 2.7.1 https://review.openstack.org/213622 | 19:05 |
| openstackgerrit | Merged openstack/sahara: Fix HBase config name when using HA with HDP 2.0.6 https://review.openstack.org/212865 | 19:07 |
| openstackgerrit | Merged openstack/sahara: Removed confusing typos in utils/openstack/base.py file https://review.openstack.org/212460 | 19:07 |
| openstackgerrit | Merged openstack/sahara: Add validation rules about IMPALAD https://review.openstack.org/215534 | 19:07 |
| vgridnev | elmiko, egafford tmckay sreshetnyak crobertsrh I need your help in reviewing 1 line long bug fix for horizon | 19:10 |
| vgridnev | https://review.openstack.org/#/c/216384/1/openstack_dashboard/contrib/sahara/content/data_processing/jobs/workflows/create.py,cm | 19:11 |
| vgridnev | https://review.openstack.org/#/c/216384/1 | 19:11 |
| egafford | elmiko: I think we're relying on tearDown to report failure status, though; if you look at the track_result decorator, we're creating a custom test_info dict for each test, and failing only if any of those dicts' statuses == CHECK_FAILED_STATUS. We reraise for each individual test, but the test driver may eat that exception to keep running further tests. I'm not actually seeing the table tearDown prints in the positive or negative | 19:11 |
| crobertsrh | will do | 19:11 |
| vgridnev | thanks | 19:11 |
| elmiko | egafford: i don't think the issue is with the test class | 19:12 |
| elmiko | i think it has more to do with the way os.system reports the exit status on unix systems | 19:12 |
| egafford | elmiko: Cool. | 19:12 |
| elmiko | egafford: and i have a very simple test that shows this | 19:13 |
| egafford | vgridnev: I'm thinking we're going to need a lot of patch sets on this one. :) | 19:13 |
| egafford | elmiko: Nice! If you've confirmed a cause, I'll stop poking your bug's bear. | 19:13 |
| vgridnev | egafford, around 50 | 19:14 |
| egafford | vgridnev: Yeah. Honestly, I don't see that we're going to be able to merge this without approval from at least a simple majority of the TC. | 19:14 |
| elmiko | egafford: here is the essence of our issue, http://paste.openstack.org/show/426307/ | 19:15 |
| egafford | vgridnev: (I will wait for CI, but obviously this looks fine. :) ) | 19:15 |
| vgridnev | CI? you mean jenkins? | 19:16 |
| egafford | vgridnev: Yes, I suppose that is what I mean, looking at my own Horizon patch for reference. | 19:17 |
| openstackgerrit | Vitaly Gridnev proposed openstack/sahara: Use internalURL endpoint by default in Sahara https://review.openstack.org/206017 | 19:21 |
| openstackgerrit | Michael McCune proposed openstack/sahara: Changing scenario runner to use subprocess https://review.openstack.org/216396 | 19:33 |
| egafford | elmiko: Nice! | 19:34 |
| elmiko | egafford: for all that it was a 2 line change lol | 19:34 |
| elmiko | but, i learned something, so that's cool | 19:35 |
| egafford | Yeah, awesome is not measured in LOC. | 19:35 |
| elmiko | +1 | 19:36 |
| elmiko | egafford: i feel like that quote would make a good motivational poster for an engineering office | 19:42 |
| egafford | elmiko: Huh; os.system reports 512 vs subprocess' 2 on one error case. Is there a modulo going on in sys.exit? | 19:42 |
| elmiko | egafford: it has to do with how os.system returns the exit code | 19:43 |
| egafford | (They both return a plain old int, type-wise.) | 19:43 |
| elmiko | os.system returns a 16-bit number, high byte is the exit code, low byte is the signum that killed | 19:43 |
| elmiko | so, 512 would make sense for a 2 | 19:43 |
| elmiko | i kept getting 256 for a 1 | 19:44 |
| egafford | Right, absolutely. There seemed to be a byteshift going on. | 19:44 |
| elmiko | and since the sys.exit only takes an 8bit, then boom | 19:44 |
| egafford | ...Ah. Rockin'. | 19:44 |
| elmiko | but, i guess it only works that way on unixen, so for windows it might work fine | 19:44 |
| egafford | It just truncates it down to 0 byte, huh. | 19:44 |
| elmiko | right | 19:44 |
| egafford | Wow, okay then. | 19:45 |
| elmiko | yea, on unix exit status can only be a byte | 19:45 |
| egafford | From msdn: System Error Codes (0-499) | 19:45 |
| elmiko | interesting... | 19:46 |
| egafford | https://msdn.microsoft.com/en-us/library/windows/desktop/ms681382%28v=vs.85%29.aspx | 19:46 |
| elmiko | but i have a feeling os.system works different on windows anyways | 19:46 |
| egafford | I guess single bytes just cramped MS' style. Quite possible, yeah. | 19:46 |
| elmiko | lol | 19:46 |
| vgridnev | tellesnobrega, let's don't nitpick on commit message on big change: https://review.openstack.org/#/c/195568/9 | 20:03 |
| tellesnobrega | sure | 20:03 |
| tellesnobrega | vgridnev, sorry for that | 20:04 |
| *** sgotliv has joined #openstack-sahara | 20:36 | |
| *** Nikolay_St has quit IRC | 20:40 | |
| *** DWfuturetec has quit IRC | 20:59 | |
| *** DWfuturetec has joined #openstack-sahara | 21:17 | |
| *** hdd has quit IRC | 21:22 | |
| *** tmckay has left #openstack-sahara | 21:24 | |
| *** crobertsrh is now known as _crobertsrh | 21:26 | |
| openstackgerrit | Evgeny Sikachev proposed openstack/sahara: Add API methods for call verifications, DB table https://review.openstack.org/196713 | 21:35 |
| *** sgotliv has quit IRC | 21:42 | |
| *** egafford is now known as _egafford | 21:48 | |
| *** pino|work has quit IRC | 21:56 | |
| *** vgridnev has quit IRC | 21:59 | |
| *** _egafford has quit IRC | 22:07 | |
| *** tosky has quit IRC | 22:49 | |
| *** vgridnev has joined #openstack-sahara | 22:50 | |
| *** tosky has joined #openstack-sahara | 22:50 | |
| *** tosky has quit IRC | 22:51 | |
| *** vgridnev has quit IRC | 22:52 | |
| *** tosky has joined #openstack-sahara | 22:53 | |
| *** hdd has joined #openstack-sahara | 22:53 | |
| *** tosky has quit IRC | 22:55 | |
| *** tosky has joined #openstack-sahara | 22:57 | |
| *** tosky has quit IRC | 22:58 | |
| *** tosky has joined #openstack-sahara | 22:59 | |
| *** tosky has quit IRC | 23:01 | |
| *** DuncanT has quit IRC | 23:38 | |
| *** DuncanT has joined #openstack-sahara | 23:38 | |
| *** DWfuturetec has quit IRC | 23:42 | |
| *** witlessb has quit IRC | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!