| *** rbudden has quit IRC | 00:01 | |
| *** joanna has quit IRC | 00:02 | |
| openstackgerrit | Merged openstack/ironic master: Address a shell syntax mistake https://review.openstack.org/443546 | 00:05 |
|---|---|---|
| openstackgerrit | Markos Chandras proposed openstack/bifrost master: roles: bifrost-ironic-install: Add PyMySQL dependency for ironic-dbsync https://review.openstack.org/444197 | 00:06 |
| *** dsneddon has quit IRC | 00:10 | |
| *** bfournie has joined #openstack-ironic | 00:20 | |
| *** bfournie has quit IRC | 00:21 | |
| *** bfournie has joined #openstack-ironic | 00:23 | |
| *** penick has quit IRC | 00:24 | |
| openstackgerrit | Markos Chandras proposed openstack/bifrost master: roles: bifrost-ironic-install: Add PyMySQL dependency for ironic-dbsync https://review.openstack.org/444197 | 00:25 |
| *** srart has quit IRC | 00:41 | |
| *** gcb has quit IRC | 00:42 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/445080 | 00:50 |
| *** phuongnh has joined #openstack-ironic | 00:56 | |
| *** Lambert has joined #openstack-ironic | 00:57 | |
| *** tuanluong has joined #openstack-ironic | 01:17 | |
| *** zhangjl has joined #openstack-ironic | 01:18 | |
| *** Sukhdev has quit IRC | 01:26 | |
| *** zhurong has joined #openstack-ironic | 01:30 | |
| *** dsneddon has joined #openstack-ironic | 01:36 | |
| *** tovin07 has joined #openstack-ironic | 01:42 | |
| openstackgerrit | Merged openstack/ironic master: Refactor waiters in our tempest plugin https://review.openstack.org/410684 | 01:53 |
| *** xuyongjie has joined #openstack-ironic | 02:12 | |
| *** rbudden has joined #openstack-ironic | 02:17 | |
| *** fragatina has quit IRC | 02:19 | |
| *** fragatina has joined #openstack-ironic | 02:20 | |
| *** fragatina has quit IRC | 02:21 | |
| *** fragatina has joined #openstack-ironic | 02:22 | |
| openstackgerrit | zack chen proposed openstack/ironic-python-agent master: Fix logs encoding in py3 https://review.openstack.org/438901 | 02:25 |
| *** fragatina has quit IRC | 02:26 | |
| *** fragatina has joined #openstack-ironic | 02:29 | |
| *** zzzeek has quit IRC | 02:30 | |
| *** jvgrant has quit IRC | 02:31 | |
| *** zzzeek has joined #openstack-ironic | 02:31 | |
| *** fragatina has quit IRC | 02:33 | |
| *** xuyongjie has quit IRC | 02:35 | |
| *** baoli has joined #openstack-ironic | 02:43 | |
| *** zackf has joined #openstack-ironic | 02:48 | |
| *** zackf has quit IRC | 02:52 | |
| *** zackf has joined #openstack-ironic | 02:53 | |
| *** dsneddon has quit IRC | 02:53 | |
| *** dsneddon has joined #openstack-ironic | 02:57 | |
| *** zackf has quit IRC | 02:57 | |
| *** crushil has joined #openstack-ironic | 03:06 | |
| *** crushil has quit IRC | 03:08 | |
| *** crushil has joined #openstack-ironic | 03:08 | |
| *** priteau has joined #openstack-ironic | 03:11 | |
| *** priteau has quit IRC | 03:15 | |
| *** jvgrant has joined #openstack-ironic | 03:28 | |
| *** lee4 has quit IRC | 03:38 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/445080 | 03:45 |
| *** sayantan_ has joined #openstack-ironic | 03:45 | |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Extends driver-list, driver-show supporting new hardware types https://review.openstack.org/419274 | 03:48 |
| *** nmathew has joined #openstack-ironic | 03:53 | |
| *** Sukhdev has joined #openstack-ironic | 03:56 | |
| *** fragatina has joined #openstack-ironic | 03:58 | |
| *** fragatina has quit IRC | 04:03 | |
| *** baoli has quit IRC | 04:04 | |
| *** jvgrant has quit IRC | 04:05 | |
| *** jvgrant has joined #openstack-ironic | 04:05 | |
| *** fragatina has joined #openstack-ironic | 04:08 | |
| *** fragatina has quit IRC | 04:08 | |
| *** fragatina has joined #openstack-ironic | 04:09 | |
| *** sasha2 has quit IRC | 04:10 | |
| *** yogi_ has joined #openstack-ironic | 04:27 | |
| *** Sukhdev has quit IRC | 04:28 | |
| *** crushil has quit IRC | 04:34 | |
| *** links has joined #openstack-ironic | 04:35 | |
| *** yogi has quit IRC | 04:36 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 04:44 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 04:45 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 04:46 |
| *** jvgrant has quit IRC | 04:49 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 04:53 |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Closes-Bug: 1672358 https://review.openstack.org/445293 | 04:56 |
| openstack | bug 1672358 in Ironic "Wong filename of port list request sample" [Low,In progress] https://launchpad.net/bugs/1672358 - Assigned to Wang KaiFeng (kaifeng) | 04:56 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector master: Fix inconsistent service naming in install guide https://review.openstack.org/437537 | 04:57 |
| *** udesale has joined #openstack-ironic | 04:59 | |
| *** vmud213 has joined #openstack-ironic | 05:04 | |
| *** vsaienko has quit IRC | 05:04 | |
| *** vmud213 has quit IRC | 05:08 | |
| openstackgerrit | M V P Nitesh proposed openstack/ironic master: Now displays maintenance and maintenance_reason https://review.openstack.org/433029 | 05:09 |
| *** gcb has joined #openstack-ironic | 05:17 | |
| *** rbudden has quit IRC | 05:19 | |
| openstackgerrit | Merged openstack/ironic master: Deprecate support for glance v1 https://review.openstack.org/441491 | 05:30 |
| *** fragatina has quit IRC | 05:33 | |
| *** vikrant has joined #openstack-ironic | 05:36 | |
| *** jvgrant has joined #openstack-ironic | 05:46 | |
| *** fragatina has joined #openstack-ironic | 05:49 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 05:50 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 05:52 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Wire in storage interface attach/detach operations https://review.openstack.org/406290 | 05:52 |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: iPXE template support for iSCSI https://review.openstack.org/413324 | 05:52 |
| *** sayantan_ has quit IRC | 06:16 | |
| *** dsneddon has quit IRC | 06:23 | |
| *** faizy has joined #openstack-ironic | 06:35 | |
| openstackgerrit | Hironori Shiina proposed openstack/ironic master: Add REST API for volume connector and volume target operation https://review.openstack.org/214586 | 06:36 |
| *** vsaienko has joined #openstack-ironic | 06:36 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Fix the exception message in tempest plugin https://review.openstack.org/444981 | 06:41 |
| *** vsaienko has quit IRC | 06:42 | |
| *** vsaienko has joined #openstack-ironic | 06:44 | |
| *** e0ne has joined #openstack-ironic | 06:51 | |
| *** mjura has joined #openstack-ironic | 06:52 | |
| *** fragatina has quit IRC | 06:52 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: Fixed wrap from taking negative values https://review.openstack.org/379436 | 06:52 |
| *** zhangjl has quit IRC | 07:00 | |
| *** zhangjl has joined #openstack-ironic | 07:01 | |
| *** ccamacho has quit IRC | 07:05 | |
| *** Guest5121 has quit IRC | 07:15 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add ironic standlaone test with ipmi dynamic driver https://review.openstack.org/439126 | 07:18 |
| *** tesseract has joined #openstack-ironic | 07:20 | |
| *** dnikishov has joined #openstack-ironic | 07:21 | |
| *** dnikishov is now known as Guest94930 | 07:22 | |
| pas-ha | morning ironic :) | 07:27 |
| *** e0ne has quit IRC | 07:28 | |
| openstackgerrit | M V P Nitesh proposed openstack/python-ironicclient master: OSC 'node list' recognizes all provision state https://review.openstack.org/431556 | 07:34 |
| *** jaganathan has joined #openstack-ironic | 07:45 | |
| *** pcaruana has joined #openstack-ironic | 07:50 | |
| *** priteau has joined #openstack-ironic | 07:51 | |
| *** mgoddard has joined #openstack-ironic | 07:59 | |
| *** zzzeek has quit IRC | 08:00 | |
| *** zzzeek has joined #openstack-ironic | 08:00 | |
| *** rbartal has joined #openstack-ironic | 08:02 | |
| *** alexpilo_ has joined #openstack-ironic | 08:03 | |
| *** alexpilotti has quit IRC | 08:03 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Fix some virtualenv-related places https://review.openstack.org/445350 | 08:08 |
| openstackgerrit | Dao Cong Tien proposed openstack/python-ironicclient master: Update OSC baremetal node set/unset supporting dynamic drivers https://review.openstack.org/431542 | 08:11 |
| *** jvgrant has quit IRC | 08:19 | |
| *** ccamacho has joined #openstack-ironic | 08:19 | |
| *** jaosorior has joined #openstack-ironic | 08:24 | |
| *** udesale has quit IRC | 08:24 | |
| *** athomas has joined #openstack-ironic | 08:27 | |
| *** pmannidi has quit IRC | 08:29 | |
| openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector master: Update broken alembic links in contributing guide https://review.openstack.org/445357 | 08:31 |
| *** aarefiev_afk is now known as aarefiev | 08:32 | |
| aarefiev | morning all! | 08:32 |
| Lambert | morning | 08:33 |
| openstackgerrit | yolanda.robla proposed openstack/bifrost stable/ocata: Updated from global requirements https://review.openstack.org/445359 | 08:33 |
| *** udesale has joined #openstack-ironic | 08:37 | |
| *** yolanda has quit IRC | 08:37 | |
| *** amoralej|off is now known as amoralej | 08:39 | |
| *** yolanda has joined #openstack-ironic | 08:42 | |
| *** ralonsoh has joined #openstack-ironic | 08:46 | |
| *** e0ne has joined #openstack-ironic | 08:50 | |
| *** udesale has quit IRC | 09:00 | |
| *** udesale has joined #openstack-ironic | 09:01 | |
| *** alineb has joined #openstack-ironic | 09:03 | |
| *** racedo has joined #openstack-ironic | 09:05 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost stable/ocata: playbooks: roles: {ironic,keystone}: Drop --force-reinstall https://review.openstack.org/445368 | 09:06 |
| *** snecklifter has joined #openstack-ironic | 09:09 | |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: Add Ironic standalone tests https://review.openstack.org/423556 | 09:09 |
| lucas-afk | morning all | 09:14 |
| *** lucas-afk is now known as lucasagomes | 09:15 | |
| *** jvgrant has joined #openstack-ironic | 09:16 | |
| *** mgoddard has quit IRC | 09:17 | |
| *** yee379 has quit IRC | 09:20 | |
| *** yee379 has joined #openstack-ironic | 09:20 | |
| yolanda | hi, can i get reviews on https://review.openstack.org/#/c/445368/ (bifrost) ? that seems to unlock the ocata gate | 09:21 |
| patchbot | patch 445368 - bifrost (stable/ocata) - playbooks: roles: {ironic,keystone}: Drop --force-... | 09:21 |
| *** amarao has joined #openstack-ironic | 09:28 | |
| *** stendulker has joined #openstack-ironic | 09:30 | |
| *** alexpilotti has joined #openstack-ironic | 09:36 | |
| *** alexpilo_ has quit IRC | 09:37 | |
| *** derekh has joined #openstack-ironic | 09:42 | |
| vdrok | good morning ironic! | 09:50 |
| openstackgerrit | Merged openstack/ironic master: Fix the exception message in tempest plugin https://review.openstack.org/444981 | 09:52 |
| stendulker | Good morning vdrok ! | 09:54 |
| *** zhurong has quit IRC | 09:54 | |
| vdrok | good morning stendulker :) | 09:55 |
| openstackgerrit | Merged openstack/ironic master: Speed up test_touch_conductor_deadlock() https://review.openstack.org/445182 | 09:55 |
| *** dtantsur|afk is now known as dtantsur | 09:59 | |
| dtantsur | morning ironic! | 09:59 |
| lucasagomes | vdrok, dtantsur stendulker morning o/ | 10:00 |
| vdrok | morning dtantsur and lucasagomes ! | 10:00 |
| stendulker | Good morning lucasagomes, dtantsur | 10:00 |
| lucasagomes | btw folks, I was thinking what to do with the redfish-simulator from sushy. Currently it's just a script living under sushy/tools/, but I don't think it's ideal to just clone the repo and run it when setting up DevStack | 10:01 |
| lucasagomes | so ideally, it would be good to have a command which we can turn into a service or something | 10:01 |
| lucasagomes | is there a way to do it with pbr or something ? | 10:01 |
| dtantsur | lucasagomes, pbr can create simple scripts for you, if that's your question | 10:02 |
| lucasagomes | dtantsur, yeah but there's a gotcha | 10:02 |
| lucasagomes | cause the simulator has it's own dependencies | 10:02 |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests https://review.openstack.org/443628 | 10:02 |
| lucasagomes | (flask, libvirt and jinja2) | 10:02 |
| lucasagomes | which is not a dependency for sushy itself (which only depends on requests and six) | 10:03 |
| dtantsur | lucasagomes, alternatively, it can be another repo | 10:03 |
| lucasagomes | so I wanted to keep it separated | 10:03 |
| dtantsur | openstack/sushy-simulator | 10:03 |
| dtantsur | or whatever | 10:03 |
| lucasagomes | dtantsur, yeah, that was what I was thinking of, but I wanted to check if there was a better way | 10:03 |
| lucasagomes | ideally, if it could live int he same repo it would fantastic but I think it's not possible :-/ (still looking tho) | 10:04 |
| lucasagomes | would be* | 10:04 |
| *** MattMan has quit IRC | 10:05 | |
| *** MattMan has joined #openstack-ironic | 10:06 | |
| *** lennyb has quit IRC | 10:06 | |
| lucasagomes | just found a bug with the simulator :-( it doesn't survive restarts to libvirt service since it tries to keep the connection open and the socket is rebuilt | 10:11 |
| * lucasagomes opens a bg | 10:11 | |
| lucasagomes | bug* | 10:11 |
| *** gcb has quit IRC | 10:11 | |
| *** lennyb has joined #openstack-ironic | 10:11 | |
| dtantsur | well, another option is to have simulator-requirements.txt and install it. downside is that it won't use global-requirements | 10:15 |
| lucasagomes | dtantsur, yeah it has a requirement inside tools/... But can I make it "optional" when people are installing stuff ? | 10:16 |
| dtantsur | well, if we only install it in devstack... | 10:17 |
| lucasagomes | hah yeah well :D | 10:17 |
| dtantsur | but as I said, then we can't use global-requirments sync | 10:17 |
| lucasagomes | yeah sounds like a receipt for problems | 10:17 |
| lucasagomes | recipe * | 10:18 |
| *** zhangjl has left #openstack-ironic | 10:20 | |
| *** phuongnh has quit IRC | 10:26 | |
| *** jvgrant has quit IRC | 10:32 | |
| *** athomas has quit IRC | 10:34 | |
| *** vills has joined #openstack-ironic | 10:38 | |
| *** jkilpatr has quit IRC | 10:39 | |
| *** Lambert has quit IRC | 10:39 | |
| *** mvpnitesh has joined #openstack-ironic | 10:44 | |
| *** mgoddard has joined #openstack-ironic | 10:56 | |
| dtantsur | JayF, I think you can remove your -2 from https://review.openstack.org/#/c/396400/ (or mention that you need an RFE now, but the driver is fine) | 10:57 |
| patchbot | patch 396400 - ironic - Make snmp udp transport settings configurable | 10:57 |
| *** athomas has joined #openstack-ironic | 10:59 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Closes-Bug: 1672358 https://review.openstack.org/445293 | 11:07 |
| openstack | bug 1672358 in Ironic "Wong filename of port list request sample" [Low,Incomplete] https://launchpad.net/bugs/1672358 - Assigned to Wang KaiFeng (kaifeng) | 11:07 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic master: Updated from global requirements https://review.openstack.org/445080 | 11:19 |
| *** jkilpatr has joined #openstack-ironic | 11:19 | |
| *** vills_ has joined #openstack-ironic | 11:21 | |
| *** vills has quit IRC | 11:21 | |
| *** udesale__ has joined #openstack-ironic | 11:24 | |
| *** udesale has quit IRC | 11:24 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Fix logs encoding in py3 https://review.openstack.org/438901 | 11:25 |
| *** jkilpatr has quit IRC | 11:27 | |
| *** jvgrant has joined #openstack-ironic | 11:29 | |
| *** links has quit IRC | 11:36 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add hardware type for HPE ProLiant servers based on iLO 4 https://review.openstack.org/439404 | 11:36 |
| *** makowals_ has joined #openstack-ironic | 11:39 | |
| *** jkilpatr has joined #openstack-ironic | 11:40 | |
| *** faizy has quit IRC | 11:40 | |
| *** stendulker has quit IRC | 11:41 | |
| *** makowals has quit IRC | 11:43 | |
| openstackgerrit | Wang KaiFeng proposed openstack/ironic master: Fix typos of filename in api-ref https://review.openstack.org/445293 | 11:44 |
| *** makowals_ has quit IRC | 11:44 | |
| openstackgerrit | Merged openstack/ironic master: Cleanup hung iscsi session https://review.openstack.org/388092 | 11:44 |
| *** nmathew has quit IRC | 11:46 | |
| *** links has joined #openstack-ironic | 11:48 | |
| *** aparnav has joined #openstack-ironic | 11:49 | |
| *** mvpnitesh has quit IRC | 11:53 | |
| *** mvpnitesh has joined #openstack-ironic | 11:53 | |
| *** jaganathan has quit IRC | 11:56 | |
| lucasagomes | dtantsur, ty for the review in the rf driver btw. I think there are some misundertands about that cache, I hope I clarified in the comments in the patch. But, for the next revision I will try to comment in the code as well | 11:59 |
| lucasagomes | to make it more explicit | 11:59 |
| dtantsur | lucasagomes, "the relation between connections to systems is 1:n" in the best case and 1:1 in the worst | 12:00 |
| dtantsur | the former is valid for RSD, the latter for HPE | 12:00 |
| lucasagomes | dtantsur, true | 12:00 |
| lucasagomes | the worst is 1:! | 12:00 |
| lucasagomes | 1:1* | 12:00 |
| lucasagomes | but still it's not concerning I think, because the load is divided between conductors in Ironic | 12:01 |
| lucasagomes | dtantsur, that said, I can very well remove the caching but I think it's a mistake | 12:02 |
| lucasagomes | we will double the # of requests if we do that | 12:03 |
| dtantsur | dunno, we don't cache IPMI connections.. ok, let's leave it, unless somebody else objecdts | 12:04 |
| dtantsur | lucasagomes, found another issue, see inline | 12:04 |
| *** tuanluong has quit IRC | 12:05 | |
| lucasagomes | dtantsur, ++ re log | 12:05 |
| dtantsur | lucasagomes, also http://redfish.dmtf.org/schemas/DSP0266_1.1.html#uris does not say it has to be /Systems | 12:05 |
| dtantsur | quite the opposite: A URI shall be treated by the client as opaque, and thus should not be attempted to be understood or deconstructed by the client outside of applying standard reference resolution rules as defined in clause 5, Reference Resolution, of RFC3986. | 12:05 |
| dtantsur | we're in violation of this clause :( | 12:06 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Adds use wsgi without port https://review.openstack.org/444337 | 12:06 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 12:07 |
| lucasagomes | dtantsur, yeah the idea was to point to that spec that uses /Systems but indeed, the spec is very loose | 12:08 |
| lucasagomes | I think the best we can do is rely on the library to translate things for us under the hood | 12:08 |
| lucasagomes | and for the driver, just for the sake of conformity we use /Systems to parse that (or split in two parameters as proposed) | 12:09 |
| lucasagomes | >.< redfish is though, it should be more opinionated on thing | 12:09 |
| lucasagomes | things* | 12:09 |
| dtantsur | oh true | 12:09 |
| dtantsur | I think we should get back to a proposal on the spec to use redfish_root_uri and redfish_system_id :( | 12:10 |
| dtantsur | this logic is dangerous and should not be relied upon | 12:10 |
| * lucasagomes is not opposed to do it | 12:10 | |
| lucasagomes | honest, I think it's more flexible | 12:10 |
| lucasagomes | dtantsur, wants me to put a patch and we can discuss it there ? | 12:10 |
| dtantsur | lucasagomes, yes please. with our founding about the redfish spec, and that only root URI is meaningful | 12:11 |
| *** jcoufal has joined #openstack-ironic | 12:11 | |
| lucasagomes | ack will do | 12:11 |
| *** mgould|afk is now known as moguld | 12:14 | |
| *** venki has quit IRC | 12:14 | |
| moguld | lucasagomes: that's the HATEOAS way :-/ | 12:14 |
| *** moguld is now known as mgould | 12:15 | |
| mgould | ffs | 12:15 |
| lucasagomes | mgould, yeah, just discover everything | 12:15 |
| lucasagomes | which is fine in terms of how sushy is done I think, cause we always start with a root URI there | 12:15 |
| *** trown|outtypewww is now known as trown | 12:15 | |
| dtantsur | HATE, yeah :D | 12:15 |
| lucasagomes | but I pass the redfish version along and it can use that to build the objects properly | 12:15 |
| dtantsur | lucasagomes, not really, you assume /System/ ther e:( | 12:15 |
| dtantsur | but we can fix it later | 12:16 |
| lucasagomes | dtantsur, well yeah, but it's a simple fix | 12:16 |
| lucasagomes | as long as we can identify those differences from the base resource onwards I think we are good | 12:16 |
| dtantsur | yep. if we split the url into root and id parts, the fix will be fully contained in sushy | 12:16 |
| lucasagomes | my fear is different systems using the same redfish version but doing things completely different | 12:17 |
| lucasagomes | that would be a problem | 12:17 |
| *** [1]cdearborn has joined #openstack-ironic | 12:17 | |
| lucasagomes | dtantsur, ++ | 12:17 |
| *** hshiina has joined #openstack-ironic | 12:18 | |
| mgould | can we store the whole URI as the node's identifier? | 12:18 |
| openstackgerrit | Merged openstack/ironic-inspector master: Updated from global requirements https://review.openstack.org/445081 | 12:19 |
| dtantsur | mgould, that's what we do, but then we need the root URL to connect to redfish itself | 12:22 |
| dtantsur | which is something we can change as well | 12:22 |
| lucasagomes | mgould, that's what how it's currently done | 12:22 |
| mgould | lucasagomes: OK, cool | 12:24 |
| *** tommylikehu has quit IRC | 12:24 | |
| *** faizy has joined #openstack-ironic | 12:25 | |
| mgould | so for each node we store {id: 'http://my-redfish-server/System/12345', 'root_uri': 'http://my-redfish-server/'} ? | 12:25 |
| lucasagomes | mgould, right now ? No, right now we only store redfish_uri=http://.../redfish/v1/Systems/12345 | 12:26 |
| mgould | aaah, and the question is whether we can reliably deconstruct redfish_uri to get the root URI? | 12:27 |
| mgould | yeah, let's just store the root URI, it's correct and won't take up much space | 12:27 |
| lucasagomes | but, if we split it as dmitry is talking about we will end up with: redfish_root_uri=http://.../redfish/v1 AND redfish_system_id=12345 | 12:27 |
| lucasagomes | mgould, correct | 12:27 |
| lucasagomes | it's just how we parse it | 12:27 |
| mgould | right | 12:28 |
| mgould | I suggest not parsing anything and just storing full URIs | 12:28 |
| mgould | which AFAICT is what the spec wants you to do | 12:28 |
| lucasagomes | mgould, right, full URI to the root resource you mean ? | 12:28 |
| mgould | yep | 12:30 |
| mgould | full URIs for both system and BMC | 12:30 |
| dtantsur | mgould, "the question is whether we can reliably deconstruct redfish_uri to get the root URI" <- no, per redfish spec | 12:30 |
| *** mjturek has joined #openstack-ironic | 12:30 | |
| mgould | yeah, that's what I thought | 12:31 |
| dtantsur | full URL for the System is also quite useless: we need an ID | 12:31 |
| *** venki has joined #openstack-ironic | 12:31 | |
| mgould | what are the constraints on the ID? | 12:31 |
| dtantsur | or rather: it's useful, but not in the way sushy and python-redfish both designed | 12:31 |
| dtantsur | they're designed to go from root and build URLs from it | 12:31 |
| mgould | bah | 12:32 |
| *** makowals has joined #openstack-ironic | 12:32 | |
| dtantsur | we could rewrite (nearly completely) sushy to be non-so-tightly integrated. so that there is no central Connector object | 12:32 |
| * mgould is sure he pointed this out as a potential problem in a review :-( | 12:32 | |
| *** aparnav has quit IRC | 12:32 | |
| dtantsur | and e.g. we use it like sushy.System( 'http://my-redfish-server/System/12345') | 12:32 |
| mgould | huh, I thought python-redfish treated URIs as opaque, hence the crawl-the-world-at-startup design | 12:33 |
| dtantsur | which may be another approach, lucasagomes ^^, but then no caching for sure | 12:33 |
| dtantsur | mgould, right, that's what I'm saying | 12:33 |
| mgould | oh, I see | 12:33 |
| vdrok | jroll: when you're around, dtantsur, I've done a couple of integration tests for the resource classes - https://review.openstack.org/443628. the question is, it seems like nova does not intend to have the placement client in tempest. is it ok to have some minimal client for placement in out tempest plugin? | 12:34 |
| patchbot | patch 443628 - ironic - [WIP] Add resource classes integration tests | 12:34 |
| vdrok | s/out/our | 12:34 |
| lucasagomes | dtantsur, yeah that's possible too, perhaps be able to pass the full URL and build a resource object from it | 12:34 |
| dtantsur | vdrok, is it their principal position or just temporary situation? | 12:34 |
| *** faizy has quit IRC | 12:34 | |
| dtantsur | lucasagomes, actually it may be even better for an ironic-specific library.. | 12:35 |
| dtantsur | lucasagomes, but it may reduce its general usefulness. so dunno really | 12:35 |
| lucasagomes | dtantsur, yeah, my idea with sushy was that we connect via the root resource and that should contain the information we need to build the resources after that properly | 12:35 |
| lucasagomes | and that would give us a good usage feel | 12:36 |
| dtantsur | lucasagomes, this is how redfish is designed to work, yes :) | 12:36 |
| vdrok | dtantsur: from what I understood it's not temporary. but will ask once again in nova channel today | 12:36 |
| dtantsur | then we need root_uri and system_id | 12:36 |
| lucasagomes | dtantsur, yeah, let's do that too ? Honestly I think it's better | 12:36 |
| dtantsur | vdrok, please do. I'd like to know their reasoning. but there is nothing preventing us from writing any client - the API is supposed to be stable after all ;) | 12:36 |
| vdrok | dtantsur: I mean, if it is, is it OK to keep it in our tree? the only things we need are get requests to 4-5 placement endpoints | 12:36 |
| jroll | vdrok: I'm fine with it, but yeah maybe ask cdent/jaypipes what they think | 12:36 |
| dtantsur | lucasagomes, ++, I was mostly addressing mgould's comments | 12:36 |
| vdrok | yup, will do | 12:36 |
| vdrok | morning jroll | 12:37 |
| lucasagomes | cause passing the full resource URL is hmm it can be a mistake I think, based on what I've seem so far. For example, in System/12345 we don't have the redfish version so it will be a lot of try and error | 12:37 |
| jroll | vdrok: though it seems like we're testing the placement service in our tempest plugin, mostly, which is odd :) | 12:37 |
| lucasagomes | dtantsur, ah cool, ok | 12:37 |
| dtantsur | jroll, morning | 12:37 |
| jroll | vdrok: and good morning :) | 12:37 |
| jroll | morning dtantsur, lucasagomes, mgould, everyone else \o/ | 12:37 |
| lucasagomes | jroll, good morning o/ | 12:37 |
| dtantsur | lucasagomes, right. let's follow the standard, whether we like it or not | 12:37 |
| vdrok | jroll: well, kind of :) | 12:37 |
| * dtantsur gets food | 12:37 | |
| TheJulia | Good morning | 12:38 |
| * TheJulia cleans up ironic-ui bugs | 12:38 | |
| jroll | vdrok: fwiw, we should have functional tests in nova's tree for the same thing, the difference is we'd mock the ironic service | 12:38 |
| vdrok | jroll: yup, they have the functional tests already | 12:38 |
| vdrok | jroll: https://review.openstack.org/#/c/441544/7/nova/tests/functional/compute/test_resource_tracker.py | 12:39 |
| patchbot | patch 441544 - nova - virt: implement get_inventory() for Ironic | 12:39 |
| jroll | vdrok: ah, cool, that's on my list today. so the question is if also testing this in our tree is valuable :) | 12:39 |
| lucasagomes | dtantsur, cool, I will put a patch up for the spec | 12:39 |
| lucasagomes | I think that's the best (or more flexible) approach | 12:40 |
| *** jaganathan has joined #openstack-ironic | 12:40 | |
| *** gema has joined #openstack-ironic | 12:44 | |
| gema | hey, if I want to install ironic to manage the lab hardware, what it the right documentation to look at? will be doing it on debian but I am planning to build from source | 12:45 |
| *** rbudden has joined #openstack-ironic | 12:46 | |
| mjturek | gema: https://docs.openstack.org/project-install-guide/baremetal/ocata/ is probably where you want to look. I would guess that when you hit 'Install and configure Bare Metal service' you'd use the Ubuntu section | 12:47 |
| gema | mjturek: great, thanks | 12:48 |
| *** makowals_ has joined #openstack-ironic | 12:48 | |
| dtantsur | morning TheJulia | 12:48 |
| *** dprince has joined #openstack-ironic | 12:49 | |
| *** makowals has quit IRC | 12:50 | |
| *** links has quit IRC | 12:52 | |
| *** baoli has joined #openstack-ironic | 12:52 | |
| *** links has joined #openstack-ironic | 12:53 | |
| *** amoralej is now known as amoralej|lunch | 12:53 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs master: Follow-up patch for the Redfish spec https://review.openstack.org/445478 | 12:53 |
| lucasagomes | dtantsur, ^ | 12:54 |
| dtantsur | thnx | 12:54 |
| lucasagomes | hope the commit message is self explanatory | 12:54 |
| *** amotoki has joined #openstack-ironic | 12:54 | |
| * lucasagomes goes grab some food | 12:55 | |
| *** lucasagomes is now known as lucas-hungry | 12:55 | |
| *** Goneri has joined #openstack-ironic | 12:55 | |
| *** baoli has quit IRC | 12:55 | |
| *** baoli has joined #openstack-ironic | 12:56 | |
| *** vikrant has quit IRC | 12:56 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/422149 | 12:57 |
| dtantsur | jroll, dunno if you reviewed the master version, but you have a chance to review the backport: https://review.openstack.org/444276 :) | 12:57 |
| patchbot | patch 444276 - ironic (stable/ocata) - Fix updating node.driver to classic | 12:57 |
| jlvillal | vsaienk0, Do we want to merge: https://review.openstack.org/#/c/439958/ ? | 12:58 |
| patchbot | patch 439958 - networking-generic-switch - Do not 'enable_plugin networking-generic-switch' i... | 12:58 |
| *** vikrant has joined #openstack-ironic | 12:58 | |
| vsaienk0 | jlvillal: we need to enable it via project-config first | 12:59 |
| jlvillal | vsaienk0, I thought it was... | 13:00 |
| jlvillal | vsaienk0, In the multi-tenant builder | 13:00 |
| *** baoli has quit IRC | 13:00 | |
| jlvillal | vsaienk0, https://github.com/openstack-infra/project-config/blob/master/jenkins/jobs/ironic.yaml#L139-L147 | 13:01 |
| *** baoli has joined #openstack-ironic | 13:02 | |
| jlvillal | vsaienk0, Or do you mean in the networking-generic-switch job? | 13:02 |
| jlvillal | I haven't looked at that | 13:02 |
| vsaienk0 | jlvillal: no, I mean grenade multitenant, yeah it is already there just checked logs http://logs.openstack.org/56/423556/84/experimental/gate-grenade-dsvm-ironic-multinode-multitenant-ubuntu-xenial-nv/618b343/logs/subnode-2/old/local_conf.txt.gz | 13:03 |
| jlvillal | vsaienk0, Thanks | 13:03 |
| *** hoangcx has joined #openstack-ironic | 13:03 | |
| *** rloo has joined #openstack-ironic | 13:03 | |
| *** chlong has joined #openstack-ironic | 13:05 | |
| jroll | dtantsur: done | 13:12 |
| *** mvpnitesh has quit IRC | 13:15 | |
| *** sasha2 has joined #openstack-ironic | 13:15 | |
| lucas-hungry | dtantsur, https://review.openstack.org/#/c/445478/ (pep8 error) | 13:19 |
| patchbot | patch 445478 - ironic-specs - Follow-up patch for the Redfish spec | 13:19 |
| * lucas-hungry goes back to food | 13:19 | |
| *** krtaylor has joined #openstack-ironic | 13:19 | |
| *** crushil has joined #openstack-ironic | 13:21 | |
| *** trown is now known as trown|brb | 13:21 | |
| *** faizy has joined #openstack-ironic | 13:21 | |
| *** trown|brb is now known as trown | 13:27 | |
| -openstackstatus- NOTICE: Gerrit is going to be restarted due to performance problems | 13:39 | |
| *** ChanServ changes topic to "Gerrit is going to be restarted due to performance problems" | 13:39 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:45 | |
| *** ChanServ changes topic to "Gerrit has been successfully restarted" | 13:45 | |
| *** ChanServ 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" | 13:51 | |
| -openstackstatus- NOTICE: Gerrit has been successfully restarted | 13:52 | |
| *** lucas-hungry is now known as lucasagomes | 13:53 | |
| *** jvgrant has quit IRC | 13:54 | |
| *** zackf has joined #openstack-ironic | 13:55 | |
| *** awiddersheim has quit IRC | 13:57 | |
| *** links has quit IRC | 13:57 | |
| *** awiddersheim has joined #openstack-ironic | 13:57 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/newton: WIP/DNM NEWTON: Testing multi-tenant multi-node grenade job https://review.openstack.org/443978 | 13:58 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic stable/ocata: WIP/DNM: OCATA: Testing multi-tenant multi-node grenade job https://review.openstack.org/443413 | 13:59 |
| *** zackf has quit IRC | 14:00 | |
| *** jaganathan has quit IRC | 14:01 | |
| *** vsaienko has quit IRC | 14:04 | |
| *** vsaienko has joined #openstack-ironic | 14:06 | |
| *** amoralej|lunch is now known as amoralej | 14:08 | |
| dtantsur | jroll, thanks! you may also want to read the comments on https://review.openstack.org/#/c/408151/ | 14:14 |
| patchbot | patch 408151 - nova-specs - Ironic: add support for instance RAID configuration | 14:14 |
| *** vsaienko has quit IRC | 14:18 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost stable/ocata: playbooks: roles: {ironic,keystone}: Drop --force-reinstall https://review.openstack.org/445368 | 14:19 |
| jroll | dtantsur: uh oh :) | 14:19 |
| TheJulia | yolanda: thank you! | 14:20 |
| *** vsaienko has joined #openstack-ironic | 14:22 | |
| openstackgerrit | yolanda.robla proposed openstack/bifrost stable/ocata: playbooks: roles: {ironic,keystone}: Drop --force-reinstall https://review.openstack.org/445368 | 14:22 |
| TheJulia | cinerama: NobodyCam ^^^ | 14:25 |
| NobodyCam | Good morning, I'll take a look at the office | 14:26 |
| dtantsur | morning NobodyCam, aren't you slightly early? :) | 14:26 |
| dtantsur | jroll, anyway, if the discussion goes where it is going now, we may need to fall back to traits approach | 14:27 |
| NobodyCam | Lol a little :) just having the first coffee if the morning :p | 14:27 |
| dtantsur | which probably implies creating "RAID templates" CRUD in ironic, and then just use traits.. | 14:27 |
| *** rama_y_ has joined #openstack-ironic | 14:28 | |
| JayF | dtantsur: how can I remove my -2? There is no virtualpdu CI yet. | 14:29 |
| jroll | dtantsur: jaypipes had a way to do it, without changing the BDM thing, I think | 14:29 |
| jroll | JayF: gate-tempest-dsvm-ironic-ipa-wholedisk-bios-pxe_snmp-tinyipa-ubuntu-xenial-nv | 14:30 |
| JayF | jroll: oh, yay! | 14:30 |
| JayF | jroll: so that's normal, not experimental? | 14:30 |
| *** vsaienko has quit IRC | 14:30 | |
| jroll | JayF: yeah, it's in the check queue | 14:30 |
| JayF | aweeesome | 14:30 |
| jroll | that said it's failing here :) | 14:30 |
| dtantsur | it was broken for some time, maybe that patch hit that period.. or is really broken :) | 14:31 |
| JayF | well I did +0 recheck | 14:31 |
| JayF | so we'll see how that patch fares | 14:31 |
| *** vsaienko has joined #openstack-ironic | 14:35 | |
| *** vikrant has quit IRC | 14:42 | |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: Add /baremetal path instead of port 6385 https://review.openstack.org/444337 | 14:42 |
| openstackgerrit | Oksana Voshchana proposed openstack/ironic master: DNM: TEST ironic with wsgi https://review.openstack.org/437299 | 14:43 |
| *** causten has joined #openstack-ironic | 14:48 | |
| JayF | jroll: dtantsur: https://review.openstack.org/#/c/445061/ does not appear to have merged into master, that means it's not in compliance with stable policy. But you both know that, and still put up the patch/+2' | 14:48 |
| patchbot | patch 445061 - ironic (stable/ocata) - Set tempest's max microversion to 1.31 | 14:48 |
| JayF | jroll: dtantsur: basically what am I missing | 14:48 |
| jroll | JayF: click the link in commit message | 14:49 |
| JayF | So if we had done it right: | 14:49 |
| JayF | commit to master to limit tempest microversion | 14:49 |
| JayF | cut branch | 14:49 |
| jroll | JayF: or think about what the implications are of capping the tempest microversion on master :) | 14:49 |
| JayF | commit to master to unlimit? | 14:49 |
| jroll | correct | 14:49 |
| JayF | That seems like a dumber process than what we appear to have ended up doing. | 14:49 |
| jroll | "dumb" is an interesting word | 14:50 |
| JayF | I mean, just like ... seems like compliance for compliance sake | 14:50 |
| jroll | oh | 14:50 |
| JayF | what's the virtue of putting it in master, cutting the branch, then reverting it | 14:50 |
| *** vsaienko has quit IRC | 14:50 | |
| JayF | vs what we're doing today (just a single commit post-cut) | 14:50 |
| JayF | it just seems like what I'd do if I were trying to comply with the letter of the policy | 14:50 |
| jroll | oh right, I didn't think about the revert thing, heh | 14:50 |
| jroll | it wasn't about compliance, I just didn't think through it | 14:51 |
| *** makowals_ has quit IRC | 14:51 | |
| *** jvgrant has joined #openstack-ironic | 14:51 | |
| JayF | "dumb" was a bad choice of word because of the connotations, but I think you get what I meant now? | 14:51 |
| jroll | yes | 14:51 |
| jroll | we should fix those docs | 14:52 |
| mgould | "over-complex"? | 14:52 |
| * jroll adds to todo list | 14:52 | |
| JayF | although something I was curious about is if we keep that microversion limit in tree | 14:52 |
| JayF | why can't you just keep the limit in master | 14:52 |
| JayF | and make it a step when bumping microversion to bump that limit, too? | 14:52 |
| *** Syed__ has joined #openstack-ironic | 14:52 | |
| jroll | JayF: less churn I guess, and if we forget we end up with untested code paths | 14:52 |
| *** athomas has quit IRC | 14:53 | |
| *** vsaienko has joined #openstack-ironic | 14:53 | |
| JayF | sounds like the answer is "meh" which is fine with me | 14:53 |
| JayF | lol | 14:53 |
| jroll | :P | 14:53 |
| JayF | looking for deeper meaning | 14:53 |
| JayF | and finding nothing but algae on the ocean floor | 14:53 |
| *** erol has joined #openstack-ironic | 14:55 | |
| erol | hi | 14:55 |
| mgould | hi erol | 14:55 |
| erol | I am getting error when barametal node provisioning from /usr/bin/provision --data_driver ironic --config-file .... proccess | 14:56 |
| erol | the error is Unexpected error Exceeded maximum http request retries for http://192.168.105.10:8080/swift/v1/glance/1b7b99db-bc31-46ba-89d2-1a7ea6c4cc00?temp_url_sig=4951acacc773299640fbe68db75cec4d4075f6c8&temp_url_expires=1489499924 | 14:57 |
| * jroll wonders what /usr/bin/provision is, have never heard of that | 14:57 | |
| jroll | it sounds like your provisioning network can't reach your swift host, though | 14:58 |
| *** faizy_ has joined #openstack-ironic | 14:58 | |
| jroll | though additional logs around that, and the agent logs, might help solidify that guess | 14:58 |
| jroll | could be an auth problem there too | 14:58 |
| erol | hmm actually I tested with telnet i can connect 192.168.105.10 8080 | 14:58 |
| JayF | erol: from your *provisioning agent*? | 14:58 |
| JayF | erol: is there no connection error or http status code with that log message? | 14:59 |
| *** athomas has joined #openstack-ironic | 14:59 | |
| erol | i m sending ouput via paste.openstack.org wait a sec | 15:00 |
| *** vsaienko has quit IRC | 15:00 | |
| *** faizy has quit IRC | 15:00 | |
| *** joanna has joined #openstack-ironic | 15:01 | |
| erol | http://paste.openstack.org/show/602698/ | 15:02 |
| erol | JayF: http://paste.openstack.org/show/602698/ | 15:02 |
| JayF | oh, that's all using fuel agent | 15:02 |
| JayF | that's not a part of ironic at all | 15:02 |
| JayF | I don't know where the right place is for support, but I don't think it's here :/ | 15:02 |
| erol | JayF: Ah sorry for that. | 15:03 |
| erol | JayF: Okay thank you. | 15:03 |
| jroll | vdrok: ^ may know the right channel | 15:03 |
| JayF | no problem, trying to think if anyone who would know the right channel for that is online | 15:03 |
| JayF | vsaienk0: ^ | 15:03 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: Add redfish driver https://review.openstack.org/438982 | 15:04 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: Add documentation for the redfish driver https://review.openstack.org/442537 | 15:04 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic master: DevStack: Setup a Redfish environment https://review.openstack.org/445529 | 15:04 |
| *** vsaienko has joined #openstack-ironic | 15:04 | |
| mgould | JayF: looks like it's #fuel: https://launchpad.net/fuel | 15:04 |
| JayF | erol: ^ mgould helpin' ya out! Good luck | 15:04 |
| mgould | good luck, erol! | 15:05 |
| vsaienk0 | JayF: going to check it with erol offline | 15:08 |
| joanna | morning vsaienk0 mgould JayF jroll | 15:10 |
| JayF | o/ | 15:10 |
| vsaienk0 | morning joanna | 15:10 |
| mjturek | hey ironic, our CI has been failing for a couple days. See end of this log for error https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/56/423556/85/check-ironic/tempest-dsvm-ironic-agent_ipmitool/267c0e0/devstacklog.txt.gz | 15:11 |
| mjturek | anyone seen this before? | 15:11 |
| jroll | joanna: morning :) | 15:11 |
| *** vsaienko has quit IRC | 15:11 | |
| mjturek | 'Timed out waiting for Nova hypervisor-stats vcpus >= 20' is the actual error | 15:12 |
| JayF | mjturek: https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/56/423556/85/check-ironic/tempest-dsvm-ironic-agent_ipmitool/267c0e0/screen-ir-cond.txt.gz full of rabbit errors | 15:12 |
| jroll | mjturek: so that happens when nova-compute isn't seeing resources | 15:12 |
| JayF | all the ironic logs are full enough of rabbitmq/rpc errors | 15:13 |
| JayF | that I think that's the source of most of the other shenanigans | 15:13 |
| jroll | mjturek: I think this is the problem "2017-03-14 12:12:07.685 31394 WARNING nova.virt.ironic.driver [req-4c1e695c-bfad-495b-b63b-d5ae822781ba - -] cpu_arch not defined for node '1c01f8bd-59c2-44c0-8183-b879b5cb1ad9' | 15:13 |
| patchbot | Error: No closing quotation | 15:13 |
| jroll | from https://dal05.objectstorage.softlayer.net/v1/AUTH_3d8e6ecb-f597-448c-8ec2-164e9f710dd6/pkvmci/ironic/56/423556/85/check-ironic/tempest-dsvm-ironic-agent_ipmitool/267c0e0/screen-n-cpu.txt.gz | 15:13 |
| JayF | jroll: afaict, from reading api/cond logs, the /v1/nodes is never returning | 15:13 |
| JayF | jroll: because it's constantly timing out | 15:13 |
| jroll | GET /v1/nodes shouldn't depend on rabbit | 15:14 |
| erol | JayF: vsaienk0: Thanks for help :) | 15:14 |
| JayF | there are more than a few posts before that | 15:14 |
| jroll | but that would cause problems enrolling and such | 15:14 |
| JayF | so even if cpu_arch is missing | 15:14 |
| JayF | it could be missing b/c the update calls are failing | 15:14 |
| jroll | ya | 15:14 |
| JayF | mjturek: I'd fix rabbit config/whatever first, re-run, and re-troubleshoot | 15:14 |
| mjturek | JayF: jroll: thanks!!! I'll also fix the cpu_arch problem because that shouldn't be happening anyway | 15:15 |
| JayF | mjturek: I think it's cause:effect | 15:15 |
| JayF | mjturek: if rpc is not going thru, your node enrollment automation is likely fubar | 15:15 |
| JayF | mjturek: and could cause your nodes to not have values populated you trust them to :) | 15:15 |
| * JayF applies the universal sysadmin rule: fix one thing at a time | 15:15 | |
| mjturek | fair enough :) | 15:16 |
| *** sayantan_ has joined #openstack-ironic | 15:16 | |
| JayF | if you fix ten things at once, you never get to know which one fixed it :P | 15:16 |
| mjturek | JayF: very true! | 15:16 |
| lucasagomes | mgould, btw replied to the comments at https://review.openstack.org/#/c/445478 , I can put a new patch-set up addressing the typos and stuff like that, but not sure if it's ideal (in case you agree with the replies, otherwise I will fix it) | 15:17 |
| patchbot | patch 445478 - ironic-specs - Follow-up patch for the Redfish spec | 15:17 |
| *** hshiina has quit IRC | 15:17 | |
| *** cdearborn has joined #openstack-ironic | 15:17 | |
| *** vsaienko has joined #openstack-ironic | 15:17 | |
| jlvillal | dtantsur: I added you to: https://review.openstack.org/#/c/443348/ This is related to the multi-node grenade stuff | 15:17 |
| patchbot | patch 443348 - openstack-infra/project-config - Run 'enable_plugin' in project-config for grenade ... | 15:17 |
| mgould | lucasagomes: looking now | 15:17 |
| lucasagomes | no rush | 15:18 |
| *** rbartal has quit IRC | 15:18 | |
| *** erol has quit IRC | 15:18 | |
| mgould | OK, looks like I *completely* misunderstood | 15:19 |
| dtantsur | jlvillal, ack | 15:20 |
| jlvillal | :) | 15:20 |
| mgould | I thought the redfish_system_id was what http://redfish.dmtf.org/schemas/DSP0266_1.1.html#uris calls the "unique resource path" | 15:20 |
| *** hoangcx has quit IRC | 15:21 | |
| mgould | precisely so we don't have to keep track of "Vendor X uses System/ and Vendor Y uses Nodes/ and Vendor Z uses PizzaBoxen/..." | 15:21 |
| *** zackf has joined #openstack-ironic | 15:21 | |
| mgould | just store "System/5" as the redfish_system_id and form URIs with the simplest code possible | 15:22 |
| mgould | so, where do you propose storing the "System/" bit? | 15:23 |
| lucasagomes | mgould, in sushy | 15:23 |
| mgould | is that going to be hardcoded in sushy and chosen based on vendor autodiscovery? | 15:23 |
| mgould | OK, I think that's a bad idea | 15:23 |
| lucasagomes | mgould, I want the library to make these types of convertions between different vendors etc | 15:23 |
| lucasagomes | so the user of the library don't need to care about it | 15:23 |
| lucasagomes | it will only call set_boot_source() and it should work | 15:24 |
| *** vsaienko has quit IRC | 15:24 | |
| *** udesale__ has quit IRC | 15:24 | |
| lucasagomes | independently of whether its /Systems or /Nodes or .... | 15:24 |
| lucasagomes | mgould, yes | 15:24 |
| mgould | OK | 15:24 |
| lucasagomes | mgould, currently the lib assume /Systems because I've never seem /Nodes or any other variant around, but when time comes we can make the changes in the lib | 15:25 |
| lucasagomes | and leave the ironic driver intact | 15:25 |
| mgould | what's wrong with having System/ as part of the redfish_system_id? | 15:25 |
| mgould | it's just a string, isn't it? | 15:25 |
| lucasagomes | mgould, yeah it's, not sure if there's anything wrong with that. Just different ways to handle the same problem | 15:25 |
| mgould | OK | 15:26 |
| lucasagomes | idk, I prefer the driver in Ironic to be agnostic about the system behind and leave these types of complexity to the library | 15:26 |
| lucasagomes | but idk, maybe some people might prefer tthe driver to handle more stuff idk | 15:26 |
| lucasagomes | something to discuss | 15:27 |
| mgould | so I think the advantage of using "System/5" as the system_id is (a) we're not deconstructing strings and looking for meaning, which the redfish spec tells us not to do, (b) we're not entering the world of keeping track of differences between firmware versions | 15:27 |
| lucasagomes | (and now is a good time to discuss) | 15:27 |
| mgould | I think "different firmwares use different URL structures" is a problem we should handle by making our code as stupid as possible and storing the varying bits as data | 15:28 |
| *** amarao has quit IRC | 15:28 | |
| *** vsaienko has joined #openstack-ironic | 15:28 | |
| mgould | also, "the way our driver works matches the way the protocol is specified" seems like a Good Thing for people trying to understand them both | 15:29 |
| lucasagomes | mgould, right, so system_id is not going to be only an ID it will be the name of the collection (for whatever firmware you use) + the ID | 15:29 |
| *** mtanino has quit IRC | 15:29 | |
| mgould | excellent | 15:29 |
| lucasagomes | which puts the burden in the user to know these differences instead of the having code to figure that out | 15:30 |
| mgould | lucasagomes: they'd be fetching it from the BMC anyway, I don't expect a human would ever have to type it in | 15:31 |
| openstackgerrit | Kyrylo Romanenko proposed openstack/python-ironicclient master: Tests for OSC baremetal node create command with options https://review.openstack.org/382352 | 15:31 |
| *** [1]cdearborn has quit IRC | 15:31 | |
| mgould | can a human even know the system ID without querying the BMC? | 15:31 |
| lucasagomes | mgould, operators will set that right ? Unless they use some sort of introspection | 15:31 |
| lucasagomes | mgould, probably yes, if he composes the node he will know the id | 15:32 |
| *** penick has joined #openstack-ironic | 15:32 | |
| mgould | huh | 15:32 |
| *** srart has joined #openstack-ironic | 15:32 | |
| mgould | I'd assumed IDs would be opaque and probably computer-generated | 15:32 |
| mgould | you think they might be human-meaningful? | 15:33 |
| mgould | and that those human-meaningful names will form part of the URIs? | 15:33 |
| NobodyCam | okay Now the Official "Good Moring Ironic'ers" | 15:34 |
| mgould | morning NobodyCam | 15:34 |
| lucasagomes | mgould, that's my assumption yeah, but I'm looking at the spec to see if they are auto generated | 15:34 |
| NobodyCam | morning mgould | 15:34 |
| mgould | lucasagomes: I'd expect a "compose node" endpoint to return the URI of the new node as part of its response | 15:35 |
| mgould | yep, "The service shall set the Location header to the URI of the newly created resource." | 15:36 |
| mgould | also, "The service may ignore any service controlled attributes (e.g., id), forcing those attributes to be overridden by the service." | 15:37 |
| *** jvgrant_ has quit IRC | 15:37 | |
| lucasagomes | mgould, yeah i assume that too, /me looks if there's anything related to valence about that... cause I haven't tried to compose a node with redfish ever | 15:37 |
| mgould | lucasagomes: looks to me ^^^ like the user can specify a node ID, but the redfish manager is free to ignore it | 15:38 |
| *** vsaienko has quit IRC | 15:38 | |
| mgould | we'd need to know the name of the collection of nodes in order to POST to it | 15:39 |
| mgould | but Ironic doesn't (yet?) need that capability | 15:39 |
| lucasagomes | mgould, we don't have any abstraction in the api to compose nodes yet | 15:41 |
| mgould | yep | 15:41 |
| *** vsaienko has joined #openstack-ironic | 15:41 | |
| lucasagomes | it's something that we may want to handle in the future since we have hardware that supports it UCS, oneview, redfish (now) | 15:41 |
| lucasagomes | so... idk, we need to see what's common accross all these different technologies and come up with a cross-vendor api | 15:41 |
| lucasagomes | which is not that easy | 15:41 |
| lucasagomes | (but needs to be done eventually) | 15:42 |
| *** jvgrant_ has joined #openstack-ironic | 15:42 | |
| vdrok | sorry, was out for lunch, but it seems vasyl handled the problem | 15:43 |
| openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic-inspector master: Fix inconsistent service naming in install guide https://review.openstack.org/437537 | 15:48 |
| openstackgerrit | Merged openstack/bifrost stable/ocata: playbooks: roles: {ironic,keystone}: Drop --force-reinstall https://review.openstack.org/445368 | 15:49 |
| *** vsaienko has quit IRC | 15:54 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: [WIP] Add resource classes integration tests https://review.openstack.org/443628 | 15:57 |
| jlvillal | vsaienk0: I think everything has landed for multi-node multi-tenant grenade :) Except I think we need this: https://review.openstack.org/#/c/439958/ | 15:57 |
| patchbot | patch 439958 - networking-generic-switch - Do not 'enable_plugin networking-generic-switch' i... | 15:57 |
| *** vsaienko has joined #openstack-ironic | 15:58 | |
| vsaienk0 | jlvillal: I will check that today | 15:59 |
| jlvillal | vsaienk0: Should we move multi-node grenade to become a non-voting job? I think yes. | 15:59 |
| vsaienk0 | jlvillal: thanks! I think we can propose patch to made grenade multitenant multinode as non-voting | 15:59 |
| jlvillal | vsaienk0: Sounds good. I will work on it now. | 15:59 |
| vsaienk0 | jlvillal: thanks! | 15:59 |
| rama_y_ | Hi aarefiev, modified the spec per recommendations: https://review.openstack.org/#/c/406790/; thanks! | 16:00 |
| patchbot | patch 406790 - ironic-inspector-specs - Collect Bios, Memory and Processor Info from Dmide... | 16:00 |
| *** penick has quit IRC | 16:00 | |
| aarefiev | rama_y_: thanks, it's in my review list | 16:01 |
| openstackgerrit | Galyna Zholtkevych proposed openstack/ironic master: [WIP] Support oslo.messaging.zmq in ironic https://review.openstack.org/445550 | 16:03 |
| jlvillal | pas-ha: Are you still -1 on https://review.openstack.org/#/c/437936/ ? | 16:04 |
| patchbot | patch 437936 - openstack-infra/project-config - Remove experimental ironic grenade multitenant sin... | 16:04 |
| *** rama_y_ has quit IRC | 16:04 | |
| pas-ha | nope, talked with vsaienk0 already, +1 | 16:05 |
| *** crushil has quit IRC | 16:05 | |
| *** vills_ has quit IRC | 16:06 | |
| jlvillal | pas-ha: Thanks! | 16:07 |
| dtantsur | jlvillal, I'm lazy to check, do you have all necessary excludes for all branches for https://review.openstack.org/#/c/445555/ in place? | 16:07 |
| patchbot | patch 445555 - openstack-infra/project-config - ironic: Make grenade multi-node job non-voting | 16:07 |
| jlvillal | dtantsur: Uh, no I don't. I didn't think about that :( | 16:07 |
| jlvillal | dtantsur: Let me fix that! Thanks. | 16:07 |
| dtantsur | sure :) | 16:07 |
| *** bcornec2 has joined #openstack-ironic | 16:10 | |
| *** jkilpatr has quit IRC | 16:10 | |
| *** vsaienko has quit IRC | 16:11 | |
| *** aarefiev is now known as aarefiev_afk | 16:14 | |
| *** vsaienko has joined #openstack-ironic | 16:16 | |
| *** hw_wutianwei has quit IRC | 16:21 | |
| *** jkilpatr has joined #openstack-ironic | 16:23 | |
| openstackgerrit | Merged openstack/ironic stable/ocata: Set tempest's max microversion to 1.31 https://review.openstack.org/445061 | 16:25 |
| openstackgerrit | Vasyl Saienko proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/436957 | 16:25 |
| *** Nisha_Agarwal has joined #openstack-ironic | 16:27 | |
| *** alexpilo_ has joined #openstack-ironic | 16:27 | |
| *** alexpilotti has quit IRC | 16:28 | |
| *** trown is now known as trown|lunch | 16:29 | |
| *** crushil has joined #openstack-ironic | 16:31 | |
| *** e0ne has quit IRC | 16:31 | |
| *** penick has joined #openstack-ironic | 16:34 | |
| openstackgerrit | Merged openstack/bifrost master: Ironic inventory source duplicates MACs https://review.openstack.org/443037 | 16:34 |
| *** mjturek has quit IRC | 16:35 | |
| *** yogi_ has quit IRC | 16:36 | |
| *** mjturek has joined #openstack-ironic | 16:36 | |
| *** yogi_ has joined #openstack-ironic | 16:36 | |
| *** jvgrant has quit IRC | 16:38 | |
| openstackgerrit | Vladyslav Drok proposed openstack/ironic master: Add resource classes integration tests https://review.openstack.org/443628 | 16:41 |
| *** ccamacho has quit IRC | 16:42 | |
| openstackgerrit | Merged openstack/bifrost master: Serial-console renamed by diskimage-builder https://review.openstack.org/444173 | 16:43 |
| rama_y | aarefiev, thanks! | 16:50 |
| *** dtantsur is now known as dtantsur|brb | 16:51 | |
| *** udesale has joined #openstack-ironic | 16:53 | |
| *** udesale has quit IRC | 16:53 | |
| *** jaganathan has joined #openstack-ironic | 16:55 | |
| *** d0ugal has quit IRC | 16:55 | |
| *** fragatina has joined #openstack-ironic | 16:56 | |
| *** fragatina has quit IRC | 16:57 | |
| *** fragatina has joined #openstack-ironic | 16:57 | |
| *** fragatina has quit IRC | 17:03 | |
| soliosg | jlvillal: any idea why one of the jobs in https://review.openstack.org/#/c/439252/ fails with 'insufficient compute resources'? | 17:08 |
| patchbot | patch 439252 - ironic - Include a copy of tempest.scenario.manager module | 17:08 |
| jlvillal | soliosg: Cosmic radiation? | 17:09 |
| jlvillal | soliosg: Or the nova bug | 17:10 |
| jroll | that sounds like that nova bug, yeah | 17:10 |
| soliosg | jlvillal: I see, so 'recheck' won't help if there's a bug ongoing | 17:10 |
| jlvillal | soliosg: https://review.openstack.org/#/c/444106/ | 17:10 |
| patchbot | patch 444106 - nova - Add populate_retry to schedule_and_build_instances | 17:10 |
| jroll | soliosg: it's intermittent | 17:10 |
| jlvillal | soliosg: It is an intermittent bug. So recheck probably will help. | 17:11 |
| rloo | soliosg: should be mentioned in our etherpad | 17:11 |
| rloo | soliosg: if not, someone should update the etherpad :) | 17:11 |
| jlvillal | soliosg: https://bugs.launchpad.net/nova/+bug/1670319 | 17:11 |
| openstack | Launchpad bug 1671648 in OpenStack Compute (nova) "duplicate for #1670319 Instances are not rescheduled after deploy fails" [High,In progress] - Assigned to Shunli Zhou (shunliz) | 17:11 |
| jroll | tis mentioned already | 17:12 |
| alezil | I get the same error in local development environment, but I was not able to resolve it. | 17:14 |
| openstackgerrit | vinay kumar muddu proposed openstack/ironic master: [WIP] Fix issue with boot and deploy interface validations https://review.openstack.org/445588 | 17:17 |
| *** Nisha_Agarwal has quit IRC | 17:20 | |
| *** mjturek has quit IRC | 17:22 | |
| *** fragatina has joined #openstack-ironic | 17:23 | |
| jlvillal | Hey stable reviewers :) https://review.openstack.org/#/c/443879/ & https://review.openstack.org/#/c/443802/ & https://review.openstack.org/443800 | 17:25 |
| patchbot | patch 443879 - python-ironicclient (stable/ocata) - Updated from global requirements | 17:25 |
| patchbot | patch 443802 - ironic-lib (stable/ocata) - Updated from global requirements | 17:25 |
| patchbot | patch 443800 - ironic (stable/ocata) - Updated from global requirements | 17:25 |
| jlvillal | and https://review.openstack.org/443804 | 17:26 |
| patchbot | patch 443804 - ironic-python-agent (stable/ocata) - Updated from global requirements | 17:26 |
| lucasagomes | folks, I'm calling it a day | 17:26 |
| lucasagomes | have a great evening all! | 17:26 |
| *** lucasagomes is now known as lucas-afk | 17:26 | |
| jlvillal | lucas-afk: Good night! | 17:27 |
| openstackgerrit | Merged openstack/bifrost master: Updated from global requirements https://review.openstack.org/445064 | 17:28 |
| *** trown|lunch is now known as trown | 17:28 | |
| *** amarao has joined #openstack-ironic | 17:28 | |
| jlvillal | soliosg: No need to do: recheck bug xxxxxxx | 17:28 |
| jlvillal | soliosg: Just recheck. I guess they gave up on the 'bug xxxxxx' part. | 17:28 |
| *** jaganathan has quit IRC | 17:29 | |
| soliosg | jlvillal: thanks, this was a suggestion in https://etherpad.openstack.org/p/IronicWhiteBoard | 17:29 |
| jlvillal | soliosg: I believe that is out of date. | 17:32 |
| jlvillal | rloo: Didn't you write up something about the gate CI? I thought that is where I read that we don't use "recheck bug xxxxx" anymore. | 17:33 |
| rloo | jlvillal: oh, i sent out email internally/downstream about it, not upstream. | 17:36 |
| rloo | jlvillal: so, there is no need to do recheck bug #, but I think it is very useful and people should do recheck <some reason>. if that is a bug number, then great. | 17:36 |
| *** jvgrant has joined #openstack-ironic | 17:36 | |
| rloo | jlvillal: at the very least, it indicates that the person looked into it, instead of blindly doing 'recheck'. | 17:37 |
| jlvillal | rloo: Thanks. | 17:37 |
| *** alineb has quit IRC | 17:38 | |
| *** jaosorior has quit IRC | 17:45 | |
| *** vsaienko has quit IRC | 17:47 | |
| openstackgerrit | Solio Sarabia proposed openstack/ironic master: Include a copy of tempest.scenario.manager module https://review.openstack.org/439252 | 17:48 |
| *** tesseract has quit IRC | 17:49 | |
| *** snecklifter has quit IRC | 17:52 | |
| openstackgerrit | Jarrod Johnson proposed openstack/pyghmi master: Change the wording of setup event https://review.openstack.org/445602 | 17:52 |
| *** rloo has quit IRC | 17:52 | |
| *** rloo has joined #openstack-ironic | 17:52 | |
| *** derekh has quit IRC | 17:55 | |
| *** fellypefca has joined #openstack-ironic | 17:55 | |
| *** athomas has quit IRC | 17:56 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 17:58 |
| *** d0ugal has joined #openstack-ironic | 17:58 | |
| *** jvgrant has quit IRC | 18:00 | |
| *** faizy_ has quit IRC | 18:00 | |
| *** mgoddard has quit IRC | 18:01 | |
| *** ralonsoh has quit IRC | 18:02 | |
| *** venki has quit IRC | 18:02 | |
| *** dtantsur|brb is now known as dtantsur | 18:03 | |
| *** pcaruana has quit IRC | 18:03 | |
| *** d0ugal has quit IRC | 18:04 | |
| *** mgagne_ is now known as mgagne | 18:05 | |
| *** snecklifter has joined #openstack-ironic | 18:05 | |
| *** Sukhdev has joined #openstack-ironic | 18:11 | |
| *** jkilpatr has quit IRC | 18:12 | |
| *** jkilpatr has joined #openstack-ironic | 18:13 | |
| openstackgerrit | Merged openstack/ironic-lib master: Updated from global requirements https://review.openstack.org/445082 | 18:13 |
| *** jvgrant has joined #openstack-ironic | 18:17 | |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Fix some virtualenv-related places https://review.openstack.org/445350 | 18:19 |
| openstackgerrit | Pavlo Shchelokovskyy proposed openstack/bifrost master: Always install shade to system https://review.openstack.org/445609 | 18:19 |
| fellypefca | Hi vdrok and dmitry. Did you had sometime to review our patchs? (386014, 436469, 377106, 408298)? | 18:19 |
| *** mgould is now known as mgould|afk | 18:26 | |
| *** hw_wutianwei has joined #openstack-ironic | 18:28 | |
| *** amarao has quit IRC | 18:34 | |
| *** hw_wutianwei has quit IRC | 18:38 | |
| *** dprince has quit IRC | 18:38 | |
| *** racedo has quit IRC | 18:40 | |
| *** mgoddard has joined #openstack-ironic | 18:42 | |
| *** fragatina has quit IRC | 18:44 | |
| *** fragatina has joined #openstack-ironic | 18:44 | |
| *** amotoki has quit IRC | 18:56 | |
| JayF | jroll: dtantsur: TheJulia: When non-frontend-devs attend the UI meeting: https://www.youtube.com/watch?v=QrGrOK8oZG8 | 18:58 |
| * dtantsur bookmarks :) | 18:59 | |
| JayF | if you haven't actaully seen it, it's wtf-worthy, but mainly the joke is that it's too many cooks :) | 18:59 |
| TheJulia | JayF: nice :) | 19:00 |
| TheJulia | Joy, looks like launchpad is timing out changing item priority at the moment :( | 19:01 |
| dtantsur | that's normal for it, wait a few minutes | 19:02 |
| dtantsur | also, 8pm, time for dinner :) see you tomorrow | 19:02 |
| JayF | o/ | 19:02 |
| *** dtantsur is now known as dtantsur|afk | 19:02 | |
| TheJulia | Goodnight dtantsur|afk | 19:03 |
| *** penick has quit IRC | 19:04 | |
| *** rloo has quit IRC | 19:09 | |
| *** rloo has joined #openstack-ironic | 19:09 | |
| *** penick has joined #openstack-ironic | 19:09 | |
| jroll | TheJulia: I hear phantom and nightmare are also used | 19:11 |
| jroll | (for frontend testing) | 19:11 |
| TheJulia | Are their names representative of the experience? | 19:11 |
| jroll | ha! | 19:11 |
| openstackgerrit | Merged openstack/ironic master: Updated from global requirements https://review.openstack.org/445080 | 19:13 |
| *** vsaienko has joined #openstack-ironic | 19:13 | |
| TheJulia | jroll: had to ask, it was too tempting :) | 19:13 |
| jroll | :P | 19:14 |
| *** vsaienko has quit IRC | 19:15 | |
| openstackgerrit | Merged openstack/ironic master: Fix typos of filename in api-ref https://review.openstack.org/445293 | 19:15 |
| *** fellypefca has quit IRC | 19:17 | |
| *** dsneddon_ has joined #openstack-ironic | 19:18 | |
| *** dsneddon_ is now known as dsneddon | 19:19 | |
| *** mgoddard has quit IRC | 19:25 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Testing multi-tenant multi-node grenade job https://review.openstack.org/422149 | 19:31 |
| openstackgerrit | Merged openstack/pyghmi master: Change the wording of setup event https://review.openstack.org/445602 | 19:32 |
| *** penick has quit IRC | 19:33 | |
| *** penick has joined #openstack-ironic | 19:36 | |
| jlvillal | JayF: hurricanerix: Any idea why it appears that Ironic conductor is running in Python 2.7 in the Python 3 job? http://logs.openstack.org/49/422149/40/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/9939f5e/ | 19:37 |
| jlvillal | Like these all say python 2.7 http://logs.openstack.org/49/422149/40/experimental/gate-tempest-dsvm-ironic-ipa-partition-pxe_ipmitool-tinyipa-python3-ubuntu-xenial-nv/9939f5e/logs/screen-ir-api.txt.gz Strange. | 19:38 |
| *** bcornec2 has quit IRC | 19:38 | |
| hurricanerix | jlvillal: that is a good question that i don't have an answer for right now, but i can take a look. | 19:40 |
| *** vsaienko has joined #openstack-ironic | 19:42 | |
| jlvillal | hurricanerix: Thanks. I saw your name listed to the Python3 item. I was looking at it and got confused... :) | 19:42 |
| *** amoralej is now known as amoralej|off | 19:43 | |
| hurricanerix | jlvillal: np | 19:46 |
| jlvillal | hurricanerix: I think I know. ENABLED_PYTHON3_PACKAGES | 19:47 |
| hurricanerix | jlvillal: where is that set? | 19:48 |
| jlvillal | hurricanerix: devstack | 19:49 |
| hurricanerix | oh i see | 19:49 |
| jroll | jlvillal: hurricanerix: like this? https://review.openstack.org/#/c/439605/ | 19:49 |
| patchbot | patch 439605 - ironic - explicitly enable python3 in dsvm jobs | 19:49 |
| jroll | it gets set automatically for anything with python 3 in the trove classifiers, btw, but we don't have that: https://github.com/openstack/ironic/blob/master/setup.cfg#L17 | 19:50 |
| jroll | so one of those two methods will be needed | 19:50 |
| jlvillal | jroll: Ah cool. Thanks. | 19:50 |
| jroll | jlvillal: note the depends-on there, might need to remove that (if we don't set the trove classifiers) | 19:50 |
| jlvillal | jroll: Thanks. | 19:51 |
| jroll | np :) | 19:51 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: Python 3 testing... https://review.openstack.org/445636 | 19:54 |
| *** penick has quit IRC | 19:58 | |
| *** vills has joined #openstack-ironic | 20:03 | |
| *** e0ne has joined #openstack-ironic | 20:13 | |
| *** vsaienko has quit IRC | 20:15 | |
| mrda | Morning Ironic | 20:16 |
| *** e0ne has quit IRC | 20:18 | |
| *** r-daneel has joined #openstack-ironic | 20:18 | |
| *** vsaienko has joined #openstack-ironic | 20:19 | |
| *** amarao has joined #openstack-ironic | 20:19 | |
| *** amarao has quit IRC | 20:19 | |
| openstackgerrit | Félix Bouliane proposed openstack/ironic master: Make snmp udp transport settings configurable https://review.openstack.org/396400 | 20:20 |
| *** e0ne has joined #openstack-ironic | 20:21 | |
| *** dprince has joined #openstack-ironic | 20:35 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 20:36 |
| *** vsaienko has quit IRC | 20:36 | |
| *** crushil has quit IRC | 20:36 | |
| *** vsaienko has joined #openstack-ironic | 20:37 | |
| *** jkilpatr has quit IRC | 20:37 | |
| TheJulia | good morning mrda | 20:38 |
| mrda | o/ | 20:40 |
| *** penick has joined #openstack-ironic | 20:42 | |
| *** vsaienko has quit IRC | 20:43 | |
| *** jaypipes has quit IRC | 20:43 | |
| *** crushil has joined #openstack-ironic | 20:45 | |
| *** Goneri has quit IRC | 20:46 | |
| *** vsaienko has joined #openstack-ironic | 20:48 | |
| *** mjura has quit IRC | 20:49 | |
| soliosg | Ironic: This change is ready to review, https://review.openstack.org/#/c/439252/, to complete the request from QA Tempest Team | 20:55 |
| patchbot | patch 439252 - ironic - Include a copy of tempest.scenario.manager module | 20:55 |
| jroll | TheJulia: we might want to add to this https://wiki.openstack.org/wiki/Forum/Boston2017 | 20:56 |
| * jroll forgets where that pad is | 20:56 | |
| *** vsaienko has quit IRC | 20:57 | |
| *** jkilpatr has joined #openstack-ironic | 20:58 | |
| *** athomas has joined #openstack-ironic | 20:59 | |
| *** trown is now known as trown|outtypewww | 20:59 | |
| *** vsaienko has joined #openstack-ironic | 21:03 | |
| *** vsaienko has quit IRC | 21:10 | |
| jroll | soliosg: +2 | 21:11 |
| jroll | someone should land that | 21:11 |
| soliosg | jroll: thank you | 21:12 |
| jroll | np! | 21:12 |
| *** e0ne has quit IRC | 21:15 | |
| *** crushil has quit IRC | 21:15 | |
| *** crushil has joined #openstack-ironic | 21:15 | |
| *** vsaienko has joined #openstack-ironic | 21:16 | |
| JayF | soliosg: jroll: +2, but not going to +A because by osic + all osic voters | 21:16 |
| *** e0ne has joined #openstack-ironic | 21:17 | |
| *** ElCoyote_ has quit IRC | 21:20 | |
| jroll | JayF: I think that's a bit silly for something we agreed on in meetings/ML, but sure | 21:20 |
| JayF | *shrug* I don't have any objection if you wanna land it, you know I'm a stickler for something like that :) | 21:21 |
| jroll | yeah it's fine, we can wait | 21:21 |
| *** vsaienko has quit IRC | 21:22 | |
| *** ElCoyote_ has joined #openstack-ironic | 21:23 | |
| *** vsaienko has joined #openstack-ironic | 21:28 | |
| *** e0ne has quit IRC | 21:28 | |
| *** crushil has quit IRC | 21:31 | |
| *** e0ne has joined #openstack-ironic | 21:32 | |
| soliosg | JayF, jroll: Understand, I like the community spirit/considerations | 21:34 |
| *** jcoufal has quit IRC | 21:36 | |
| *** Sukhdev has quit IRC | 21:37 | |
| *** vsaienko has quit IRC | 21:37 | |
| *** vsaienko has joined #openstack-ironic | 21:41 | |
| *** e0ne has quit IRC | 21:42 | |
| *** Sukhdev has joined #openstack-ironic | 21:51 | |
| *** vsaienko has quit IRC | 21:56 | |
| *** dprince has quit IRC | 21:56 | |
| *** kjw3 has quit IRC | 22:06 | |
| *** penick has quit IRC | 22:14 | |
| *** priteau has quit IRC | 22:20 | |
| *** jcoufal has joined #openstack-ironic | 22:24 | |
| *** Goneri has joined #openstack-ironic | 22:25 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic master: WIP/DNM: devstack: When Python 3 enabled, use Python 3 https://review.openstack.org/445636 | 22:31 |
| *** jvgrant has quit IRC | 22:33 | |
| *** ijw has joined #openstack-ironic | 22:45 | |
| *** jvgrant has joined #openstack-ironic | 22:58 | |
| *** Goneri has quit IRC | 23:12 | |
| *** sambetts has joined #openstack-ironic | 23:24 | |
| sambetts | Morning all | 23:24 |
| *** causten has quit IRC | 23:26 | |
| *** baoli has quit IRC | 23:26 | |
| *** causten has joined #openstack-ironic | 23:27 | |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add base cinder common interface https://review.openstack.org/355625 | 23:28 |
| *** causten has quit IRC | 23:31 | |
| rama_y | Hi, any idea about this bug? https://bugs.launchpad.net/ironic/+bug/1672622 | 23:31 |
| openstack | Launchpad bug 1672622 in Ironic "Timed out waiting for Nova hypervisor-stats in installing devstack" [Undecided,New] - Assigned to Tao Li (eric-litao) | 23:31 |
| rama_y | I see the same error: Timed out waiting for Nova hypervisor-stats vcpus >= 3 | 23:31 |
| openstackgerrit | Joanna Taryma proposed openstack/ironic master: Add Cinder storage driver https://review.openstack.org/366197 | 23:36 |
| *** rloo has quit IRC | 23:37 | |
| *** pmannidi has joined #openstack-ironic | 23:41 | |
| *** jcoufal has quit IRC | 23:51 | |
| *** jvgrant has quit IRC | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!