*** logan- has quit IRC | 00:02 | |
*** logan- has joined #openstack-ironic | 00:02 | |
*** pmannidi has joined #openstack-ironic | 00:10 | |
*** bfournie has joined #openstack-ironic | 00:11 | |
*** gyee has quit IRC | 00:11 | |
*** mjturek has quit IRC | 00:19 | |
*** exodusftw has joined #openstack-ironic | 00:29 | |
*** cjloader has joined #openstack-ironic | 00:32 | |
*** cjloader has quit IRC | 00:33 | |
*** cjloader has joined #openstack-ironic | 00:34 | |
*** hshiina has joined #openstack-ironic | 00:44 | |
*** rloo has quit IRC | 00:52 | |
*** tiendc has joined #openstack-ironic | 01:04 | |
*** fanzhang has joined #openstack-ironic | 01:04 | |
*** zhangfei has joined #openstack-ironic | 01:10 | |
*** jcoufal has joined #openstack-ironic | 01:31 | |
*** namnh has joined #openstack-ironic | 02:08 | |
*** r-daneel has joined #openstack-ironic | 02:36 | |
*** rbudden has quit IRC | 02:52 | |
*** jcoufal has quit IRC | 03:07 | |
*** cjloader has quit IRC | 03:16 | |
*** cjloader_ has joined #openstack-ironic | 03:16 | |
*** harlowja has quit IRC | 03:16 | |
*** ianychoi has quit IRC | 03:16 | |
*** cjloader_ has quit IRC | 03:19 | |
*** cjloader has joined #openstack-ironic | 03:20 | |
*** ianychoi has joined #openstack-ironic | 03:20 | |
openstackgerrit | Kaifeng Wang proposed openstack/ironic master: Implementation of inspect wait state https://review.openstack.org/555610 | 03:24 |
---|---|---|
openstackgerrit | shuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml https://review.openstack.org/517883 | 03:28 |
*** phuongnh has joined #openstack-ironic | 03:36 | |
*** harlowja has joined #openstack-ironic | 03:57 | |
*** pcaruana has joined #openstack-ironic | 04:06 | |
*** vabada2 has joined #openstack-ironic | 04:10 | |
*** liuzz has quit IRC | 04:11 | |
*** strigazi has quit IRC | 04:11 | |
*** vabada has quit IRC | 04:13 | |
*** strigazi has joined #openstack-ironic | 04:14 | |
*** ianychoi has quit IRC | 04:14 | |
*** ianychoi has joined #openstack-ironic | 04:16 | |
*** pcaruana has quit IRC | 04:16 | |
openstackgerrit | shuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml https://review.openstack.org/517883 | 04:17 |
*** liuzz has joined #openstack-ironic | 04:25 | |
*** cjloader has quit IRC | 04:54 | |
*** marios has joined #openstack-ironic | 04:58 | |
openstackgerrit | shuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml https://review.openstack.org/517883 | 05:06 |
*** links has joined #openstack-ironic | 05:11 | |
*** strigazi_ has joined #openstack-ironic | 05:13 | |
*** vabada has joined #openstack-ironic | 05:14 | |
*** vabada2 has quit IRC | 05:17 | |
*** strigazi has quit IRC | 05:17 | |
*** fragatina has quit IRC | 05:31 | |
*** etingof has quit IRC | 05:42 | |
*** ianychoi has quit IRC | 05:47 | |
openstackgerrit | shuaili.wang proposed openstack/ironic master: modify the Ironic api-ref's parameters in parameters.yaml https://review.openstack.org/517883 | 05:48 |
*** ianychoi has joined #openstack-ironic | 05:49 | |
*** fragatina has joined #openstack-ironic | 05:57 | |
*** fragatin_ has joined #openstack-ironic | 05:58 | |
*** fragatina has quit IRC | 06:01 | |
*** fragatin_ has quit IRC | 06:02 | |
*** racedo has joined #openstack-ironic | 06:13 | |
*** cjloader has joined #openstack-ironic | 06:16 | |
*** namnh has quit IRC | 06:18 | |
*** cjloader has quit IRC | 06:19 | |
*** e0ne has joined #openstack-ironic | 06:29 | |
*** AlexeyAbashkin has joined #openstack-ironic | 06:32 | |
*** pcaruana has joined #openstack-ironic | 06:40 | |
*** harlowja has quit IRC | 06:40 | |
*** rcernin has quit IRC | 06:41 | |
*** fragatina has joined #openstack-ironic | 06:47 | |
*** arnewiebalck has joined #openstack-ironic | 06:47 | |
*** AlexeyAbashkin has quit IRC | 06:49 | |
*** AlexeyAbashkin has joined #openstack-ironic | 06:50 | |
*** AlexeyAbashkin has quit IRC | 06:54 | |
*** fragatina has quit IRC | 06:56 | |
*** jtomasek has joined #openstack-ironic | 07:02 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:02 | |
*** milan has joined #openstack-ironic | 07:04 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 07:14 |
*** tesseract has joined #openstack-ironic | 07:19 | |
*** livelace has joined #openstack-ironic | 07:22 | |
*** fragatina has joined #openstack-ironic | 07:30 | |
*** jhesketh_ has joined #openstack-ironic | 07:31 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 07:33 |
*** jhesketh has quit IRC | 07:37 | |
*** mgoddard has joined #openstack-ironic | 07:37 | |
*** hoonetorg has quit IRC | 07:38 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 07:41 |
*** hoonetorg has joined #openstack-ironic | 07:52 | |
*** AlexeyAbashkin has quit IRC | 07:53 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:56 | |
*** mgoddard has quit IRC | 07:56 | |
*** milan has quit IRC | 08:00 | |
*** ktibi has joined #openstack-ironic | 08:01 | |
*** lucas-afk is now known as lucasagomes | 08:05 | |
*** pmannidi has quit IRC | 08:17 | |
*** dougsz has joined #openstack-ironic | 08:19 | |
*** jesusaur has quit IRC | 08:23 | |
*** jesusaur has joined #openstack-ironic | 08:27 | |
*** priteau has joined #openstack-ironic | 08:27 | |
*** mgoddard has joined #openstack-ironic | 08:30 | |
*** milan has joined #openstack-ironic | 08:37 | |
*** etingof has joined #openstack-ironic | 08:41 | |
*** rbartal has joined #openstack-ironic | 08:44 | |
openstackgerrit | jiapei proposed openstack/ironic master: Add documentatin for XClarity Driver https://review.openstack.org/559960 | 08:45 |
*** tiendc has quit IRC | 08:48 | |
*** tiendc has joined #openstack-ironic | 08:49 | |
*** etingof has quit IRC | 08:51 | |
*** milan_ has joined #openstack-ironic | 08:51 | |
*** milan has quit IRC | 08:51 | |
*** hshiina has quit IRC | 09:02 | |
*** pcaruana has quit IRC | 09:04 | |
vdrok | good morning ironic! | 09:10 |
*** milan_ has quit IRC | 09:11 | |
*** etingof has joined #openstack-ironic | 09:17 | |
*** links has quit IRC | 09:18 | |
*** derekh has joined #openstack-ironic | 09:30 | |
*** links has joined #openstack-ironic | 09:33 | |
*** stendulker has joined #openstack-ironic | 09:46 | |
*** nizam037 has joined #openstack-ironic | 09:51 | |
*** strigazi_ is now known as strigazi | 10:01 | |
*** jesusaur has quit IRC | 10:02 | |
*** jesusaur has joined #openstack-ironic | 10:15 | |
*** strigazi is now known as strigaz_ | 10:59 | |
*** strigaz_ is now known as strigazi_ | 10:59 | |
*** strigazi_ is now known as strigazi | 10:59 | |
jroll | morning vdrok and anyone else :) | 11:00 |
*** sambetts|afk is now known as sambetts | 11:04 | |
sambetts | o/ | 11:04 |
*** stendulker_ has joined #openstack-ironic | 11:08 | |
*** stendulker has quit IRC | 11:12 | |
*** bfournie has quit IRC | 11:14 | |
*** stendulker_ has quit IRC | 11:14 | |
*** lucasagomes is now known as lucas-hungry | 11:17 | |
*** zhangfei has quit IRC | 11:19 | |
*** dprince has joined #openstack-ironic | 11:20 | |
*** links has quit IRC | 11:26 | |
*** dprince has quit IRC | 11:28 | |
*** ajya has joined #openstack-ironic | 11:30 | |
*** dprince has joined #openstack-ironic | 11:31 | |
*** phuongnh has quit IRC | 11:35 | |
*** milan_ has joined #openstack-ironic | 11:36 | |
*** tiendc has quit IRC | 11:39 | |
*** links has joined #openstack-ironic | 11:40 | |
*** bfournie has joined #openstack-ironic | 12:02 | |
*** weshay_pto is now known as weshay | 12:03 | |
*** openstackgerrit has quit IRC | 12:04 | |
*** lucas-hungry is now known as lucasagomes | 12:11 | |
vdrok | good morning jroll , sambetts | 12:11 |
*** hjensas has quit IRC | 12:17 | |
*** hjensas has joined #openstack-ironic | 12:20 | |
*** hjensas has quit IRC | 12:20 | |
*** hjensas has joined #openstack-ironic | 12:20 | |
*** trown|outtypewww is now known as trown | 12:20 | |
*** openstackgerrit has joined #openstack-ironic | 12:21 | |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: Switch to none auth for standalone mode https://review.openstack.org/359061 | 12:21 |
openstackgerrit | Vladyslav Drok proposed openstack/python-ironicclient master: WIP allow to use none auth in functional tests https://review.openstack.org/508330 | 12:21 |
*** jcoufal has joined #openstack-ironic | 12:29 | |
*** rloo has joined #openstack-ironic | 12:30 | |
*** rbudden has joined #openstack-ironic | 12:35 | |
*** pcaruana has joined #openstack-ironic | 12:44 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 12:51 |
*** logan- has quit IRC | 12:55 | |
*** logan- has joined #openstack-ironic | 12:56 | |
*** zhurong has joined #openstack-ironic | 12:59 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 13:00 |
*** mjturek has joined #openstack-ironic | 13:01 | |
*** zhurong has quit IRC | 13:13 | |
*** jjohnson2 has joined #openstack-ironic | 13:18 | |
TheJulia | good morning everyone | 13:22 |
rloo | good morning TheJulia and everyone else! | 13:26 |
*** Goneri has joined #openstack-ironic | 13:32 | |
*** baha has joined #openstack-ironic | 13:38 | |
*** lbragstad has joined #openstack-ironic | 13:42 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 13:47 |
*** cdearborn has joined #openstack-ironic | 13:50 | |
*** serverascode has quit IRC | 13:50 | |
*** serverascode has joined #openstack-ironic | 13:50 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 13:52 |
*** fdegir has quit IRC | 13:54 | |
*** gregwork has quit IRC | 13:54 | |
*** fdegir has joined #openstack-ironic | 13:54 | |
*** gregwork has joined #openstack-ironic | 13:54 | |
*** betherly has quit IRC | 13:56 | |
*** betherly has joined #openstack-ironic | 13:57 | |
*** cdearborn has quit IRC | 13:58 | |
*** r-daneel has quit IRC | 14:02 | |
*** ianychoi has quit IRC | 14:02 | |
*** ianychoi has joined #openstack-ironic | 14:03 | |
*** ajya has quit IRC | 14:03 | |
*** ajya has joined #openstack-ironic | 14:03 | |
*** cdearborn has joined #openstack-ironic | 14:08 | |
*** timirnich has quit IRC | 14:08 | |
*** ildikov has quit IRC | 14:08 | |
*** timirnich has joined #openstack-ironic | 14:09 | |
*** ildikov has joined #openstack-ironic | 14:09 | |
*** patrickeast has quit IRC | 14:11 | |
*** patrickeast has joined #openstack-ironic | 14:11 | |
*** davidlenwell has quit IRC | 14:13 | |
*** davidlenwell has joined #openstack-ironic | 14:13 | |
*** links has quit IRC | 14:13 | |
*** portdirect has quit IRC | 14:19 | |
*** portdirect has joined #openstack-ironic | 14:19 | |
*** csatari has quit IRC | 14:19 | |
*** csatari has joined #openstack-ironic | 14:19 | |
*** samueldmq has quit IRC | 14:21 | |
*** sergek_ has quit IRC | 14:21 | |
*** samueldmq has joined #openstack-ironic | 14:21 | |
*** sergek_ has joined #openstack-ironic | 14:22 | |
*** zul has quit IRC | 14:22 | |
*** r-daneel has joined #openstack-ironic | 14:22 | |
*** zul has joined #openstack-ironic | 14:22 | |
*** ajya has quit IRC | 14:23 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: add warning log about empty pxe_enabled_ports https://review.openstack.org/531320 | 14: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)\n | 14:31 |
olivierb- | however everything seems to be ok | 14:31 |
olivierb- | retried deploying but wtihout more success | 14:32 |
olivierb- | rings any bell ??? | 14:32 |
*** r-daneel has quit IRC | 14:32 | |
olivierb- | hidden dependency broken ? | 14:32 |
olivierb- | thanks for your understanding | 14:32 |
sambetts | olivierb-: 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 error | 14:35 |
*** r-daneel has joined #openstack-ironic | 14:35 | |
olivierb- | yes but again this was running in the exact same conf yesteday | 14:35 |
sambetts | olivierb-: 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 keystone | 14:36 |
sambetts | so soemthing is wrong with keystone, perhaps something conflicting from the previous install | 14:37 |
olivierb- | ok willlook deeper into log shortly | 14:37 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Adds boot mode support to ManagementInterface https://review.openstack.org/526773 | 14:38 |
olivierb- | in a conf right now :-( | 14:39 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added redfish boot mode management https://review.openstack.org/528314 | 14:42 |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added https://review.openstack.org/523879 | 14:49 |
*** fragatina has quit IRC | 14:50 | |
*** fragatina has joined #openstack-ironic | 14:50 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added https://review.openstack.org/523879 | 14:52 |
*** cjloader has joined #openstack-ironic | 14:57 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: added system boot mode support https://review.openstack.org/526101 | 15:01 |
*** zhangfei has joined #openstack-ironic | 15: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 base | 15:13 |
olivierb- | +keystone_git_branch: stable/pike | 15:14 |
olivierb- | in playbooks/roles/bifrost-prep-for-install/defaults/main.yml | 15:15 |
olivierb- | /opt/stack/requirements/upper-constraints.txt:pysaml2===4.5.0 | 15:18 |
olivierb- | 15:18 | |
olivierb- | et dans /opt/stack/requirements/upper-constraints.txt:pysaml2===4.5.0 | 15: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.0 | 15:20 |
*** cjloader has quit IRC | 15:20 | |
*** cjloader has joined #openstack-ironic | 15:20 | |
*** r-daneel_ has joined #openstack-ironic | 15:25 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports https://review.openstack.org/531320 | 15:26 |
*** r-daneel has quit IRC | 15:26 | |
*** r-daneel_ is now known as r-daneel | 15:26 | |
*** rwsu has quit IRC | 15:26 | |
TheJulia | okay, morning meeting over and brain back to semi-sane state | 15:29 |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports https://review.openstack.org/531320 | 15:30 |
TheJulia | olivierb-: 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-ironic | 15:31 | |
openstackgerrit | Ruby Loo proposed openstack/ironic master: Check for PXE-enabled ports when creating neutron ports https://review.openstack.org/531320 | 15: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 IRC | 15:33 | |
*** e0ne has quit IRC | 15:37 | |
*** zhangfei has quit IRC | 15:38 | |
*** rwsu has joined #openstack-ironic | 15:39 | |
TheJulia | olivierb-: oh! Yeah, dependencies are not specific per package, thus deployment will fail | 15:44 |
TheJulia | s/deployment/but new dependency differences may fail | 15:44 |
* TheJulia wonders if she still has properly articulated that | 15:44 | |
*** AlexeyAbashkin has joined #openstack-ironic | 15:47 | |
NobodyCam | Good Morning Ironic'ers | 15:55 |
*** anupn has joined #openstack-ironic | 15:56 | |
TheJulia | good morning NobodyCam | 15:56 |
NobodyCam | :) | 15:57 |
*** ktibi_ has joined #openstack-ironic | 16:02 | |
*** nizam037 has quit IRC | 16:03 | |
*** ktibi has quit IRC | 16:04 | |
*** mgoddard has quit IRC | 16:04 | |
*** e0ne has joined #openstack-ironic | 16:06 | |
anupn | Good morning Ironic'ers :) | 16:08 |
olivierb- | way better now that keystone is also master, thx for your help | 16:11 |
*** rbartal has quit IRC | 16:11 | |
*** lucasagomes is now known as lucas-afk | 16:18 | |
NobodyCam | Morning anupn :) | 16:22 |
TheJulia | olivierb-: awesome | 16:22 |
TheJulia | anupn: good morning | 16:22 |
anupn | NobodyCam, TheJulia good morning | 16:23 |
NobodyCam | :) | 16:23 |
anupn | feel good to be back working on Ironic | 16:29 |
*** dprince has quit IRC | 16:34 | |
*** e0ne has quit IRC | 16:35 | |
*** milan_ has quit IRC | 16:35 | |
*** e0ne has joined #openstack-ironic | 16:36 | |
jlvillal | anupn, Welcome back :) | 16:41 |
*** marios has quit IRC | 16:41 | |
*** mgoddard has joined #openstack-ironic | 16:41 | |
*** fragatina has quit IRC | 16:49 | |
*** dprince has joined #openstack-ironic | 16:50 | |
* sambetts --> home | 16:50 | |
*** sambetts is now known as sambetts|afk | 16:50 | |
sambetts|afk | night all | 16:50 |
*** dougsz has quit IRC | 16:58 | |
*** AlexeyAbashkin has quit IRC | 16:59 | |
*** derekh has quit IRC | 17:02 | |
*** AlexeyAbashkin has joined #openstack-ironic | 17:07 | |
TheJulia | anupn: awesome | 17:08 |
*** trown is now known as trown|lunch | 17:10 | |
*** AlexeyAbashkin has quit IRC | 17:12 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Add ownership information field https://review.openstack.org/560089 | 17:13 |
*** itlinux has joined #openstack-ironic | 17:14 | |
*** fragatina has joined #openstack-ironic | 17:15 | |
*** tesseract has quit IRC | 17:16 | |
*** fragatina has quit IRC | 17:17 | |
jroll | ooo. ahhhh. | 17:17 |
*** fragatina has joined #openstack-ironic | 17:18 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Correct linter check for petitboot driver spec https://review.openstack.org/560091 | 17:21 |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Add ownership information field https://review.openstack.org/560089 | 17:23 |
*** racedo has quit IRC | 17:33 | |
*** openstackgerrit has quit IRC | 17:34 | |
*** r-daneel_ has joined #openstack-ironic | 17:35 | |
*** r-daneel has quit IRC | 17:36 | |
*** r-daneel_ is now known as r-daneel | 17:36 | |
*** e0ne has quit IRC | 17:54 | |
*** mmethot has quit IRC | 17:59 | |
*** r-daneel_ has joined #openstack-ironic | 18:16 | |
*** r-daneel has quit IRC | 18:17 | |
*** r-daneel_ is now known as r-daneel | 18:17 | |
*** ianychoi has quit IRC | 18:22 | |
*** ianychoi has joined #openstack-ironic | 18:23 | |
*** trown|lunch is now known as trown | 18:23 | |
*** harlowja has joined #openstack-ironic | 18:25 | |
*** fragatina has quit IRC | 18:26 | |
*** fragatina has joined #openstack-ironic | 18:27 | |
*** mjturek has quit IRC | 18:30 | |
*** AlexeyAbashkin has joined #openstack-ironic | 18:36 | |
*** mgoddard has quit IRC | 18:37 | |
*** ianychoi has quit IRC | 18:41 | |
*** ianychoi has joined #openstack-ironic | 18:42 | |
*** e0ne has joined #openstack-ironic | 18:59 | |
*** pbourke has joined #openstack-ironic | 19:02 | |
pbourke | could anyone tell me which ironic service consumes the [deploy] conf group? | 19:04 |
pbourke | is it the conductor? | 19:04 |
*** cdearborn_ has joined #openstack-ironic | 19:10 | |
jroll | pbourke: should be all conductor, but once in a while the api will consume conductor configs like that for odd reasons | 19:12 |
pbourke | jroll: gotcha thanks | 19:12 |
jroll | so I can't guarantee but should be fine to only configure those on conductor | 19:12 |
jroll | np | 19:12 |
pbourke | im mostly interested in deploy/power_off_after_deploy_failure which doesn't seem to be getting consumed for me | 19:12 |
pbourke | unfortunately | 19:12 |
jroll | yeah, that should be only conductor | 19:13 |
jroll | but, | 19:13 |
jroll | if you're using nova, it'll call destroy and blow the server away anyway :/ | 19:13 |
jroll | I just hit this the other day | 19:13 |
TheJulia | yeah, I had fun with that with heat a long time ago | 19:20 |
TheJulia | jroll: oh, I'm working on writing the federation idea out.... so more oooos and ahhhhhhhs :) | 19:20 |
jroll | \o/ | 19:21 |
TheJulia | since I'm on a spec roll, any other ideas we've rambled about that need to be written? | 19:22 |
jroll | not off the top of my head | 19:23 |
*** Nisha_Agarwal has joined #openstack-ironic | 19:25 | |
TheJulia | federation will be in a much rougher shape, but it is a stupidly complex idea | 19:27 |
*** cdearborn has quit IRC | 19:27 | |
rloo | hmm, something changed and pep8 is complaining. eg: https://review.openstack.org/#/c/531320/ | 19:35 |
patchbot | patch 531320 - ironic - Check for PXE-enabled ports when creating neutron ... | 19:35 |
rloo | i think it breaks all patches | 19:36 |
TheJulia | whatever it is, yes | 19:36 |
rloo | it breaks for me locally too, master. and i tested that patch this morning and it worked. | 19:39 |
*** pbourke has quit IRC | 19:50 | |
TheJulia | I'm sure many many many people are broken since it is complaining that the license header is there | 19:52 |
rloo | so 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.0 | 20:00 |
*** pcaruana has quit IRC | 20:01 | |
* rloo not sure pep8 failure is due to pycodestyle change based on description of the package. | 20:10 | |
*** ElCoyote_ has joined #openstack-ironic | 20:10 | |
*** rpioso|afk is now known as rpioso | 20:10 | |
*** AlexeyAbashkin has quit IRC | 20:16 | |
*** AlexeyAbashkin has joined #openstack-ironic | 20:17 | |
jroll | I think we may have also switched pep8 to py3 in the gate, by default? | 20:18 |
jroll | idk if that was only nova, or all projects | 20:18 |
rloo | jroll: just asking infra now about py3 | 20:19 |
* jroll sees, hrm | 20:21 | |
rloo | jroll: 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 |
rloo | we haven't modified that code for a long time. | 20:21 |
jroll | btw, pycodestyle *is* pep8 | 20:21 |
jroll | "This package used to be called pep8 but was renamed to pycodestyle to reduce confusion" | 20:21 |
*** Nisha_Agarwal has quit IRC | 20:21 | |
rloo | right. 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 |
clarkb | I would run tox -repep8 locally | 20:22 |
clarkb | that is typically a good first step for debugging these problems | 20:22 |
jroll | ya | 20:23 |
jroll | rloo: yeah, was referring to the version bump | 20:23 |
*** MattMan has quit IRC | 20:25 | |
* jroll is looking locally | 20:25 | |
*** AlexeyAbashkin has quit IRC | 20:25 | |
*** MattMan has joined #openstack-ironic | 20:25 | |
*** VisBits has quit IRC | 20:27 | |
*** liuzz has quit IRC | 20:27 | |
*** e0ne has quit IRC | 20:30 | |
*** hw_wutianwei has quit IRC | 20:30 | |
*** hw_wutianwei has joined #openstack-ironic | 20:32 | |
*** ktibi_ has quit IRC | 20:33 | |
clarkb | rloo: 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 in | 20:46 |
jroll | clarkb: yeah, looking like it | 20:46 |
* jroll still trying to make it work | 20:46 | |
clarkb | https://github.com/PyCQA/flake8/blob/2.5.5/setup.py#L53 | 20:47 |
clarkb | for example there | 20:47 |
jroll | right | 20:47 |
jroll | this is what grabbed it in my env https://github.com/PyCQA/flake8-import-order/blob/master/setup.py#L33 | 20:48 |
jroll | just testing if capping that works | 20:48 |
clarkb | though https://git.openstack.org/cgit/openstack-dev/hacking/tree/requirements.txt?h=1.0.0 should pin it | 20:48 |
jroll | slow vm is slow | 20:48 |
jroll | that doesn't pin pycodestyle though | 20:48 |
clarkb | oh right and pep8 is no longer the name | 20:49 |
clarkb | ha | 20:49 |
jroll | :) | 20:49 |
clarkb | worth noting that current master of hacking doesn't pin either | 20:49 |
clarkb | so ya I'm guessing that is why this happens | 20:49 |
jroll | yeah | 20:50 |
rloo | sigh. 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 |
clarkb | in the past hacking ahs controlled these versions | 20:51 |
jroll | rloo: yeah I can handle it | 20:51 |
clarkb | I'm wondering if there was an expectation that constraints would handle it for hacking going forward? | 20:51 |
*** trown is now known as trown|outtypewww | 20:51 | |
jroll | pinning made it pass fwiw | 20:51 |
jroll | yeah, not sure, gonna bug the requirements channel | 20:51 |
rloo | jroll: thx. btw, the pep8 & py3 thing has to be done per repo, so we do the switch when we're ready/want | 20:51 |
jroll | rloo: ok, kinda figured that, thanks | 20:52 |
rloo | jroll: great about the pinning! | 20:52 |
rloo | TheJulia: do you know anything about ironi-inspector stable/queens broken (in whiteboard). am updating with this pep8 breakage. | 20:53 |
TheJulia | rloo: I think that was fixed, lost track of it in my brain yesterday morning | 20:54 |
jroll | rloo: it looks like the patch merged | 20:55 |
rloo | TheJulia, jroll: two patches merged. i'll consider it fixed. i think we really should fire off weekly CI tests for our branches... | 20:56 |
jroll | that is a thing that can be done | 20:56 |
*** bfournie has quit IRC | 20:56 | |
* TheJulia just wonders time | 20:57 | |
jroll | the stable folks do this for some projects, they should be able to help set it up | 20:57 |
rloo | jroll: oh, didn't realize that. are you volunteering to look into it? /me hopes | 20:58 |
jroll | rloo: no, I'm not, sorry | 20:58 |
rloo | TheJulia: 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 |
rloo | jroll: i tried :) | 20:59 |
*** priteau has quit IRC | 20:59 | |
*** Goneri has quit IRC | 20:59 | |
*** priteau has joined #openstack-ironic | 21:00 | |
TheJulia | rloo: I believe one of the last things dmitry did last week was push that through | 21:01 |
TheJulia | so 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 moment | 21:01 |
rloo | TheJulia: ok, i'll just 'mark' it as fixed :) thx. | 21:02 |
*** dprince has quit IRC | 21:03 | |
*** priteau has quit IRC | 21:04 | |
rloo | jroll: i updated the whiteboard; I put you down as looking into it. please update with status etc. Thx! | 21:07 |
*** openstackgerrit has joined #openstack-ironic | 21:07 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Proposing the federation of ironic https://review.openstack.org/560152 | 21:07 |
jroll | will do | 21:07 |
*** jjohnson2 has quit IRC | 21:08 | |
jroll | rloo: did you file a bug, btw? | 21:10 |
rloo | jroll: nope. you want me to open a story on it? | 21:11 |
jroll | rloo: could you, please? | 21:11 |
rloo | jroll: sure. i love stories :) | 21:11 |
jroll | making an ironic commit for the short-term, though I'm not sure that's the best way to do it | 21:11 |
jroll | thanks | 21:11 |
rloo | jroll: https://storyboard.openstack.org/#!/story/2001819 | 21:14 |
jroll | thank you :) | 21:14 |
rloo | jroll: i need to fix the description etc but thought you might want the number/task | 21:14 |
jroll | yep :) | 21:14 |
*** jrist has quit IRC | 21:14 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Pin pycodestyle to <=2.3.1 https://review.openstack.org/560157 | 21:15 |
jroll | doing that for now, though not sure if it'll pass the global-requirements check | 21:15 |
*** jcoufal has quit IRC | 21:16 | |
*** cjloader has quit IRC | 21:19 | |
rloo | jroll: i looked in requirements, and it has https://github.com/openstack/requirements/blob/master/openstack_requirements/tests/files/upper-constraints.txt#L411 | 21:20 |
rloo | jroll: BUT i have no idea how it all works | 21:20 |
jroll | rloo: this is the file used for projects: https://github.com/openstack/requirements/blob/master/upper-constraints.txt | 21:20 |
jroll | and it isn't there, because, | 21:20 |
jroll | it's here https://github.com/openstack/requirements/blob/master/blacklist.txt#L13 | 21:21 |
jroll | to allow projects to set pins themselves | 21:21 |
rloo | jroll: 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 team | 21:21 | |
*** baha has quit IRC | 21:22 | |
openstackgerrit | Jim Rollenhagen proposed openstack/ironic master: Pin pycodestyle to <=2.3.1 https://review.openstack.org/560157 | 21:23 |
jroll | updated commit message to match | 21:23 |
jroll | rloo: TheJulia: that should be ready to go when tests pass ^ | 21:24 |
rloo | jroll: yup. i'll try to keep an eye on it. am not around for long, but i'll check on/off. | 21:24 |
jroll | thanks | 21:24 |
TheJulia | jroll: thanks | 21:25 |
jroll | np | 21:25 |
*** bfournie has joined #openstack-ironic | 21:30 | |
*** jrist has joined #openstack-ironic | 21:41 | |
*** priteau has joined #openstack-ironic | 21:58 | |
*** priteau has quit IRC | 22:08 | |
*** itlinux has quit IRC | 22:15 | |
*** rcernin has joined #openstack-ironic | 22:23 | |
*** rpioso is now known as rpioso|afk | 22:34 | |
*** lbragstad has quit IRC | 22:42 | |
*** livelace has quit IRC | 22:50 | |
*** r-daneel has quit IRC | 22:59 | |
*** r-daneel has joined #openstack-ironic | 22:59 | |
*** fragatina has quit IRC | 22:59 | |
*** fragatina has joined #openstack-ironic | 23:02 | |
*** lbragstad has joined #openstack-ironic | 23:09 | |
*** r-daneel has quit IRC | 23:14 | |
*** cjloader has joined #openstack-ironic | 23:20 | |
rloo | TheJulia, jlvillal, any other core, would be good to approve this to fix the pep8 failures: https://review.openstack.org/#/c/560157/ | 23:21 |
patchbot | patch 560157 - ironic - Pin pycodestyle to <=2.3.1 | 23:21 |
jlvillal | rloo, Looking | 23:22 |
TheJulia | I cliked approved | 23:22 |
rloo | thx TheJulia! jlvillal, thx too :) | 23:23 |
jlvillal | :) | 23:23 |
rloo | and I'm off til tomorrow | 23:23 |
*** rloo has quit IRC | 23:23 | |
jlvillal | rloo, Good night! | 23:23 |
* jlvillal looks into pep8 errors with new pycodestyle | 23:24 | |
* jlvillal is kind of scared of seeing how bad it is | 23:24 | |
*** cjloader has quit IRC | 23:24 | |
jlvillal | TheJulia, jroll: I wonder why not just ignore E402 and W503 instead of locking pycodestyle version. | 23:33 |
*** cdearborn_ has quit IRC | 23:33 | |
openstackgerrit | Merged openstack/ironic master: Pin pycodestyle to <=2.3.1 https://review.openstack.org/560157 | 23:37 |
clarkb | jlvillal: assuming the rules make sense you may want to push a change that fixes them and unpins pycodestyle or bumps up the pin | 23:38 |
jlvillal | clarkb, Will W503 for sure is not something that I feel should be enforced. | 23:39 |
jlvillal | Unsure about E402 | 23:39 |
* clarkb looks them up | 23:39 | |
jlvillal | s/Will/Well/ | 23:40 |
clarkb | oh I like W503 | 23:40 |
clarkb | though paired with W504 not sure it makes sense | 23:41 |
clarkb | oh good they are mutually exclusive. Docs say that W504 and W503 are not enforced by default so this may also be a bug in pycodestyle | 23:41 |
jlvillal | clarkb, Right. It is a style decision. | 23:41 |
clarkb | https://pycodestyle.readthedocs.io/en/latest/intro.html#error-codes asterisk note at bottom of that section | 23:42 |
jlvillal | clarkb, thanks | 23:42 |
clarkb | I 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 ignore | 23:43 |
clarkb | basically overriding the default list? | 23:43 |
jlvillal | clarkb, I would hope not | 23:43 |
TheJulia | realistically we should backport the pin and possibly the branches at a stable point in time | 23:46 |
TheJulia | s/possibly/leave/ | 23:46 |
clarkb | jlvillal: I'm working to test that theory really quickly | 23:48 |
jlvillal | clarkb, cool | 23:49 |
clarkb | /ironic/tests/unit/common/test_images.py:264:9: E731 do not assign a lambda expression, use a def | 23:49 |
clarkb | thats an interesting rule too | 23:49 |
*** rbudden has quit IRC | 23:50 | |
clarkb | jlvillal: theory disproven so I think it may be an upstream bug | 23:50 |
clarkb | (W503/4 specifically) | 23:50 |
jlvillal | clarkb, Thanks. Maybe sigmavirus knows | 23:51 |
openstackgerrit | John L. Villalovos proposed openstack/ironic master: Remove pycodestyle version pin. Add E402 and W503 to ignore. https://review.openstack.org/560181 | 23:55 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!