Tuesday, 2018-04-10

*** logan- has quit IRC00:02
*** logan- has joined #openstack-ironic00:02
*** pmannidi has joined #openstack-ironic00:10
*** bfournie has joined #openstack-ironic00:11
*** gyee has quit IRC00:11
*** mjturek has quit IRC00:19
*** exodusftw has joined #openstack-ironic00:29
*** cjloader has joined #openstack-ironic00:32
*** cjloader has quit IRC00:33
*** cjloader has joined #openstack-ironic00:34
*** hshiina has joined #openstack-ironic00:44
*** rloo has quit IRC00:52
*** tiendc has joined #openstack-ironic01:04
*** fanzhang has joined #openstack-ironic01:04
*** zhangfei has joined #openstack-ironic01:10
*** jcoufal has joined #openstack-ironic01:31
*** namnh has joined #openstack-ironic02:08
*** r-daneel has joined #openstack-ironic02:36
*** rbudden has quit IRC02:52
*** jcoufal has quit IRC03:07
*** cjloader has quit IRC03:16
*** cjloader_ has joined #openstack-ironic03:16
*** harlowja has quit IRC03:16
*** ianychoi has quit IRC03:16
*** cjloader_ has quit IRC03:19
*** cjloader has joined #openstack-ironic03:20
*** ianychoi has joined #openstack-ironic03:20
openstackgerritKaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state  https://review.openstack.org/55561003:24
openstackgerritshuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml  https://review.openstack.org/51788303:28
*** phuongnh has joined #openstack-ironic03:36
*** harlowja has joined #openstack-ironic03:57
*** pcaruana has joined #openstack-ironic04:06
*** vabada2 has joined #openstack-ironic04:10
*** liuzz has quit IRC04:11
*** strigazi has quit IRC04:11
*** vabada has quit IRC04:13
*** strigazi has joined #openstack-ironic04:14
*** ianychoi has quit IRC04:14
*** ianychoi has joined #openstack-ironic04:16
*** pcaruana has quit IRC04:16
openstackgerritshuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml  https://review.openstack.org/51788304:17
*** liuzz has joined #openstack-ironic04:25
*** cjloader has quit IRC04:54
*** marios has joined #openstack-ironic04:58
openstackgerritshuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml  https://review.openstack.org/51788305:06
*** links has joined #openstack-ironic05:11
*** strigazi_ has joined #openstack-ironic05:13
*** vabada has joined #openstack-ironic05:14
*** vabada2 has quit IRC05:17
*** strigazi has quit IRC05:17
*** fragatina has quit IRC05:31
*** etingof has quit IRC05:42
*** ianychoi has quit IRC05:47
openstackgerritshuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml  https://review.openstack.org/51788305:48
*** ianychoi has joined #openstack-ironic05:49
*** fragatina has joined #openstack-ironic05:57
*** fragatin_ has joined #openstack-ironic05:58
*** fragatina has quit IRC06:01
*** fragatin_ has quit IRC06:02
*** racedo has joined #openstack-ironic06:13
*** cjloader has joined #openstack-ironic06:16
*** namnh has quit IRC06:18
*** cjloader has quit IRC06:19
*** e0ne has joined #openstack-ironic06:29
*** AlexeyAbashkin has joined #openstack-ironic06:32
*** pcaruana has joined #openstack-ironic06:40
*** harlowja has quit IRC06:40
*** rcernin has quit IRC06:41
*** fragatina has joined #openstack-ironic06:47
*** arnewiebalck has joined #openstack-ironic06:47
*** AlexeyAbashkin has quit IRC06:49
*** AlexeyAbashkin has joined #openstack-ironic06:50
*** AlexeyAbashkin has quit IRC06:54
*** fragatina has quit IRC06:56
*** jtomasek has joined #openstack-ironic07:02
*** AlexeyAbashkin has joined #openstack-ironic07:02
*** milan has joined #openstack-ironic07:04
openstackgerrityolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API  https://review.openstack.org/51257907:14
*** tesseract has joined #openstack-ironic07:19
*** livelace has joined #openstack-ironic07:22
*** fragatina has joined #openstack-ironic07:30
*** jhesketh_ has joined #openstack-ironic07:31
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116207:33
*** jhesketh has quit IRC07:37
*** mgoddard has joined #openstack-ironic07:37
*** hoonetorg has quit IRC07:38
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB API  https://review.openstack.org/51140207:41
*** hoonetorg has joined #openstack-ironic07:52
*** AlexeyAbashkin has quit IRC07:53
*** AlexeyAbashkin has joined #openstack-ironic07:56
*** mgoddard has quit IRC07:56
*** milan has quit IRC08:00
*** ktibi has joined #openstack-ironic08:01
*** lucas-afk is now known as lucasagomes08:05
*** pmannidi has quit IRC08:17
*** dougsz has joined #openstack-ironic08:19
*** jesusaur has quit IRC08:23
*** jesusaur has joined #openstack-ironic08:27
*** priteau has joined #openstack-ironic08:27
*** mgoddard has joined #openstack-ironic08:30
*** milan has joined #openstack-ironic08:37
*** etingof has joined #openstack-ironic08:41
*** rbartal has joined #openstack-ironic08:44
openstackgerritjiapei proposed openstack/ironic master: Add documentatin for XClarity Driver  https://review.openstack.org/55996008:45
*** tiendc has quit IRC08:48
*** tiendc has joined #openstack-ironic08:49
*** etingof has quit IRC08:51
*** milan_ has joined #openstack-ironic08:51
*** milan has quit IRC08:51
*** hshiina has quit IRC09:02
*** pcaruana has quit IRC09:04
vdrokgood morning ironic!09:10
*** milan_ has quit IRC09:11
*** etingof has joined #openstack-ironic09:17
*** links has quit IRC09:18
*** derekh has joined #openstack-ironic09:30
*** links has joined #openstack-ironic09:33
*** stendulker has joined #openstack-ironic09:46
*** nizam037 has joined #openstack-ironic09:51
*** strigazi_ is now known as strigazi10:01
*** jesusaur has quit IRC10:02
*** jesusaur has joined #openstack-ironic10:15
*** strigazi is now known as strigaz_10:59
*** strigaz_ is now known as strigazi_10:59
*** strigazi_ is now known as strigazi10:59
jrollmorning vdrok and anyone else :)11:00
*** sambetts|afk is now known as sambetts11:04
sambettso/11:04
*** stendulker_ has joined #openstack-ironic11:08
*** stendulker has quit IRC11:12
*** bfournie has quit IRC11:14
*** stendulker_ has quit IRC11:14
*** lucasagomes is now known as lucas-hungry11:17
*** zhangfei has quit IRC11:19
*** dprince has joined #openstack-ironic11:20
*** links has quit IRC11:26
*** dprince has quit IRC11:28
*** ajya has joined #openstack-ironic11:30
*** dprince has joined #openstack-ironic11:31
*** phuongnh has quit IRC11:35
*** milan_ has joined #openstack-ironic11:36
*** tiendc has quit IRC11:39
*** links has joined #openstack-ironic11:40
*** bfournie has joined #openstack-ironic12:02
*** weshay_pto is now known as weshay12:03
*** openstackgerrit has quit IRC12:04
*** lucas-hungry is now known as lucasagomes12:11
vdrokgood morning jroll , sambetts12:11
*** hjensas has quit IRC12:17
*** hjensas has joined #openstack-ironic12:20
*** hjensas has quit IRC12:20
*** hjensas has joined #openstack-ironic12:20
*** trown|outtypewww is now known as trown12:20
*** openstackgerrit has joined #openstack-ironic12:21
openstackgerritVladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode  https://review.openstack.org/35906112:21
openstackgerritVladyslav Drok proposed openstack/python-ironicclient master: WIP allow to use none auth in functional tests  https://review.openstack.org/50833012:21
*** jcoufal has joined #openstack-ironic12:29
*** rloo has joined #openstack-ironic12:30
*** rbudden has joined #openstack-ironic12:35
*** pcaruana has joined #openstack-ironic12:44
openstackgerrityolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model  https://review.openstack.org/51116212:51
*** logan- has quit IRC12:55
*** logan- has joined #openstack-ironic12:56
*** zhurong has joined #openstack-ironic12:59
openstackgerritIlya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt  https://review.openstack.org/56001913:00
*** mjturek has joined #openstack-ironic13:01
*** zhurong has quit IRC13:13
*** jjohnson2 has joined #openstack-ironic13:18
TheJuliagood morning everyone13:22
rloogood morning TheJulia and everyone else!13:26
*** Goneri has joined #openstack-ironic13:32
*** baha has joined #openstack-ironic13:38
*** lbragstad has joined #openstack-ironic13:42
openstackgerritIlya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt  https://review.openstack.org/56001913:47
*** cdearborn has joined #openstack-ironic13:50
*** serverascode has quit IRC13:50
*** serverascode has joined #openstack-ironic13:50
openstackgerrityolanda.robla proposed openstack/ironic master: Add bios_interface db field  https://review.openstack.org/52860913:52
*** fdegir has quit IRC13:54
*** gregwork has quit IRC13:54
*** fdegir has joined #openstack-ironic13:54
*** gregwork has joined #openstack-ironic13:54
*** betherly has quit IRC13:56
*** betherly has joined #openstack-ironic13:57
*** cdearborn has quit IRC13:58
*** r-daneel has quit IRC14:02
*** ianychoi has quit IRC14:02
*** ianychoi has joined #openstack-ironic14:03
*** ajya has quit IRC14:03
*** ajya has joined #openstack-ironic14:03
*** cdearborn has joined #openstack-ironic14:08
*** timirnich has quit IRC14:08
*** ildikov has quit IRC14:08
*** timirnich has joined #openstack-ironic14:09
*** ildikov has joined #openstack-ironic14:09
*** patrickeast has quit IRC14:11
*** patrickeast has joined #openstack-ironic14:11
*** davidlenwell has quit IRC14:13
*** davidlenwell has joined #openstack-ironic14:13
*** links has quit IRC14:13
*** portdirect has quit IRC14:19
*** portdirect has joined #openstack-ironic14:19
*** csatari has quit IRC14:19
*** csatari has joined #openstack-ironic14:19
*** samueldmq has quit IRC14:21
*** sergek_ has quit IRC14:21
*** samueldmq has joined #openstack-ironic14:21
*** sergek_ has joined #openstack-ironic14:22
*** zul has quit IRC14:22
*** r-daneel has joined #openstack-ironic14:22
*** zul has joined #openstack-ironic14:22
*** ajya has quit IRC14:23
openstackgerritRuby Loo proposed openstack/ironic master: add warning log about empty pxe_enabled_ports  https://review.openstack.org/53132014:29
olivierb-Hello everyone, sorry to ask without having had time to look deeper into the issue but I am quite in a hurry today. Without any notable change from my side, bifrost deployement fails today (was working yesterday)14:31
olivierb-TASK [bifrost-ironic-install : Ensure service project is present]14:31
olivierb-Internal Server Error (HTTP 500)\n14:31
olivierb-however everything seems to be ok14:31
olivierb-retried deploying but wtihout more success14:32
olivierb-rings any bell ???14:32
*** r-daneel has quit IRC14:32
olivierb-hidden dependency broken ?14:32
olivierb-thanks for your understanding14:32
sambettsolivierb-: that error relates to setting up things in keystone, are you running with keystone enabled? if so then you might need to check the keystone logs to identify the error14:35
*** r-daneel has joined #openstack-ironic14:35
olivierb-yes but again this was running in the exact same conf yesteday14:35
sambettsolivierb-: the 500 error is coming from keystone, so there should be information about what is causing it there, the task that is failing is trying to talk to keystone14:36
sambettsso soemthing is wrong with keystone, perhaps something conflicting from the previous install14:37
olivierb-ok willlook deeper into log shortly14:37
openstackgerritIlya Etingof proposed openstack/ironic master: Adds boot mode support to ManagementInterface  https://review.openstack.org/52677314:38
olivierb-in a conf right now :-(14:39
openstackgerritIlya Etingof proposed openstack/ironic master: Added redfish boot mode management  https://review.openstack.org/52831414:42
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387914:49
*** fragatina has quit IRC14:50
*** fragatina has joined #openstack-ironic14:50
openstackgerritIlya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added  https://review.openstack.org/52387914:52
*** cjloader has joined #openstack-ironic14:57
openstackgerritIlya Etingof proposed openstack/sushy-tools master: added system boot mode support  https://review.openstack.org/52610115:01
*** zhangfei has joined #openstack-ironic15:10
olivierb-Apr 10 16:00:11 master uwsgi[14269]: 2018-04-10 16:00:11.182 14272 CRITICAL keystone [-] Unhandled error: ContextualVersionConflict: (pysaml2 4.5.0 (/usr/local/lib/python2.7/dist-packages), Requirement.parse('pysaml2<4.0.3,>=2.4.0'), set(['keystone']))15:11
olivierb-note that if I remember correctly I am using spike as my main base15:13
olivierb-+keystone_git_branch: stable/pike15:14
olivierb-in playbooks/roles/bifrost-prep-for-install/defaults/main.yml15:15
olivierb- /opt/stack/requirements/upper-constraints.txt:pysaml2===4.5.015:18
olivierb-15:18
olivierb-et dans /opt/stack/requirements/upper-constraints.txt:pysaml2===4.5.015:19
olivierb-sorry for the french part ;-)15:20
olivierb-and /opt/stack/keystone/requirements.txt:pysaml2<4.0.3,>=2.4.0 # Apache-2.015:20
*** cjloader has quit IRC15:20
*** cjloader has joined #openstack-ironic15:20
*** r-daneel_ has joined #openstack-ironic15:25
openstackgerritRuby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports  https://review.openstack.org/53132015:26
*** r-daneel has quit IRC15:26
*** r-daneel_ is now known as r-daneel15:26
*** rwsu has quit IRC15:26
TheJuliaokay, morning meeting over and brain back to semi-sane state15:29
openstackgerritRuby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports  https://review.openstack.org/53132015:30
TheJuliaolivierb-: Interesting, by chance have you gone through your log and determined what caused the newer package to be installed?15:31
*** gyee has joined #openstack-ironic15:31
openstackgerritRuby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports  https://review.openstack.org/53132015:32
olivierb-in fact I am currently thinking that my setup was previously working by luck because I mix pike for keystone with master for all other components :-(15:33
*** AlexeyAbashkin has quit IRC15:33
*** e0ne has quit IRC15:37
*** zhangfei has quit IRC15:38
*** rwsu has joined #openstack-ironic15:39
TheJuliaolivierb-: oh! Yeah, dependencies are not specific per package, thus deployment will fail15:44
TheJulias/deployment/but new dependency differences may fail15:44
* TheJulia wonders if she still has properly articulated that15:44
*** AlexeyAbashkin has joined #openstack-ironic15:47
NobodyCamGood Morning Ironic'ers15:55
*** anupn has joined #openstack-ironic15:56
TheJuliagood morning NobodyCam15:56
NobodyCam:)15:57
*** ktibi_ has joined #openstack-ironic16:02
*** nizam037 has quit IRC16:03
*** ktibi has quit IRC16:04
*** mgoddard has quit IRC16:04
*** e0ne has joined #openstack-ironic16:06
anupnGood morning Ironic'ers :)16:08
olivierb-way better now that keystone is also master, thx for your help16:11
*** rbartal has quit IRC16:11
*** lucasagomes is now known as lucas-afk16:18
NobodyCamMorning anupn :)16:22
TheJuliaolivierb-: awesome16:22
TheJuliaanupn: good morning16:22
anupnNobodyCam, TheJulia good morning16:23
NobodyCam:)16:23
anupnfeel good to be back working on Ironic16:29
*** dprince has quit IRC16:34
*** e0ne has quit IRC16:35
*** milan_ has quit IRC16:35
*** e0ne has joined #openstack-ironic16:36
jlvillalanupn, Welcome back :)16:41
*** marios has quit IRC16:41
*** mgoddard has joined #openstack-ironic16:41
*** fragatina has quit IRC16:49
*** dprince has joined #openstack-ironic16:50
* sambetts --> home16:50
*** sambetts is now known as sambetts|afk16:50
sambetts|afknight all16:50
*** dougsz has quit IRC16:58
*** AlexeyAbashkin has quit IRC16:59
*** derekh has quit IRC17:02
*** AlexeyAbashkin has joined #openstack-ironic17:07
TheJuliaanupn: awesome17:08
*** trown is now known as trown|lunch17:10
*** AlexeyAbashkin has quit IRC17:12
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add ownership information field  https://review.openstack.org/56008917:13
*** itlinux has joined #openstack-ironic17:14
*** fragatina has joined #openstack-ironic17:15
*** tesseract has quit IRC17:16
*** fragatina has quit IRC17:17
jrollooo. ahhhh.17:17
*** fragatina has joined #openstack-ironic17:18
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Correct linter check for petitboot driver spec  https://review.openstack.org/56009117:21
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Add ownership information field  https://review.openstack.org/56008917:23
*** racedo has quit IRC17:33
*** openstackgerrit has quit IRC17:34
*** r-daneel_ has joined #openstack-ironic17:35
*** r-daneel has quit IRC17:36
*** r-daneel_ is now known as r-daneel17:36
*** e0ne has quit IRC17:54
*** mmethot has quit IRC17:59
*** r-daneel_ has joined #openstack-ironic18:16
*** r-daneel has quit IRC18:17
*** r-daneel_ is now known as r-daneel18:17
*** ianychoi has quit IRC18:22
*** ianychoi has joined #openstack-ironic18:23
*** trown|lunch is now known as trown18:23
*** harlowja has joined #openstack-ironic18:25
*** fragatina has quit IRC18:26
*** fragatina has joined #openstack-ironic18:27
*** mjturek has quit IRC18:30
*** AlexeyAbashkin has joined #openstack-ironic18:36
*** mgoddard has quit IRC18:37
*** ianychoi has quit IRC18:41
*** ianychoi has joined #openstack-ironic18:42
*** e0ne has joined #openstack-ironic18:59
*** pbourke has joined #openstack-ironic19:02
pbourkecould anyone tell me which ironic service consumes the [deploy] conf group?19:04
pbourkeis it the conductor?19:04
*** cdearborn_ has joined #openstack-ironic19:10
jrollpbourke: should be all conductor, but once in a while the api will consume conductor configs like that for odd reasons19:12
pbourkejroll: gotcha thanks19:12
jrollso I can't guarantee but should be fine to only configure those on conductor19:12
jrollnp19:12
pbourkeim mostly interested in deploy/power_off_after_deploy_failure which doesn't seem to be getting consumed for me19:12
pbourkeunfortunately19:12
jrollyeah, that should be only conductor19:13
jrollbut,19:13
jrollif you're using nova, it'll call destroy and blow the server away anyway :/19:13
jrollI just hit this the other day19:13
TheJuliayeah, I had fun with that with heat a long time ago19:20
TheJuliajroll: oh, I'm working on writing the federation idea out.... so more oooos and ahhhhhhhs :)19:20
jroll\o/19:21
TheJuliasince I'm on a spec roll, any other ideas we've rambled about that need to be written?19:22
jrollnot off the top of my head19:23
*** Nisha_Agarwal has joined #openstack-ironic19:25
TheJuliafederation will be in a much rougher shape, but it is a stupidly complex idea19:27
*** cdearborn has quit IRC19:27
rloohmm, something changed and pep8 is complaining. eg: https://review.openstack.org/#/c/531320/19:35
patchbotpatch 531320 - ironic - Check for PXE-enabled ports when creating neutron ...19:35
rlooi think it breaks all patches19:36
TheJuliawhatever it is, yes19:36
rlooit breaks for me locally too, master. and i tested that patch this morning and it worked.19:39
*** pbourke has quit IRC19:50
TheJuliaI'm sure many many many people are broken since it is complaining that the license header is there19:52
rlooso i did a diff between a patch were pep8 passed, and where it didn't pass, and it looks like pycodestyle might be the culprit. 2.3.1 vs 2.4.020:00
*** pcaruana has quit IRC20:01
* rloo not sure pep8 failure is due to pycodestyle change based on description of the package. 20:10
*** ElCoyote_ has joined #openstack-ironic20:10
*** rpioso|afk is now known as rpioso20:10
*** AlexeyAbashkin has quit IRC20:16
*** AlexeyAbashkin has joined #openstack-ironic20:17
jrollI think we may have also switched pep8 to py3 in the gate, by default?20:18
jrollidk if that was only nova, or all projects20:18
rloojroll: just asking infra now about py320:19
* jroll sees, hrm20:21
rloojroll: they don't know if there was a change. i did look at a couple of warnings and they are legit. makes me wonder if some hacking stuff was changed.20:21
rloowe haven't modified that code for a long time.20:21
jrollbtw, pycodestyle *is* pep820:21
jroll"This package used to be called pep8 but was renamed to pycodestyle to reduce confusion"20:21
*** Nisha_Agarwal has quit IRC20:21
rlooright. but i think we've been using pycodestyle for awhile. i can try some stuff locally to see if i can pinpoint it. but gotta take a break first.20:22
clarkbI would run tox -repep8 locally20:22
clarkbthat is typically a good first step for debugging these problems20:22
jrollya20:23
jrollrloo: yeah, was referring to the version bump20:23
*** MattMan has quit IRC20:25
* jroll is looking locally20:25
*** AlexeyAbashkin has quit IRC20:25
*** MattMan has joined #openstack-ironic20:25
*** VisBits has quit IRC20:27
*** liuzz has quit IRC20:27
*** e0ne has quit IRC20:30
*** hw_wutianwei has quit IRC20:30
*** hw_wutianwei has joined #openstack-ironic20:32
*** ktibi_ has quit IRC20:33
clarkbrloo: jroll my guess is that since we don't constrain pycodestyle and it isn't otherwise capped some dependency is depending on it in an uncapped manner and pulling it in20:46
jrollclarkb: yeah, looking like it20:46
* jroll still trying to make it work20:46
clarkbhttps://github.com/PyCQA/flake8/blob/2.5.5/setup.py#L5320:47
clarkbfor example there20:47
jrollright20:47
jrollthis is what grabbed it in my env https://github.com/PyCQA/flake8-import-order/blob/master/setup.py#L3320:48
jrolljust testing if capping that works20:48
clarkbthough https://git.openstack.org/cgit/openstack-dev/hacking/tree/requirements.txt?h=1.0.0 should pin it20:48
jrollslow vm is slow20:48
jrollthat doesn't pin pycodestyle though20:48
clarkboh right and pep8 is no longer the name20:49
clarkbha20:49
jroll:)20:49
clarkbworth noting that current master of hacking doesn't pin either20:49
clarkbso ya I'm guessing that is why this happens20:49
jrollyeah20:50
rloosigh. so this might be a problem for other projects too. jroll, should i look into it too or you've got it under control?20:50
clarkbin the past hacking ahs controlled these versions20:51
jrollrloo: yeah I can handle it20:51
clarkbI'm wondering if there was an expectation that constraints would handle it for hacking going forward?20:51
*** trown is now known as trown|outtypewww20:51
jrollpinning made it pass fwiw20:51
jrollyeah, not sure, gonna bug the requirements channel20:51
rloojroll: thx. btw, the pep8 & py3 thing has to be done per repo, so we do the switch when we're ready/want20:51
jrollrloo: ok, kinda figured that, thanks20:52
rloojroll: great about the pinning!20:52
rlooTheJulia: do you know anything about ironi-inspector stable/queens broken (in whiteboard). am updating with this pep8 breakage.20:53
TheJuliarloo: I think that was fixed, lost track of it in my brain yesterday morning20:54
jrollrloo: it looks like the patch merged20:55
rlooTheJulia, jroll: two patches merged. i'll consider it fixed. i think we really should fire off weekly CI tests for our branches...20:56
jrollthat is a thing that can be done20:56
*** bfournie has quit IRC20:56
* TheJulia just wonders time20:57
jrollthe stable folks do this for some projects, they should be able to help set it up20:57
rloojroll: oh, didn't realize that. are you volunteering to look into it? /me hopes20:58
jrollrloo: no, I'm not, sorry20:58
rlooTheJulia: another question, there's an entry about Ansible deploy driver broken by ansible 2.5 release. i think you fixed that? although yours is a temporary fix?20:58
rloojroll: i tried :)20:59
*** priteau has quit IRC20:59
*** Goneri has quit IRC20:59
*** priteau has joined #openstack-ironic21:00
TheJuliarloo: I believe one of the last things dmitry did last week was push that through21:01
TheJuliaso the perm fix should be there at this point and I believe he backported it to queens, but I'm not 100% sure at the moment21:01
rlooTheJulia: ok, i'll just 'mark' it as fixed :) thx.21:02
*** dprince has quit IRC21:03
*** priteau has quit IRC21:04
rloojroll: i updated the whiteboard; I put you down as looking into it. please update with status etc. Thx!21:07
*** openstackgerrit has joined #openstack-ironic21:07
openstackgerritJulia Kreger proposed openstack/ironic-specs master: Proposing the federation of ironic  https://review.openstack.org/56015221:07
jrollwill do21:07
*** jjohnson2 has quit IRC21:08
jrollrloo: did you file a bug, btw?21:10
rloojroll: nope. you want me to open a story on it?21:11
jrollrloo: could you, please?21:11
rloojroll: sure. i love stories :)21:11
jrollmaking an ironic commit for the short-term, though I'm not sure that's the best way to do it21:11
jrollthanks21:11
rloojroll: https://storyboard.openstack.org/#!/story/200181921:14
jrollthank you :)21:14
rloojroll: i need to fix the description etc but thought you might want the number/task21:14
jrollyep :)21:14
*** jrist has quit IRC21:14
openstackgerritJim Rollenhagen proposed openstack/ironic master: Pin pycodestyle to <=2.3.1  https://review.openstack.org/56015721:15
jrolldoing that for now, though not sure if it'll pass the global-requirements check21:15
*** jcoufal has quit IRC21:16
*** cjloader has quit IRC21:19
rloojroll: i looked in requirements, and it has https://github.com/openstack/requirements/blob/master/openstack_requirements/tests/files/upper-constraints.txt#L41121:20
rloojroll: BUT i have no idea how it all works21:20
jrollrloo: this is the file used for projects: https://github.com/openstack/requirements/blob/master/upper-constraints.txt21:20
jrolland it isn't there, because,21:20
jrollit's here https://github.com/openstack/requirements/blob/master/blacklist.txt#L1321:21
jrollto allow projects to set pins themselves21:21
rloojroll: oh, in that case, I guess we should pin. we might want to look at the others in the blacklist too. in our spare time.21:21
* jroll just re-learned this from the requirements team21:21
*** baha has quit IRC21:22
openstackgerritJim Rollenhagen proposed openstack/ironic master: Pin pycodestyle to <=2.3.1  https://review.openstack.org/56015721:23
jrollupdated commit message to match21:23
jrollrloo: TheJulia: that should be ready to go when tests pass ^21:24
rloojroll: yup. i'll try to keep an eye on it. am not around for long, but i'll check on/off.21:24
jrollthanks21:24
TheJuliajroll: thanks21:25
jrollnp21:25
*** bfournie has joined #openstack-ironic21:30
*** jrist has joined #openstack-ironic21:41
*** priteau has joined #openstack-ironic21:58
*** priteau has quit IRC22:08
*** itlinux has quit IRC22:15
*** rcernin has joined #openstack-ironic22:23
*** rpioso is now known as rpioso|afk22:34
*** lbragstad has quit IRC22:42
*** livelace has quit IRC22:50
*** r-daneel has quit IRC22:59
*** r-daneel has joined #openstack-ironic22:59
*** fragatina has quit IRC22:59
*** fragatina has joined #openstack-ironic23:02
*** lbragstad has joined #openstack-ironic23:09
*** r-daneel has quit IRC23:14
*** cjloader has joined #openstack-ironic23:20
rlooTheJulia, jlvillal, any other core, would be good to approve this to fix the pep8 failures: https://review.openstack.org/#/c/560157/23:21
patchbotpatch 560157 - ironic - Pin pycodestyle to <=2.3.123:21
jlvillalrloo, Looking23:22
TheJuliaI cliked approved23:22
rloothx TheJulia! jlvillal, thx too :)23:23
jlvillal:)23:23
rlooand I'm off til tomorrow23:23
*** rloo has quit IRC23:23
jlvillalrloo, Good night!23:23
* jlvillal looks into pep8 errors with new pycodestyle23:24
* jlvillal is kind of scared of seeing how bad it is23:24
*** cjloader has quit IRC23:24
jlvillalTheJulia, jroll: I wonder why not just ignore E402 and W503 instead of locking pycodestyle version.23:33
*** cdearborn_ has quit IRC23:33
openstackgerritMerged openstack/ironic master: Pin pycodestyle to <=2.3.1  https://review.openstack.org/56015723:37
clarkbjlvillal: assuming the rules make sense you may want to push a change that fixes them and unpins pycodestyle or bumps up the pin23:38
jlvillalclarkb, Will W503 for sure is not something that I feel should be enforced.23:39
jlvillalUnsure about E40223:39
* clarkb looks them up23:39
jlvillals/Will/Well/23:40
clarkboh I like W50323:40
clarkbthough paired with W504 not sure it makes sense23:41
clarkboh good they are mutually exclusive. Docs say that W504 and W503 are not enforced by default so this may also be a bug in pycodestyle23:41
jlvillalclarkb, Right. It is a style decision.23:41
clarkbhttps://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes asterisk note at bottom of that section23:42
jlvillalclarkb, thanks23:42
clarkbI wonder if the ignore at https://git.openstack.org/cgit/openstack/ironic/tree/tox.ini#n104 means that is the only thing that pycodestyle will ignore23:43
clarkbbasically overriding the default list?23:43
jlvillalclarkb, I would hope not23:43
TheJuliarealistically we should backport the pin and possibly the branches at a stable point in time23:46
TheJulias/possibly/leave/23:46
clarkbjlvillal: I'm working to test that theory really quickly23:48
jlvillalclarkb, cool23:49
clarkb/ironic/tests/unit/common/test_images.py:264:9: E731 do not assign a lambda expression, use a def23:49
clarkbthats an interesting rule too23:49
*** rbudden has quit IRC23:50
clarkbjlvillal: theory disproven so I think it may be an upstream bug23:50
clarkb(W503/4 specifically)23:50
jlvillalclarkb, Thanks. Maybe sigmavirus knows23:51
openstackgerritJohn L. Villalovos proposed openstack/ironic master: Remove pycodestyle version pin. Add E402 and W503 to ignore.  https://review.openstack.org/56018123:55

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