| *** d0ugal_ has quit IRC | 00:10 | |
| *** d0ugal_ has joined #openstack-ironic | 00:24 | |
| *** lbragstad has joined #openstack-ironic | 00:34 | |
| *** dims has quit IRC | 00:42 | |
| *** liuzz has joined #openstack-ironic | 00:46 | |
| *** lbragstad has quit IRC | 00:49 | |
| *** sambetts_ has quit IRC | 00:59 | |
| *** lbragstad has joined #openstack-ironic | 00:59 | |
| *** sambetts_ has joined #openstack-ironic | 01:00 | |
| *** cjloader has joined #openstack-ironic | 01:02 | |
| *** hshiina has joined #openstack-ironic | 01:03 | |
| *** cjloader has quit IRC | 01:07 | |
| *** etingof has quit IRC | 01:07 | |
| *** d0ugal__ has joined #openstack-ironic | 01:08 | |
| *** d0ugal_ has quit IRC | 01:11 | |
| *** d0ugal has joined #openstack-ironic | 01:14 | |
| *** lbragstad has quit IRC | 01:14 | |
| *** d0ugal__ has quit IRC | 01:16 | |
| *** sambetts_ has quit IRC | 01:19 | |
| *** etingof has joined #openstack-ironic | 01:20 | |
| *** sambetts_ has joined #openstack-ironic | 01:21 | |
| *** cjloader has joined #openstack-ironic | 01:31 | |
| openstackgerrit | Zachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group https://review.openstack.org/551922 | 01:47 |
|---|---|---|
| *** masuberu has joined #openstack-ironic | 01:54 | |
| *** masber has quit IRC | 01:58 | |
| *** masber has joined #openstack-ironic | 02:00 | |
| *** masuberu has quit IRC | 02:01 | |
| *** masber has quit IRC | 02:13 | |
| *** cjloader has quit IRC | 02:20 | |
| *** cjloader has joined #openstack-ironic | 02:21 | |
| *** cjloader has quit IRC | 02:25 | |
| *** Sukhdev has joined #openstack-ironic | 02:34 | |
| *** pvc has joined #openstack-ironic | 03:03 | |
| pvc | hi | 03:03 |
| *** liuzz_ has joined #openstack-ironic | 03:16 | |
| *** liuzz has quit IRC | 03:19 | |
| openstackgerrit | Zachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group https://review.openstack.org/551922 | 03:28 |
| *** Sukhdev has quit IRC | 03:33 | |
| *** zhangfei has joined #openstack-ironic | 03:47 | |
| *** lbragstad has joined #openstack-ironic | 03:48 | |
| *** masber has joined #openstack-ironic | 03:49 | |
| *** pbourke_ has quit IRC | 04:05 | |
| *** pbourke_ has joined #openstack-ironic | 04:06 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 04:07 | |
| *** mvpnitesh has joined #openstack-ironic | 04:26 | |
| *** fragatina has quit IRC | 05:01 | |
| *** marios has joined #openstack-ironic | 05:10 | |
| *** d0ugal has quit IRC | 05:27 | |
| *** d0ugal has joined #openstack-ironic | 05:29 | |
| *** mvpnitesh has quit IRC | 05:47 | |
| *** e0ne has joined #openstack-ironic | 05:49 | |
| *** mjura has joined #openstack-ironic | 05:54 | |
| *** mjura has quit IRC | 05:54 | |
| *** mjura has joined #openstack-ironic | 05:56 | |
| *** zhangfei has quit IRC | 05:57 | |
| *** mvpnitesh has joined #openstack-ironic | 05:58 | |
| *** liuzz has joined #openstack-ironic | 05:59 | |
| *** liuzz has quit IRC | 05:59 | |
| *** liuzz has joined #openstack-ironic | 06:00 | |
| *** liuzz_ has quit IRC | 06:02 | |
| *** rbartal has joined #openstack-ironic | 06:05 | |
| *** zhangfei has joined #openstack-ironic | 06:10 | |
| *** fragatina has joined #openstack-ironic | 06:10 | |
| *** parlos has joined #openstack-ironic | 06:11 | |
| *** sambetts_ has quit IRC | 06:38 | |
| *** sambetts_ has joined #openstack-ironic | 06:40 | |
| *** jtomasek has joined #openstack-ironic | 06:46 | |
| *** rcernin has quit IRC | 06:56 | |
| *** fragatina has quit IRC | 07:08 | |
| *** livelace-link has quit IRC | 07:09 | |
| *** livelace-link has joined #openstack-ironic | 07:10 | |
| *** livelace-link has quit IRC | 07:11 | |
| *** pcaruana has joined #openstack-ironic | 07:13 | |
| *** racedo has joined #openstack-ironic | 07:19 | |
| *** tesseract has joined #openstack-ironic | 07:21 | |
| *** jaganathan has joined #openstack-ironic | 07:26 | |
| pas-ha | morning ironic :-) | 07:27 |
| *** tesseract has quit IRC | 07:33 | |
| *** tesseract has joined #openstack-ironic | 07:35 | |
| etingof | o/ | 07:36 |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add storage disk drive https://review.openstack.org/561533 | 07:43 |
| *** mgoddard has joined #openstack-ironic | 07:44 | |
| *** threestrands has quit IRC | 07:47 | |
| *** itlinux has joined #openstack-ironic | 07:47 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 07:58 | |
| *** itlinux_ has joined #openstack-ironic | 08:02 | |
| pvc | hi | 08:02 |
| *** itlinux has quit IRC | 08:04 | |
| *** athomas has joined #openstack-ironic | 08:05 | |
| *** itlinux_ has quit IRC | 08:06 | |
| *** sambetts_ has quit IRC | 08:09 | |
| *** lucas-afk is now known as lucasagomes | 08:10 | |
| *** cjloader has joined #openstack-ironic | 08:10 | |
| *** sambetts_ has joined #openstack-ironic | 08:13 | |
| *** dougsz has joined #openstack-ironic | 08:15 | |
| *** mvpnitesh has quit IRC | 08:15 | |
| *** Nisha_Agarwal has quit IRC | 08:19 | |
| openstackgerrit | Zachary Ma proposed openstack/networking-generic-switch master: Huawei vrpv8 switch support port group https://review.openstack.org/551922 | 08:26 |
| *** derekh has joined #openstack-ironic | 08:28 | |
| *** milan has joined #openstack-ironic | 08:29 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 08:37 | |
| *** mvpnitesh has joined #openstack-ironic | 08:37 | |
| hshiina | TheJulia: Hi, I updated nova rescue patch: https://review.openstack.org/#/c/416487 | 08:39 |
| patchbot | patch 416487 - nova - virt/ironic: Implement rescue and unrescue | 08:39 |
| *** itlinux has joined #openstack-ironic | 08:53 | |
| *** itlinux has quit IRC | 08:54 | |
| *** itlinux has joined #openstack-ironic | 08:57 | |
| *** devananda has quit IRC | 09:13 | |
| *** itlinux has quit IRC | 09:28 | |
| *** rbartal has quit IRC | 09:35 | |
| Nisha_Agarwal | jroll, hi | 09:51 |
| *** milan has quit IRC | 09:57 | |
| *** milan has joined #openstack-ironic | 09:57 | |
| *** pbourke_ is now known as pbourke | 10:05 | |
| *** rbartal has joined #openstack-ironic | 10:13 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: documentation on simulators added https://review.openstack.org/523445 | 10:16 |
| *** niteshmvp has joined #openstack-ironic | 10:20 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: SNMPv3 security features added to the `snmp` driver https://review.openstack.org/547999 | 10:30 |
| *** fragatina has joined #openstack-ironic | 10:32 | |
| *** milan has joined #openstack-ironic | 10:48 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add simple storage https://review.openstack.org/561582 | 10:50 |
| *** Nisha_Agarwal has quit IRC | 10:55 | |
| *** bfournie has joined #openstack-ironic | 10:56 | |
| *** fragatina has quit IRC | 10:57 | |
| *** slagle has joined #openstack-ironic | 10:57 | |
| *** lucasagomes is now known as lucas-hungry | 11:04 | |
| *** sambetts_ is now known as sambetts | 11:06 | |
| sambetts | Morning all | 11:10 |
| *** zhangfei has quit IRC | 11:12 | |
| *** itlinux has joined #openstack-ironic | 11:16 | |
| *** Nisha_Agarwal has joined #openstack-ironic | 11:23 | |
| Nisha_Agarwal | jroll, TheJulia hi. We commented out https://github.com/openstack/nova/blob/stable/pike/nova/objects/instance.py#L824 and the nova boot succeeded. The issue was reproducible on another pike setup up as well till we commented out this line | 11:24 |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Fix ``agent`` deploy interface to call ``boot.prepare_instance`` https://review.openstack.org/499050 | 11:40 |
| *** hshiina is now known as hshiina|afk | 11:42 | |
| *** dprince has joined #openstack-ironic | 11:47 | |
| *** pvc has quit IRC | 12:00 | |
| *** cjloader has quit IRC | 12:01 | |
| *** cjloader has joined #openstack-ironic | 12:01 | |
| *** lucas-hungry is now known as lucasagomes | 12:02 | |
| *** cjloader has quit IRC | 12:06 | |
| *** jcoufal has joined #openstack-ironic | 12:09 | |
| *** rh-jelabarre has joined #openstack-ironic | 12:09 | |
| etingof | vdrok, hey, any pointers on how to best mock a non-existing module attr? | 12:12 |
| sambetts | etingof: like this: https://github.com/openstack/ironic/blob/master/ironic/tests/unit/drivers/third_party_driver_mocks.py | 12:13 |
| *** d0ugal_ has joined #openstack-ironic | 12:13 | |
| vdrok | etingof: morning. yes, that :) | 12:13 |
| *** d0ugal has quit IRC | 12:15 | |
| etingof | sambetts, vdrok I see, thanks! I've been thinking of asking the Mock class itself to *add* a new attr if it's not there at patching time | 12:16 |
| *** Nisha_Agarwal has quit IRC | 12:17 | |
| *** mgoddard has quit IRC | 12:21 | |
| *** dims has joined #openstack-ironic | 12:30 | |
| *** liuzz_ has joined #openstack-ironic | 12:37 | |
| jroll | morning everyone | 12:41 |
| *** liuzz has quit IRC | 12:41 | |
| *** itlinux has quit IRC | 12:46 | |
| *** oanson has quit IRC | 12:49 | |
| *** trown|outtypewww is now known as trown | 12:53 | |
| *** mjturek has joined #openstack-ironic | 12:54 | |
| *** mgoddard has joined #openstack-ironic | 12:56 | |
| *** oanson has joined #openstack-ironic | 12:57 | |
| *** marios has quit IRC | 12:57 | |
| *** serlex has joined #openstack-ironic | 12:57 | |
| *** cjloader has joined #openstack-ironic | 13:02 | |
| *** Goneri has joined #openstack-ironic | 13:02 | |
| *** cjloader has quit IRC | 13:06 | |
| *** marios has joined #openstack-ironic | 13:08 | |
| * TheJulia wipes sleep from her eyes | 13:17 | |
| TheJulia | hshiina|afk: thanks! | 13:17 |
| *** jjohnson2 has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Adds boot mode support to ManagementInterface https://review.openstack.org/526773 | 13:25 |
| *** jroll has quit IRC | 13:26 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Added redfish boot mode management https://review.openstack.org/528314 | 13:26 |
| *** mvpnitesh has quit IRC | 13:26 | |
| *** niteshmvp has quit IRC | 13:27 | |
| *** jroll has joined #openstack-ironic | 13:27 | |
| TheJulia | Well, looks like my worst hack fix ever seems to... work | 13:33 |
| *** superdan is now known as dansmith | 13:35 | |
| jroll | TheJulia: it does? where do you see it passing? | 13:36 |
| TheJulia | jroll: the grenade job ran ever so long and hit the 3 hour timeout | 13:37 |
| TheJulia | it was literally done as far as I can tell and it got killed | 13:37 |
| jroll | oh! do it did | 13:37 |
| TheJulia | ? | 13:37 |
| jroll | so it did* | 13:38 |
| jroll | :) | 13:38 |
| * jroll finds more coffee | 13:38 | |
| TheJulia | heh | 13:38 |
| *** d0ugal_ has quit IRC | 13:40 | |
| *** d0ugal has joined #openstack-ironic | 13:41 | |
| *** d0ugal has quit IRC | 13:41 | |
| *** d0ugal has joined #openstack-ironic | 13:41 | |
| *** baha has joined #openstack-ironic | 13:43 | |
| TheJulia | not entirely sure if its own branch grenade job is healthy. failed 1 hour 20 minutes in, looks like messaging issues | 13:44 |
| jroll | isn't that job still busted on queens due to the segv thing? | 13:45 |
| TheJulia | it doesn't look like it even got to upgrading | 13:46 |
| TheJulia | Apr 14 03:34:45.026925 ubuntu-xenial-inap-mtl01-0003515050 devstack@n-api.service[12887]: !!! uWSGI process 12890 got Segmentation Fault !!! | 13:47 |
| * TheJulia guesses unstable node | 13:47 | |
| jroll | could be | 13:47 |
| jroll | at any rate, going to recheck that master patch and see if the timeout is a fluke | 13:48 |
| TheJulia | http://logs.openstack.org/58/561358/4/check/ironic-grenade-dsvm-multinode-multitenant/1039016/logs/screen-n-api.txt.gz#_Apr_14_03_34_43_786131 | 13:48 |
| TheJulia | ack | 13:48 |
| jroll | oh, you know what, there's some issues with neutron going on. not sure if related | 13:52 |
| * jroll looks around | 13:52 | |
| TheJulia | looks like the job failed due to an incompatible oslo_db library... I wonder if this is the result of getting rid of the programatic requirements sync :\ | 13:54 |
| *** derekh has quit IRC | 13:54 | |
| jroll | which, master? | 13:55 |
| *** derekh has joined #openstack-ironic | 13:56 | |
| *** r-daneel has quit IRC | 13:57 | |
| TheJulia | queens, but I don't remember if that was turned off, looks like the oslo.db version that it thinks is last queens... odd | 13:57 |
| jroll | hm | 13:59 |
| TheJulia | I think I need more coffee | 13:59 |
| jroll | that was the same issue as when we tried to restart n-cpu to handle the segv stuff, iirc | 13:59 |
| jroll | ++ | 14:00 |
| TheJulia | Yeah, and uwsgi will auto-restart things | 14:00 |
| TheJulia | which means... bad things can occur | 14:00 |
| * jroll is only going to worry about master atm | 14:00 | |
| TheJulia | ++ | 14:00 |
| TheJulia | our grenade is a half-completed upgrade scenario anyway | 14:01 |
| * TheJulia has strong feels and a strong desire to drink when thinking about it | 14:01 | |
| jroll | half? | 14:01 |
| * TheJulia goes and gets showered and then more coffeed | 14:01 | |
| TheJulia | yeah, we never finish nova | 14:01 |
| TheJulia | because we can't change the order of operations | 14:02 |
| jroll | that's what "normal" grenade is for, multinode only tests the mixed versions bit | 14:02 |
| jroll | anyway, go coffee :) | 14:02 |
| *** rbartal has quit IRC | 14:04 | |
| *** hjensas has quit IRC | 14:06 | |
| *** trown|brb has joined #openstack-ironic | 14:11 | |
| *** trown has quit IRC | 14:11 | |
| TheJulia | yeah, except were running grenade in that case, not normal multinode. *shrugs* | 14:17 |
| *** rloo has joined #openstack-ironic | 14:20 | |
| *** kaifeng has joined #openstack-ironic | 14:29 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB model https://review.openstack.org/511162 | 14:30 |
| *** rpioso|afk is now known as rpioso | 14:38 | |
| rpioso | Good morning | 14:39 |
| *** stendulker has joined #openstack-ironic | 14:39 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: Add bios_interface db field https://review.openstack.org/528609 | 14:42 |
| rloo | good morning rpioso and everyone else! | 14:43 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add DB API https://review.openstack.org/511402 | 14:43 |
| *** stendulker_ has joined #openstack-ironic | 14:43 | |
| rloo | etingof: wrt https://review.openstack.org/#/c/560019/. if this is blocking stuff, i can approve if you want to do a follow up patch. | 14:44 |
| patchbot | patch 560019 - sushy-tools - update lower-constraints.txt to match *requirement... | 14:44 |
| *** hshiina has joined #openstack-ironic | 14:46 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add RPC object https://review.openstack.org/511714 | 14:46 |
| *** stendulker has quit IRC | 14:46 | |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOSInterface https://review.openstack.org/507793 | 14:46 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: BIOS Settings: Add BIOS caching https://review.openstack.org/512200 | 14:47 |
| openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add Node BIOS support - REST API https://review.openstack.org/512579 | 14:48 |
| NobodyCam | Good Morning Ironic'ers | 14:50 |
| rpioso | Hey rloo and NobodyCam :) | 14:51 |
| rloo | morning NobodyCam! | 14:51 |
| rloo | 9 minutes til ironic meeting. | 14:51 |
| NobodyCam | hey hey Happy monday rpioso and rloo | 14:51 |
| rloo | please update status for ironic meeting etc | 14:51 |
| *** hjensas has joined #openstack-ironic | 14:54 | |
| *** hjensas has quit IRC | 14:54 | |
| *** hjensas has joined #openstack-ironic | 14:54 | |
| *** gcb has joined #openstack-ironic | 14:56 | |
| TheJulia | Good morning everyone | 14:57 |
| TheJulia | rloo: thanks for the reminder :) | 14:57 |
| rloo | TheJulia: you're welcome, and good morning! | 14:57 |
| *** namnh has joined #openstack-ironic | 14:57 | |
| NobodyCam | morning TheJulia :) | 14:58 |
| *** beekneemech is now known as bnemec | 14:59 | |
| *** rajinir has joined #openstack-ironic | 14:59 | |
| TheJulia | #startmeeting ironic | 15:00 |
| openstack | Meeting started Mon Apr 16 15:00:10 2018 UTC and is due to finish in 60 minutes. The chair is TheJulia. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
| *** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
| openstack | The meeting name has been set to 'ironic' | 15:00 |
| vdrok | o/ | 15:00 |
| TheJulia | o/ everone | 15:00 |
| kaifeng | o/ | 15:00 |
| rloo | o/ | 15:00 |
| NobodyCam | o/ | 15:00 |
| mgoddard | o/ | 15:00 |
| hshiina | o/ | 15:00 |
| *** jiapei has joined #openstack-ironic | 15:00 | |
| TheJulia | Our agenda can be found on the wiki, and seems quite light today. | 15:00 |
| hjensas | o/ | 15:00 |
| TheJulia | #link https://wiki.openstack.org/wiki/Meetings/Ironic#Agenda_for_next_meeting | 15:00 |
| TheJulia | #topic Announcements/Reminders | 15:01 |
| stendulker_ | o/ | 15:01 |
| *** openstack changes topic to "Announcements/Reminders (Meeting topic: ironic)" | 15:01 | |
| jroll | \o | 15:01 |
| *** trown|brb is now known as trown | 15:01 | |
| TheJulia | #info This week is the Rocky-1 Milestone | 15:01 |
| rpioso | o/ | 15:01 |
| rloo | TheJulia: is there anything ironic needs to do wrt the milestone? | 15:02 |
| jiapei | o/ | 15:02 |
| TheJulia | rloo: I do not believe so, we should only keep in mind that we need to be ready to release around milestone 3 | 15:02 |
| TheJulia | or shortly after I should say. | 15:02 |
| rloo | TheJulia: right. | 15:02 |
| *** r-daneel has joined #openstack-ironic | 15:03 | |
| *** Nisha_ has joined #openstack-ironic | 15:03 | |
| TheJulia | #info We have a board in Storyboard to aid in triaging stories. For those that do not know, we've been reviewing items across the community and adding a triaged tag if they have been reviewed. This allows us to make sure things are still valid, or if they are no longer valid, mark them as such. | 15:03 |
| jroll | link? :) | 15:03 |
| TheJulia | One moment! :) | 15:04 |
| TheJulia | It is slow to load, because it is running somewhat complex queries across ALL of the projects in ironic. Please keep that in mind. | 15:04 |
| TheJulia | #link https://storyboard.openstack.org/#!/board/67 | 15:04 |
| jroll | nice, thanks | 15:04 |
| rloo | TheJulia: nice! | 15:05 |
| * rloo isn't crazy about 'panopticon though, seems somewhat negative' | 15:05 | |
| TheJulia | It is also entirely automatic, so it will update as things change. No need to drag/drop. | 15:05 |
| TheJulia | rloo: I was thinking about architecture at the time... you can rename it if you want, you have that power :) | 15:05 |
| * jroll puts it in the 'dashboards' bit in the whiteboard | 15:05 | |
| rloo | TheJulia: thx. Others are welcome too :) | 15:06 |
| TheJulia | Anyone have anything else to announce or reminds us of?? | 15:06 |
| rloo | there's elections for something coming up? | 15:06 |
| rloo | tc | 15:06 |
| jiapei | Yes | 15:06 |
| rloo | and our CI is still down, right? | 15:06 |
| jroll | just a reminder that multinode grenade is currently broken on master, julia and I have been working on it | 15:06 |
| jroll | yes | 15:07 |
| jroll | :) | 15:07 |
| TheJulia | Yes, TC election starts next week | 15:07 |
| jroll | seems we probably have a fix, attempting to validate it | 15:07 |
| rloo | TheJulia: is there a deadline wrt getting forum topics? | 15:07 |
| TheJulia | #info Multinode Grenade CI testing is broken at the moment as Jim mentioned, "fix" being tested | 15:08 |
| TheJulia | rloo: That was Sunday :( | 15:08 |
| * rloo crosses fingers that fix works. | 15:08 | |
| rloo | TheJulia: OH. ok, scratch that one off the list! | 15:08 |
| jroll | rloo: fwiw, the fix seems to work, but the job is timing out for (probably unrelated) reasons | 15:08 |
| jroll | waiting to see if it was a fluke or not | 15:08 |
| TheJulia | Anyway, we should move on if there is nothing else | 15:08 |
| rloo | grrrrr... could be another problem that snuck in... | 15:08 |
| rloo | + move on | 15:09 |
| TheJulia | #topic Review action items from previous meeting | 15:09 |
| *** openstack changes topic to "Review action items from previous meeting (Meeting topic: ironic)" | 15:09 | |
| TheJulia | #info One action item in preparation for the call we had later in the week. Completed. | 15:09 |
| TheJulia | #topic Review subteam status reports | 15:09 |
| *** openstack changes topic to "Review subteam status reports (Meeting topic: ironic)" | 15:09 | |
| TheJulia | #link https://etherpad.openstack.org/p/IronicWhiteBoard | 15:09 |
| *** pcaruana has quit IRC | 15:09 | |
| TheJulia | So I realize our CI was broken for a good chunk of last week, and that prevented us from landing things in ironic, so the status does seem to be relatively unchanged. I went ahead and added some python-ironicclient and a housekeeping patch to the priority list for this week. | 15:10 |
| jroll | looks like decent progress on bios stuff | 15:11 |
| mjturek | o/ | 15:12 |
| TheJulia | Yeah, it looks like we're just going to need to recheck some stuff once CI is happy again | 15:12 |
| TheJulia | It really is looking like it is in good shape and I think it would be good for us to try and review and land soon | 15:13 |
| TheJulia | Although I've not looked beyond the RPC patches yet, I need to do that. | 15:13 |
| *** Nisha_ has quit IRC | 15:13 | |
| rloo | wrt sighup support for ironic-inspector. i looked at the patch. it doesn' tuse oslo.service cuz it doesn't seem like flask can be used with it. Does anyone know? I suggested they send email to ask. | 15:14 |
| jroll | yeah, we can land the first few after CI is healthy | 15:14 |
| jroll | rloo: oh, this came up in keystone... lemme find a ML link | 15:14 |
| jroll | thread starts here: http://lists.openstack.org/pipermail/openstack-dev/2018-March/128471.html | 15:14 |
| jroll | there's some suggestions in replies | 15:15 |
| jroll | not sure if anyone has written shared code, though | 15:15 |
| *** pc_m- has quit IRC | 15:15 | |
| rloo | jroll: ok, will add link to the patch, thx! | 15:15 |
| TheJulia | jroll: awesome, thanks | 15:15 |
| jroll | :) | 15:15 |
| rloo | TheJulia: wrt mox -- L369. have you done that? declared it done? | 15:15 |
| TheJulia | rloo: No, I've not, I've been literally swamped | 15:16 |
| rloo | TheJulia: it doesn't need to be you. any volunteers? | 15:16 |
| TheJulia | #action TheJulia to declare ironic done for mox | 15:16 |
| rloo | heh, that's another way to do it :) | 15:16 |
| *** pc_m has joined #openstack-ironic | 15:16 | |
| TheJulia | :) | 15:17 |
| TheJulia | Anyone have any other questions or comments regarding status updates? | 15:17 |
| * jroll does not | 15:17 | |
| rloo | TheJulia: wrt rescue, are there blockers on that? last week, there was mention of back/forth... | 15:17 |
| TheJulia | rloo: Great question, stendulker_ revised the patch that we're blocked on this morning after the back and forth last week. | 15:17 |
| TheJulia | I've not had a chance to look at it, but we really need to review and proceed forth in some direction with rescue this week. | 15:18 |
| rloo | TheJulia: so presumably he revised it on some agreed upon approach by the parties that were doing the back/forth? or are more eyes needed? | 15:18 |
| jlvillal | jroll, Was https://review.openstack.org/#/c/560884/ done to fix the multi-node grenade CI? | 15:19 |
| patchbot | patch 560884 - networking-generic-switch - Sync hacking requirement with ironic (MERGED) | 15:19 |
| rloo | TheJulia: more eyes needed wrt direction? | 15:19 |
| TheJulia | He summarized the takeaways, he revised it, I think more eyes are needed once we have CI up and running to just go ahead and merge it if we're in consensus | 15:19 |
| TheJulia | But I've not looked yet this morning, so I can't say for sure | 15:20 |
| jroll | jlvillal: that was the hope, but alas it did not help | 15:20 |
| jlvillal | jroll, Thanks | 15:20 |
| rloo | TheJulia: ok, I'll try to look later this week. thx. | 15:20 |
| jroll | jlvillal: current thought: https://review.openstack.org/#/c/561358/ | 15:20 |
| patchbot | patch 561358 - networking-generic-switch (stable/queens) - Ensure pep8 is present before trying to proceed | 15:20 |
| *** Nisha_Agarwal has joined #openstack-ironic | 15:20 | |
| jlvillal | okay | 15:20 |
| TheJulia | heh | 15:21 |
| TheJulia | Anyway, seems like we're good to move along to priorities for this coming week? | 15:21 |
| jroll | ++ | 15:21 |
| rloo | ++ | 15:21 |
| TheJulia | #topic Priorities for the coming week | 15:21 |
| *** openstack changes topic to "Priorities for the coming week (Meeting topic: ironic)" | 15:21 | |
| TheJulia | I've already updated the list in the etherpad of what I think would be good, it includes a few smaller patches that we can consider | 15:22 |
| TheJulia | I'm good with what is there, I'm not aware of anything that absolutely has to go on that list this week. | 15:22 |
| jroll | fine with it as well | 15:23 |
| jroll | +A'd that last one, but will leave it on the list in case it fails | 15:23 |
| *** xdjiapei has joined #openstack-ironic | 15:24 | |
| TheJulia | jroll: awesome, thanks | 15:24 |
| rloo | Thx TheJulia, I'm fine with it (although rescue seems higher than client but i doubt it matters) | 15:24 |
| TheJulia | It really doesn't in the grand scheme of things | 15:24 |
| *** mjura has quit IRC | 15:24 | |
| TheJulia | They are also fairly small patches for the client | 15:25 |
| *** xdjiapei has quit IRC | 15:25 | |
| TheJulia | Does anyone have anything they wish to raise before we move on? | 15:25 |
| jiapei | Yes, about the XClarity driver | 15:26 |
| jiapei | a story is created | 15:26 |
| TheJulia | Is it something that should be a project priority this week? | 15:26 |
| jiapei | oh, perhaps not... | 15:26 |
| TheJulia | Otherwise it might be good to discuss during our Open Discussion | 15:26 |
| jiapei | sorry | 15:27 |
| TheJulia | jiapei: no worries! | 15:27 |
| TheJulia | jiapei: you may want to take a look at https://etherpad.openstack.org/p/IronicWhiteBoard around line 148 | 15:27 |
| TheJulia | Anyway, I guess I'll move on | 15:28 |
| TheJulia | #topic Bug Triaging for the coming week | 15:28 |
| *** openstack changes topic to "Bug Triaging for the coming week (Meeting topic: ironic)" | 15:28 | |
| TheJulia | I think this portion of the meeting can be skipped moving forward, at least until we've gone through our backlog of items and have a better idea of what is outstanding | 15:30 |
| rloo | + | 15:31 |
| rloo | + | 15:31 |
| rloo | + | 15:31 |
| jroll | I'm fine with that | 15:31 |
| rloo | :) | 15:31 |
| TheJulia | heh | 15:31 |
| TheJulia | Okay, moving on! | 15:31 |
| TheJulia | #topic Open Discussion | 15:31 |
| *** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:31 | |
| TheJulia | So what shall we discuss today? Taking over the world perhaps? :) | 15:31 |
| rloo | i wanted to thank mjturek for organizing the bug bash last week | 15:32 |
| TheJulia | thanks mjturek! | 15:32 |
| rloo | Thank you mjturek for organizing the bug bash last week :) | 15:32 |
| mjturek | np rloo and TheJulia! | 15:32 |
| mjturek | If we ever want to do another one I'd be happy to help. It was real fun | 15:32 |
| TheJulia | It was fun, and we should do it again... perhaps we can find a time that works every couple weeks? | 15:33 |
| mjturek | +1 | 15:33 |
| * rloo thinks once a month might be more realistic... | 15:33 | |
| TheJulia | rloo: agreed | 15:33 |
| *** jistr is now known as jistr|mtg | 15:33 | |
| mjturek | TheJulia: I'll send something out on the ML about it, maybe something like the first Thursday of the month or such | 15:34 |
| jroll | jiapei wanted to discuss xclarity | 15:34 |
| jroll | and yes, thanks mjturek :) | 15:34 |
| jiapei | Yes | 15:34 |
| TheJulia | mjturek: Awesome, thanks! | 15:34 |
| mjturek | np jroll :) | 15:34 |
| jiapei | ty jroll | 15:34 |
| TheJulia | jiapei: You have the floor | 15:34 |
| jiapei | Well, I've created a story 2001841 | 15:35 |
| jiapei | with 2 tasks, could you help review the patches :) | 15:35 |
| TheJulia | \o/ storyboard is tracking links again | 15:35 |
| TheJulia | well, gerrit really | 15:35 |
| mjturek | #link https://storyboard.openstack.org/#!/story/2001841 | 15:35 |
| mjturek | oh whoops... that didn't work | 15:36 |
| TheJulia | nope, I need to do it :( | 15:36 |
| jroll | jiapei: that code patch needs some unit test updates and pep8 fixes, but yes, will try to take a look once they're updated | 15:36 |
| TheJulia | #link https://storyboard.openstack.org/#!/story/2001841 | 15:36 |
| rloo | yay! (that the links are being added to storyboard) | 15:36 |
| TheJulia | jiapei: I was going to say what jroll said. Looks like your editor may be adding blank lines to the end of files being opened as well, which will cause pep8 errors. | 15:37 |
| jiapei | Ah | 15:38 |
| jiapei | It's blank line... | 15:38 |
| jiapei | I tried to find the pep8 errors but didnt realize it's the blank that leads to | 15:39 |
| TheJulia | yeah, you can run the tests locally `tox -epep8`, `tox -epy35` and look at some of the result data to help you locally so your not having to wait on CI. | 15:39 |
| rloo | jiapei: there is some documentation here wrt running tests locally: https://docs.openstack.org/ironic/latest/contributor/dev-quickstart.html#running-unit-tests-locally | 15:40 |
| TheJulia | Running pep8 validation results in the various hacking rules getting loaded which govern things that the overall openstack community has agreed to ignore or be strict about. | 15:41 |
| jiapei | I also have a question about the 3rd CI, do we have a doc about how to setup it? | 15:41 |
| openstackgerrit | Clark Boylan proposed openstack/ironic stable/queens: Preserve env when running vbmc https://review.openstack.org/559856 | 15:42 |
| TheJulia | jiapei: There is the upstream documentation on how to connect to gerrit event streamer and there should be general docs on how to do that. Ironic specific you'd want to listen for change events on ironic that are appropriate for the xclarity driver to execute a test on | 15:42 |
| openstackgerrit | Clark Boylan proposed openstack/ironic master: Preserve env when running vbmc https://review.openstack.org/559811 | 15:42 |
| TheJulia | jiapei: from there, it gets a little more complex depending on your test scenario, and typically everyone's requirements are a little different so we'll have to kind of work with you there | 15:43 |
| TheJulia | #link https://docs.openstack.org/infra/system-config/third_party.html | 15:43 |
| jiapei | Yeah, I'll look through the links above | 15:44 |
| jiapei | Seems a little complex | 15:44 |
| TheJulia | jiapei: Sadly it does tend to be a little complex, that last link also covers two different ways, which might seem a bit confusing | 15:45 |
| TheJulia | jiapei: Hope that at least helps. Feel free to ask questions, and welcome! | 15:45 |
| mgoddard | I have a thing, if jiapei is finished? | 15:46 |
| mgoddard | Will take silence as a yes | 15:46 |
| TheJulia | Always a good plan :_ | 15:46 |
| TheJulia | err :) | 15:46 |
| jiapei | TheJulia: :) | 15:47 |
| *** lucasagomes is now known as lucas-brb | 15:47 | |
| mgoddard | We have a summer intern who has chosen a project called 'Save the inspector' | 15:47 |
| TheJulia | mgoddard: I like it! | 15:47 |
| mgoddard | It's perhaps a little optimistic, but will hopefully at least lead to some contributions | 15:47 |
| TheJulia | mgoddard: are they in need of a crash of sorts? | 15:48 |
| jroll | sweet! | 15:48 |
| mgoddard | I've worked with him before and he's very switched on | 15:48 |
| rloo | awesome | 15:48 |
| TheJulia | err, crash course | 15:48 |
| TheJulia | excellent! | 15:48 |
| mgoddard | he spent a year with us at cray so is familiar with openstack and ironic etc. | 15:48 |
| TheJulia | most excellent | 15:48 |
| mgoddard | so if anyone has any particular bugs or features that are urgent, let me know | 15:49 |
| rloo | did the HA work get finished? | 15:49 |
| mgoddard | when the time gets closer I'll start compiling a list of suggestions for him | 15:49 |
| TheJulia | rloo: essentially yes, but not entirely into TripleO AIUI | 15:49 |
| TheJulia | I think derekh was going to pick that up | 15:49 |
| mgoddard | is it enabled by default? | 15:50 |
| rloo | TheJulia: ah. the other thing that sticks in my mind might be too big a chunk, but we talked about refactoring/putting some inspector stuff into ironic | 15:50 |
| TheJulia | in OoO? | 15:50 |
| *** stendulker has joined #openstack-ironic | 15:50 | |
| *** mgagne_ is now known as mgagne | 15:50 | |
| mgoddard | in inspector | 15:50 |
| TheJulia | mgoddard: There are a few things in storyboard for ironic-inspector, we had a few ideas about ways to improve interaction with the days of traits since properties -> trait based scheduling now | 15:50 |
| TheJulia | mgoddard: The HA model that is in place is slightly racey if I understand it correctly off hand, so in essence, yes | 15:51 |
| TheJulia | at least racey regarding the initial dhcp | 15:51 |
| TheJulia | which.. *shrug* | 15:51 |
| mgoddard | yeah, traits could be a useful thing to look at | 15:51 |
| jroll | there's also the config reload thing that needs doing | 15:51 |
| TheJulia | mgoddard: I don't think we have an explicit story for that, but it would go a long way | 15:51 |
| TheJulia | yes, config reload is also needed | 15:52 |
| *** stendulker_ has quit IRC | 15:52 | |
| TheJulia | Anyway, we have 8 minutes left of our scheduled time block, does anyone have anything else to discuss? | 15:52 |
| mgoddard | ok. All good ideas, I'll make a note. If anyone has any others, ping me. | 15:52 |
| rpioso | Talking of crash courses for someone new to OpenStack and ironic, ... Any suggestions? | 15:52 |
| rloo | by 'config reload' you mean the sighup stuff? that is being worked on. | 15:52 |
| jroll | rloo: ah, ok | 15:53 |
| TheJulia | rpioso: I need to start getting things together for the Summit project onboarding | 15:53 |
| TheJulia | but I was planning on doing that next week | 15:53 |
| rloo | jroll: that was related to my question earlier about flask & oslo.service :) | 15:53 |
| jroll | rloo: oh right. thanks :) | 15:54 |
| rloo | mgoddard: dtantsur|pto will most likely have lots of ideas! | 15:54 |
| mgoddard | rloo: that's true :) | 15:54 |
| jroll | running ironic in production is the quickest way to learn :P | 15:54 |
| rpioso | I've signed up to give a one hour introductory presentation to a few folks this Wednesday. | 15:54 |
| rloo | mgoddard: there's also he openstack priority wrt deploying with apache & wsgi in CI. | 15:54 |
| rpioso | jroll: lol | 15:54 |
| rloo | one hour... that might just cover openstack in general :) | 15:55 |
| TheJulia | mgoddard: rloo: I think inspector going into wsgi would require for us to split the service, and I kind of doubt that will ever happen | 15:56 |
| TheJulia | then again, I might be surprised :) | 15:56 |
| rpioso | Pointers to introductory content is probably all I'll have time to cover. | 15:57 |
| rpioso | Any favorites? | 15:57 |
| rloo | rpioso: the bible https://docs.openstack.org/ironic/latest/ :D | 15:58 |
| TheJulia | lol | 15:58 |
| rpioso | rloo: Amen :) | 15:58 |
| * jroll is hungry | 15:58 | |
| jiapei | lol | 15:58 |
| TheJulia | rpioso: perhaps maybe some presentations from summits might help, but they are going to be more than an hour | 15:58 |
| kaifeng | TheJulia: in the long term do you think inspector will take the same arch with ironic? | 15:58 |
| TheJulia | Anyway, seems like we're done meeting for today, we can discuss taking over the world in our normal channel discussion. | 15:59 |
| jiapei | gn | 15:59 |
| jroll | thanks TheJulia | 15:59 |
| TheJulia | kaifeng: I don't know, that is going to be dependent upon the individual contributors to inspector. The original idea was that it was required to make it HA-ready, but in reality splitting it was not needed. | 15:59 |
| *** namnh has quit IRC | 16:00 | |
| TheJulia | kaifeng: Any reason why you ask? | 16:00 |
| TheJulia | #endmeeting | 16:00 |
| *** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/developer/ironic/ | Bugs: https://bugs.launchpad.net/ironic" | 16:00 | |
| *** hjensas has quit IRC | 16:00 | |
| TheJulia | Thanks everyone! | 16:00 |
| openstack | Meeting ended Mon Apr 16 16:00:18 2018 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-16-15.00.html | 16:00 |
| kaifeng | till end meeting.. | 16:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-16-15.00.txt | 16:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2018/ironic.2018-04-16-15.00.log.html | 16:00 |
| TheJulia | kaifeng: huh? | 16:00 |
| * jroll notices he didn't give the most important update in the meeting, nor pre-meeting: https://i.imgur.com/Yswxtpy.jpg | 16:00 | |
| kaifeng | I recently have a look at inspector ha | 16:00 |
| *** marios has quit IRC | 16:00 | |
| TheJulia | jroll: AWWWW death by cute! | 16:01 |
| jroll | :D | 16:01 |
| kaifeng | due to an api worker patch of mine doesn't work | 16:01 |
| * jroll finds lunch now | 16:01 | |
| TheJulia | kaifeng: oh! to fix the concurrency issues I think you were encountering? | 16:01 |
| *** erolg has joined #openstack-ironic | 16:02 | |
| *** gcb has quit IRC | 16:02 | |
| kaifeng | yes, at first raise the api worker count can address an issue relates to nessus attack | 16:02 |
| *** jiapei has quit IRC | 16:02 | |
| kaifeng | but since the patch cannot pass ci, and dmitry think it might never work, due to current model of inspector | 16:02 |
| *** derekh has quit IRC | 16:03 | |
| kaifeng | so i was looking milan's patch of introducing rpc model | 16:03 |
| TheJulia | Then maybe we do need to change it | 16:03 |
| *** jistr|mtg is now known as jistr | 16:03 | |
| *** stendulker has quit IRC | 16:03 | |
| TheJulia | kaifeng: that seems reasonable path to begin taking. One issue is going to be review bandwidth but if you get the time to work on the patches I'm sure we can wrangle some people who are knowledgable in inspector to take a look at the patches. | 16:04 |
| TheJulia | Although if we're going to do it, sooner rather than later is better because changing the architecture is going to be a really difficult change to land later in the cycle | 16:05 |
| TheJulia | jroll: the recheck results came back, another job ran long. I suspect it might be time to ask in infra if we've got a slow cloud out there | 16:06 |
| *** jiapei has joined #openstack-ironic | 16:07 | |
| kaifeng | TheJulia: understood, it's quite amount of work, I guess it will never make in this cycle :) | 16:08 |
| TheJulia | kaifeng: never say never, but if we can merge before milestone 2, then we're in good shape to unbreak other places quickly | 16:08 |
| kaifeng | TheJulia: nothing is impossible, right ;) | 16:10 |
| milan | kaifeng, hey, any TL;DR of the issues you're hitting? | 16:11 |
| * milan wonders whether it's a conceptual design flaw | 16:12 | |
| *** hshiina has quit IRC | 16:12 | |
| kaifeng | milan: hey man | 16:12 |
| milan | hey :) | 16:12 |
| *** gyee has joined #openstack-ironic | 16:13 | |
| kaifeng | milan: i rebased your rpc patch | 16:13 |
| kaifeng | as you changed the sync call to async, many tests breaked | 16:13 |
| milan | aha, kaifeng, I got stuck at the POST@/continue/ endpoint that time | 16:15 |
| milan | so I never finished it | 16:15 |
| kaifeng | the functional test about some edge state is failing | 16:15 |
| milan | kaifeng, I wasn't sure whether an async POST@/continue/ made sense actually | 16:15 |
| milan | kaifeng, oh, can be | 16:16 |
| milan | you have a pastebin somewhere? | 16:17 |
| kaifeng | milan: oh, i want taking another approach, you may have some suggestions :) | 16:17 |
| kaifeng | milan: i'm off work at this time, you like to see some output result? | 16:18 |
| kaifeng | milan: here is the thing, I want to introduce messaging and sync rpc call for the first step, hope that will not break much thing | 16:19 |
| TheJulia | There are times and places for sync calls, perhaps it is one that it makes sense | 16:20 |
| kaifeng | milan: then separate api and worker to different service (require gate change) | 16:20 |
| milan | kaifeng, makes sense | 16:21 |
| kaifeng | milan: and last change sync to async one by one, I think it might can be done little by little, keeping things in control | 16:21 |
| *** trown is now known as trown|lunch | 16:21 | |
| milan | kaifeng, totally | 16:21 |
| * milan looks at his old patch | 16:22 | |
| *** lbragstad has quit IRC | 16:24 | |
| TheJulia | kaifeng: that sounds like a good plan | 16:25 |
| milan | kaifeng, https://review.openstack.org/#/c/497689/7/ironic_inspector/node_cache.py@107 might be the issue here | 16:28 |
| patchbot | patch 497689 - ironic-inspector - Introducing RPC | 16:28 |
| milan | IIRC the corner case scenarios in the unittests try to stress the state tracking of a NodeInfo object | 16:29 |
| milan | that part allows pickling a NodeInfo object so that a worker can pick it up | 16:30 |
| *** derekh has joined #openstack-ironic | 16:30 | |
| kaifeng | TheJulia: thanks, actually i'm not an expert on this, i'll try to do what i can do | 16:32 |
| milan | based on the version_id the worker would be able to "tell" the nodeinfo object state was modified (by some other worker) before the cast happened and the worker tried to perform the state update | 16:33 |
| *** etingof has quit IRC | 16:33 | |
| milan | so the tests might need updating but I dunno what is legitimate from the top of my head w/r workers | 16:34 |
| milan | might require some thinking O:-) | 16:34 |
| kaifeng | milan: yes, this is the part i'm not quite understand (haven't checked pickle protocol) | 16:35 |
| TheJulia | kaifeng: but it is how you can become an expert in it ;) | 16:35 |
| milan | TheJulia +1 | 16:35 |
| milan | kaifeng, :P :D | 16:36 |
| kaifeng | milan: it seems you are avoiding some race condition | 16:36 |
| milan | kaifeng, yes | 16:36 |
| milan | between a wsgi script asking a worker to work and the work happening | 16:37 |
| derekh | TheJulia: Sorry got disconnected earlier after you pinged my name, the inspector work I was going to be initially focused on is getting it to run in the tripleo overcloud, I doubt that will include full HA in this cycle | 16:37 |
| kaifeng | but why multiple worker can work on the same request, isn't it scheduled from rpc? | 16:37 |
| TheJulia | derekh: ahh! Okay! Thanks! | 16:38 |
| milan | kaifeng, depends | 16:38 |
| *** jrist has quit IRC | 16:39 | |
| milan | kaifeng, not same request but a request asked for multiple times | 16:39 |
| *** erolg has quit IRC | 16:39 | |
| milan | HA | 16:40 |
| kaifeng | milan, is it relates to ExplicitRPCAccessPolicy? I see ironic use DefaultRPCAccessPolicy | 16:40 |
| kaifeng | milan, Oh, i see, this is reentrance | 16:40 |
| milan | or a user hitting a submit button twice ;) | 16:40 |
| milan | yeah | 16:40 |
| milan | kaifeng, I have to run | 16:41 |
| milan | but feel free to ping me tomorrow | 16:41 |
| kaifeng | milan, sure, its nice to see you here | 16:41 |
| milan | same same :) | 16:41 |
| * milan out | 16:41 | |
| milan | ohhhh | 16:41 |
| milan | night Ironic! :) | 16:41 |
| *** milan has quit IRC | 16:42 | |
| kaifeng | hooray ironic, and goodnight :P | 16:42 |
| *** kaifeng has quit IRC | 16:42 | |
| *** fragatina has joined #openstack-ironic | 16:46 | |
| *** etingof has joined #openstack-ironic | 16:46 | |
| *** Nisha_Agarwal has quit IRC | 16:49 | |
| *** baha has quit IRC | 16:50 | |
| *** dougsz has quit IRC | 16:59 | |
| *** derekh has quit IRC | 17:01 | |
| *** jrist has joined #openstack-ironic | 17:01 | |
| *** e0ne has quit IRC | 17:03 | |
| *** mgoddard has quit IRC | 17:05 | |
| jroll | TheJulia: 3 hours feels like a super long time, even for a slow cloud | 17:09 |
| TheJulia | indeed | 17:12 |
| TheJulia | although the most recent was a different test, hour and fourty five minutes I think is the threshold where it can fail and it typically runs 1 hour 20-1hr 30 if memory serves | 17:12 |
| jroll | ah | 17:15 |
| jroll | TheJulia: different test indeed, I vote we merge your hack, as it's clear it fixes the problem | 17:16 |
| jroll | we'll always have intermittent CI things | 17:16 |
| TheJulia | yeah | 17:16 |
| rloo | do we have one test run that passes? | 17:17 |
| *** AlexeyAbashkin has quit IRC | 17:17 | |
| jroll | rloo: yes, here: https://review.openstack.org/#/c/560886/ | 17:17 |
| patchbot | patch 560886 - ironic - Test n-g-s fix | 17:17 |
| jroll | ironic-grenade-dsvm-multinode-multitenant was the break | 17:17 |
| rloo | jroll: i think that's good enuf. | 17:17 |
| jroll | :) | 17:17 |
| TheJulia | rloo: if you'll do the honors since it is a stable branch change | 17:18 |
| rloo | done! | 17:19 |
| TheJulia | rloo: thanks! | 17:19 |
| rloo | TheJulia: thank YOU! | 17:19 |
| TheJulia | now to press recheck on a whole bunch of patches a little later | 17:19 |
| rloo | although i really wish we knew why. cuz i am worried that this will bite us in the future. | 17:20 |
| jroll | sweet, thank you both | 17:20 |
| * jroll couldn't find any reasonable explanation, unfortunately :( | 17:21 | |
| *** jrist has quit IRC | 17:22 | |
| *** e0ne has joined #openstack-ironic | 17:22 | |
| rloo | jroll: you tried :) | 17:22 |
| TheJulia | it might and there is something super suspicious going on there, but... no real way to see it short of being in the node while the job is running and following ever action | 17:22 |
| jroll | which should mostly be happening with set -x anyway | 17:22 |
| TheJulia | For bash, yeah | 17:23 |
| TheJulia | no signs of uninstall or anything... it is bizzar | 17:23 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 17:23 |
| rloo | etingof: isn't Sphinx 1.6.5 ^^ | 17:24 |
| *** parlos has quit IRC | 17:25 | |
| etingof | rloo, hey, sorry for being slow! that probably changed while patch was on review... fixed. | 17:27 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 17:27 |
| TheJulia | interesting | 17:27 |
| TheJulia | I think 1.6.5 has been out for a while but *Shrug* | 17:27 |
| *** serlex has quit IRC | 17:28 | |
| TheJulia | So about a half an hour and we should be able to begin rechecking | 17:29 |
| rloo | thx etingof! | 17:30 |
| etingof | TheJulia, that's about lower constraint, it used to be 1.6.2 at least 10 days back | 17:32 |
| *** jrist has joined #openstack-ironic | 17:32 | |
| *** jrist has quit IRC | 17:32 | |
| *** jrist has joined #openstack-ironic | 17:32 | |
| * TheJulia shrugs | 17:32 | |
| *** r-daneel has quit IRC | 17:34 | |
| *** r-daneel has joined #openstack-ironic | 17:35 | |
| TheJulia | shoot, retry_limit on the docs job | 17:36 |
| *** trown|lunch is now known as trown | 17:36 | |
| *** mgoddard has joined #openstack-ironic | 17:41 | |
| *** lbragstad has joined #openstack-ironic | 17:43 | |
| jroll | TheJulia: did you see the ML about that? | 17:43 |
| TheJulia | no, I didn't | 17:44 |
| * jroll finds | 17:44 | |
| jroll | http://lists.openstack.org/pipermail/openstack-dev/2018-April/129402.html | 17:44 |
| TheJulia | ugh | 17:44 |
| TheJulia | yeah, just read it | 17:45 |
| * TheJulia rebuidls her tox env | 17:48 | |
| *** rbartal has joined #openstack-ironic | 17:57 | |
| *** cjloader has joined #openstack-ironic | 18:01 | |
| *** baha has joined #openstack-ironic | 18:04 | |
| *** cjloader has quit IRC | 18:05 | |
| *** cjloader has joined #openstack-ironic | 18:05 | |
| gyee | is there a way to increase ramdisk_size when using agent ilo driver and booting the iso via virtual media? something equivalent to pxe_append_params? | 18:07 |
| *** jiapei has quit IRC | 18:10 | |
| *** hjensas has joined #openstack-ironic | 18:11 | |
| *** hjensas has quit IRC | 18:11 | |
| *** hjensas has joined #openstack-ironic | 18:11 | |
| TheJulia | gyee: it would have to be burned into the agent ramdisk image to know to consume more ram | 18:12 |
| etingof | rloo, uh, oh: http://logs.openstack.org/19/560019/4/check/requirements-check/a8a71fe/job-output.txt.gz#_2018-04-16_17_31_11_822211 | 18:12 |
| rloo | etingof: sigh | 18:13 |
| * TheJulia glares at pip10 | 18:13 | |
| TheJulia | almost have a simple fix for tox_install | 18:14 |
| TheJulia | ""<-- that close... but pip is exiting 1 :\ | 18:14 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: update lower-constraints.txt to match *requirements.txt https://review.openstack.org/560019 | 18:14 |
| gyee | TheJulia, right now if the user image is big, we need a way to configuration a bigger ramdisk. | 18:14 |
| gyee | I am trying to avoid having to remaster the iso | 18:15 |
| TheJulia | gyee: No real way to avoid it since the bootloader is entirely contained in the iso image | 18:15 |
| gyee | I presume this patch may help? https://review.openstack.org/#/c/555257/5 | 18:16 |
| patchbot | patch 555257 - ironic - Add vmedia_append_params option to iLO driver | 18:16 |
| gyee | though I haven't tried it yet | 18:16 |
| TheJulia | gyee: no, ipa is already running by that point | 18:16 |
| gyee | ahh | 18:16 |
| openstackgerrit | Julia Kreger proposed openstack/networking-generic-switch stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561685 | 18:20 |
| *** jjohnson2 has quit IRC | 18:22 | |
| TheJulia | can I get a core reviewer to look at https://review.openstack.org/#/c/550587/ real quick | 18:23 |
| patchbot | patch 550587 - networking-baremetal - Avoid tox_install.sh | 18:23 |
| TheJulia | that will fix master | 18:23 |
| openstackgerrit | Julia Kreger proposed openstack/networking-baremetal stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561689 | 18:27 |
| *** rbartal has quit IRC | 18:33 | |
| openstackgerrit | Julia Kreger proposed openstack/networking-generic-switch stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561685 | 18:35 |
| openstackgerrit | Julia Kreger proposed openstack/networking-baremetal stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561689 | 18:35 |
| TheJulia | jroll: rloo: ^^^ _should_ work, but job logs will need to be checked | 18:36 |
| *** bfournie has quit IRC | 18:36 | |
| *** tesseract has quit IRC | 18:36 | |
| jroll | thanks TheJulia | 18:39 |
| *** harlowja has joined #openstack-ironic | 18:40 | |
| jroll | +A on that master patch | 18:40 |
| TheJulia | Thanks! | 18:41 |
| TheJulia | welp, looks like the tiny minimal change to tox_install works | 18:42 |
| TheJulia | and docs built | 18:43 |
| jroll | nice! | 18:44 |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:44 | |
| *** AlexeyAbashkin has quit IRC | 18:49 | |
| *** AlexeyAbashkin has joined #openstack-ironic | 18:49 | |
| TheJulia | yup, rloo up for some super quick stable changes? | 18:52 |
| rloo | TheJulia: in 30 min or so? busy right now. | 18:53 |
| rloo | TheJulia: i'll ping you. | 18:53 |
| TheJulia | rloo: sure, just hoping for today so we can get the gate unjammed :) | 18:53 |
| rloo | TheJulia: for sure today! | 18:53 |
| * TheJulia suddenly sees spaceballs and a jar of jam in the radar dish | 18:53 | |
| *** mgoddard has quit IRC | 18:53 | |
| *** AlexeyAbashkin has quit IRC | 18:59 | |
| *** openstackgerrit has quit IRC | 19:05 | |
| *** jaganathan has quit IRC | 19:20 | |
| *** jaganathan has joined #openstack-ironic | 19:21 | |
| rloo | hi TheJulia | 19:23 |
| rloo | which changes? | 19:24 |
| jroll | rloo: https://review.openstack.org/561685 https://review.openstack.org/561689 | 19:24 |
| patchbot | patch 561685 - networking-generic-switch (stable/queens) - Fix tox_install.sh for pip10 | 19:24 |
| patchbot | patch 561689 - networking-baremetal (stable/queens) - Fix tox_install.sh for pip10 | 19:24 |
| * jroll reviews just for fun | 19:24 | |
| rloo | TheJulia: done. are they only issues for those two projects? | 19:27 |
| TheJulia | rloo: as far as I'm aware | 19:28 |
| jroll | most (all?) of our projects shouldn't have tox_install.sh anymore, but many neutron plugins still rely on it | 19:28 |
| jroll | is how I understand things. I could be wrong | 19:28 |
| TheJulia | Yeah, just stable might need to be fixed | 19:29 |
| TheJulia | We'll see, it is a quick fix | 19:29 |
| jroll | ya | 19:29 |
| TheJulia | Those were the only two that I found locally on my machine *shrugs* | 19:29 |
| rloo | Hopefully that's all. we'll find out if not :) | 19:30 |
| *** openstackgerrit has joined #openstack-ironic | 19:36 | |
| openstackgerrit | Merged openstack/networking-generic-switch stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561685 | 19:36 |
| *** trown has quit IRC | 19:51 | |
| openstackgerrit | Merged openstack/networking-baremetal master: Avoid tox_install.sh https://review.openstack.org/550587 | 19:52 |
| openstackgerrit | Merged openstack/networking-baremetal stable/queens: Fix tox_install.sh for pip10 https://review.openstack.org/561689 | 19:52 |
| TheJulia | rloo: replied to the python-ironiccleint patch from ed that you commented on. I think that covers the basis and we can create all those things, more than anything agreement before we go down yet another path of thinking we have agreement before we really do. | 19:53 |
| rloo | TheJulia: yeah. We just need a story that lists out all that stuff and approve it. | 19:55 |
| rloo | TheJulia: I don't like process, but since we have process, i don't feel like it is fair to jump the queue just cuz we have the power. | 19:56 |
| TheJulia | I think if we get consensus on the patch, which if you agree in the patch, then I can create the story and just mark it approved | 19:56 |
| TheJulia | that is a good point about jumping the queue | 19:56 |
| rloo | TheJulia: i think the idea of adding a new parameter/API version, to individual requests is a good idea. I think I said that in my review. | 19:57 |
| TheJulia | I thought is that you didn't see anything wrong | 19:57 |
| rloo | TheJulia: and I suspect that the actual code is good too :) | 19:57 |
| * TheJulia needs to look again | 19:57 | |
| rloo | TheJulia: I just skimmed, and i have no test env right now. the -1 is for procedural stuff. | 19:57 |
| TheJulia | regardless, I can create the story and write the other method patches if there is agremeent that they cover most possible cases where it is needed since they are the majority of actions one would take | 19:58 |
| rloo | TheJulia: yeah, please create the story, list the methods where we want to add this new parameter. then we shoudl be good to go. i haven't put much thought into this but i think that's all that is needed. | 19:59 |
| TheJulia | I'm fairly sure ther ealready is, but we've also changed direction on this a few different times | 19:59 |
| TheJulia | I just asked ed if he would be okay with me updating his patch | 20:00 |
| rloo | TheJulia: i'm not sure how the CLI works with these new changes, but we can worry about that later. | 20:00 |
| rloo | TheJulia: ed? I thought it was fried. | 20:00 |
| TheJulia | these would all be under the hood changes | 20:00 |
| TheJulia | for python api users | 20:00 |
| TheJulia | ed fried | 20:00 |
| rloo | TheJulia: eric :) | 20:00 |
| *** e0ne has quit IRC | 20:00 | |
| TheJulia | oh, is it eric | 20:00 |
| TheJulia | i guess it is | 20:00 |
| * TheJulia might be fried | 20:00 | |
| rloo | ++ that's how i feel | 20:01 |
| * TheJulia needs to take a vacation | 20:01 | |
| rloo | ++ me too :D | 20:01 |
| *** r-daneel has quit IRC | 20:01 | |
| *** dprince has quit IRC | 20:01 | |
| *** trown has joined #openstack-ironic | 20:06 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Rescue mode tempest tests https://review.openstack.org/528699 | 20:08 |
| TheJulia | rajinir: Looks like dell CI is down hard | 20:10 |
| TheJulia | Ansible not found in the errors | 20:10 |
| TheJulia | looks like fujitsu as well :( | 20:11 |
| TheJulia | looks like a devstack change maybe | 20:11 |
| TheJulia | http://logs.openstack.org/97/561097/1/check/ironic-grenade-dsvm-multinode-multitenant/0c23a1e/logs/grenade.sh.txt.gz#_2018-04-16_09_06_58_088 \o/ | 20:15 |
| TheJulia | how many breaking things can we have in one day? | 20:15 |
| TheJulia | We're at two-ish | 20:15 |
| *** etingof has quit IRC | 20:18 | |
| *** r-daneel has joined #openstack-ironic | 20:19 | |
| jroll | lol. | 20:22 |
| jroll | that's a good one | 20:22 |
| *** mrunge has quit IRC | 20:23 | |
| jroll | TheJulia: you may have just missed this https://review.openstack.org/#/c/561471/ | 20:24 |
| patchbot | patch 561471 - openstack-dev/devstack (stable/queens) - Do not use pip 10 or higher (MERGED) | 20:24 |
| jroll | (or in other words, should be fixed by now) | 20:24 |
| TheJulia | *bangs head into desk* | 20:24 |
| *** mrunge has joined #openstack-ironic | 20:25 | |
| jroll | I was just thinking, there's no way that's going on without -infra or -qa channels having related scrollback | 20:25 |
| *** MattMan has quit IRC | 20:28 | |
| *** MattMan has joined #openstack-ironic | 20:28 | |
| rajinir | TheJulia: We had AC failure in the labs and we had to shutdown due to heat. Will be bringing them backup soon.. | 20:30 |
| *** etingof has joined #openstack-ironic | 20:30 | |
| TheJulia | rajinir: Ahh, thanks! | 20:31 |
| TheJulia | jroll: good point, additioanl streams can sometimes be difficult to keep track of | 20:31 |
| *** AlexeyAbashkin has joined #openstack-ironic | 20:41 | |
| *** AlexeyAbashkin has quit IRC | 20:45 | |
| *** jcoufal has quit IRC | 20:46 | |
| *** mtreinish has quit IRC | 20:51 | |
| *** harlowja has quit IRC | 20:55 | |
| *** mtreinish has joined #openstack-ironic | 20:57 | |
| *** fragatina has quit IRC | 20:59 | |
| *** sambetts is now known as sambetts|afk | 21:03 | |
| *** Goneri has quit IRC | 21:05 | |
| *** trown is now known as trown|outtypewww | 21:07 | |
| *** baha has quit IRC | 21:12 | |
| *** fragatina has joined #openstack-ironic | 21:22 | |
| *** devananda has joined #openstack-ironic | 21:36 | |
| *** rcernin has joined #openstack-ironic | 21:51 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic-specs master: Add cleanhold specification https://review.openstack.org/507910 | 22:15 |
| *** rpioso is now known as rpioso|afk | 22:18 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: OpenStack cloud driver added https://review.openstack.org/523879 | 22:31 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: added system boot mode support https://review.openstack.org/526101 | 22:32 |
| openstackgerrit | Merged openstack/networking-generic-switch stable/queens: Ensure pep8 is present before trying to proceed https://review.openstack.org/561358 | 22:34 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: documentation on simulators added https://review.openstack.org/523445 | 22:34 |
| *** Nisha_Agarwal has joined #openstack-ironic | 22:45 | |
| *** Nisha_Agarwal has quit IRC | 22:49 | |
| *** bfournie has joined #openstack-ironic | 22:58 | |
| *** cjloader has quit IRC | 23:08 | |
| *** cjloader has joined #openstack-ironic | 23:08 | |
| *** cjloader has quit IRC | 23:13 | |
| *** ianychoi has quit IRC | 23:18 | |
| *** ianychoi has joined #openstack-ironic | 23:25 | |
| *** r-daneel has quit IRC | 23:29 | |
| *** rajinir has quit IRC | 23:34 | |
| *** mrunge has quit IRC | 23:35 | |
| *** d0ugal has quit IRC | 23:36 | |
| *** mrunge has joined #openstack-ironic | 23:38 | |
| *** cjloader has joined #openstack-ironic | 23:39 | |
| *** jroll has quit IRC | 23:42 | |
| *** gyee has quit IRC | 23:43 | |
| *** cjloader has quit IRC | 23:44 | |
| *** cjloader has joined #openstack-ironic | 23:50 | |
| *** d0ugal has joined #openstack-ironic | 23:52 | |
| *** cjloader has quit IRC | 23:54 | |
| *** ianychoi has quit IRC | 23:55 | |
| *** cjloader has joined #openstack-ironic | 23:59 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!
