*** achanda has quit IRC | 00:01 | |
*** smoriya has joined #openstack-ironic | 00:05 | |
jlvillal | mrda: the patch https://review.openstack.org/216802 should unblock the gate. Maybe it will merge in the next hour. | 00:05 |
---|---|---|
*** romcheg has quit IRC | 00:06 | |
*** dims_ has joined #openstack-ironic | 00:06 | |
*** thrash is now known as thrash|g0ne | 00:07 | |
*** dims has quit IRC | 00:07 | |
*** baoli_ has quit IRC | 00:08 | |
*** dims has joined #openstack-ironic | 00:12 | |
*** asd112z has joined #openstack-ironic | 00:14 | |
*** tsekiyam_ has joined #openstack-ironic | 00:15 | |
*** dims_ has quit IRC | 00:15 | |
*** puranamr has quit IRC | 00:16 | |
*** mtanino has quit IRC | 00:18 | |
*** tsekiyama has quit IRC | 00:19 | |
*** tsekiyam_ has quit IRC | 00:20 | |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #openstack-ironic | 00:23 | |
jlvillal | JoshNang: Now with passing python34 unit-tests! https://review.openstack.org/#/c/215854/ :) | 00:27 |
*** VikasC has joined #openstack-ironic | 00:30 | |
*** openstack has joined #openstack-ironic | 00:34 | |
*** VikasC has quit IRC | 00:37 | |
*** meghal has quit IRC | 00:39 | |
*** Sukhdev has quit IRC | 00:45 | |
*** baoli has quit IRC | 00:47 | |
*** shoutm has quit IRC | 00:47 | |
*** baoli has joined #openstack-ironic | 00:48 | |
*** shoutm has joined #openstack-ironic | 00:49 | |
*** shoutm has quit IRC | 01:00 | |
*** shoutm has joined #openstack-ironic | 01:00 | |
*** shoutm has quit IRC | 01:07 | |
*** VikasC_ has quit IRC | 01:08 | |
*** shoutm has joined #openstack-ironic | 01:09 | |
*** zhenguo has joined #openstack-ironic | 01:17 | |
*** puranamr has joined #openstack-ironic | 01:20 | |
*** humble__ has joined #openstack-ironic | 01:23 | |
*** puranamr has quit IRC | 01:24 | |
*** puranamr has joined #openstack-ironic | 01:25 | |
*** bnemec has quit IRC | 01:26 | |
*** shoutm has quit IRC | 01:27 | |
*** puranamr has quit IRC | 01:31 | |
*** chenglch has joined #openstack-ironic | 01:34 | |
*** shoutm has joined #openstack-ironic | 01:40 | |
*** Haomeng has joined #openstack-ironic | 01:56 | |
*** Haomeng|2 has quit IRC | 01:59 | |
*** Marga_ has quit IRC | 02:09 | |
*** rloo has quit IRC | 02:10 | |
*** meghal has joined #openstack-ironic | 02:19 | |
*** boris-42 has quit IRC | 02:20 | |
*** meghal1 has joined #openstack-ironic | 02:21 | |
*** achanda has joined #openstack-ironic | 02:23 | |
*** meghal has quit IRC | 02:24 | |
*** baoli has quit IRC | 02:26 | |
*** baoli has joined #openstack-ironic | 02:35 | |
*** achanda has quit IRC | 02:38 | |
*** mtanino has joined #openstack-ironic | 02:41 | |
*** dims has quit IRC | 02:46 | |
*** ramineni has joined #openstack-ironic | 02:52 | |
*** hakimo has joined #openstack-ironic | 02:52 | |
*** hakimo_ has quit IRC | 02:54 | |
*** ramineni1 has joined #openstack-ironic | 02:56 | |
*** ramineni has quit IRC | 02:56 | |
*** ramineni has joined #openstack-ironic | 02:58 | |
*** Nisha has joined #openstack-ironic | 03:00 | |
*** alex_xu has quit IRC | 03:00 | |
*** ramineni1 has quit IRC | 03:00 | |
*** ramineni1 has joined #openstack-ironic | 03:00 | |
*** ramineni2 has joined #openstack-ironic | 03:02 | |
*** ramineni has quit IRC | 03:03 | |
*** ramineni has joined #openstack-ironic | 03:04 | |
*** ramineni1 has quit IRC | 03:05 | |
*** ramineni2 has quit IRC | 03:06 | |
*** ramineni has quit IRC | 03:08 | |
*** alex_xu has joined #openstack-ironic | 03:17 | |
*** sonuk has joined #openstack-ironic | 03:24 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Base IronicObject on VersionedObject instead of VersionedObjectDictCompat https://review.openstack.org/213602 | 03:26 |
openstackgerrit | Tan Lin proposed openstack/ironic: Base IronicObject on VersionedObject https://review.openstack.org/213602 | 03:28 |
*** achanda has joined #openstack-ironic | 03:30 | |
*** alex_xu has quit IRC | 03:32 | |
*** alex_xu has joined #openstack-ironic | 03:33 | |
*** achanda has quit IRC | 03:35 | |
*** Marga_ has joined #openstack-ironic | 03:41 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add new method validate_raid_config to RAIDInterface https://review.openstack.org/216946 | 03:44 |
*** links has joined #openstack-ironic | 03:45 | |
*** boris-42 has joined #openstack-ironic | 03:45 | |
*** Marga_ has quit IRC | 03:49 | |
*** rameshg87 has joined #openstack-ironic | 03:50 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add RPC APIs for RAID configuration https://review.openstack.org/196006 | 03:50 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add new method validate_raid_config to RAIDInterface https://review.openstack.org/216946 | 03:55 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add RPC APIs for RAID configuration https://review.openstack.org/196006 | 03:56 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add APIs for RAID configuration https://review.openstack.org/196007 | 03:57 |
*** baoli has quit IRC | 04:04 | |
*** naohirot has joined #openstack-ironic | 04:06 | |
*** VikasC has joined #openstack-ironic | 04:25 | |
*** humble__ has quit IRC | 04:28 | |
*** shoutm_ has joined #openstack-ironic | 04:32 | |
*** shoutm has quit IRC | 04:34 | |
*** meghal1 has quit IRC | 04:38 | |
*** Marga_ has joined #openstack-ironic | 04:45 | |
*** Marga_ has quit IRC | 04:49 | |
*** Sukhdev has joined #openstack-ironic | 04:51 | |
rameshg87 | naohirot: hi | 04:59 |
naohirot | rameshg87: Hi | 04:59 |
rameshg87 | naohirot: can you help in refactoring irmc driver into a boot interface. I hope you are already aware of the ongoing work of refactoring drivers into boot/deploy interface | 05:00 |
rameshg87 | naohirot: refactoring of pxe_* drivers is merged, agent_* drivers will be merged very soon | 05:01 |
naohirot | rameshg87: yes, I noticed that you sent an email to openstack-dev | 05:01 |
rameshg87 | naohirot: *ilo drivers in progress - https://review.openstack.org/#/c/216538/ | 05:01 |
*** achanda has joined #openstack-ironic | 05:01 | |
rameshg87 | naohirot: the only other driver left is iscsi_irmc and agent_irmc | 05:01 |
naohirot | rameshg87: really, only irmc's are letft. that's not good :) | 05:02 |
naohirot | rameshg87: let me see the gerrit patch. | 05:03 |
rameshg87 | naohirot: pxe_irmc is already done | 05:03 |
naohirot | rameshg87: Oh, I didn't realize it, when did you do? | 05:03 |
*** PradeepV has joined #openstack-ironic | 05:06 | |
naohirot | rameshg87: I just browsed the ilo patch. My first impression seems not trivial task. | 05:07 |
PradeepV | Good morning Ironic | 05:07 |
rameshg87 | naohirot: https://review.openstack.org/166513 | 05:07 |
rameshg87 | naohirot: yeah, not trivial, but not that difficult too. the major time consuming thing will be fixing unit tests | 05:08 |
naohirot | rameshg87: I see | 05:08 |
openstackgerrit | Merged openstack/ironic: Update tests to reflect WSME 0.8 fixes https://review.openstack.org/216802 | 05:09 |
naohirot | rameshg87: 166513 has been merged nearly one month ago. | 05:09 |
naohirot | rameshg87: 166513 has only a few lines of code are changed in terms of iRMC | 05:10 |
rameshg87 | naohirot: yeah, it has been a month :) | 05:10 |
naohirot | rameshg87: maybe that's the reason I didn't realized :) | 05:11 |
rameshg87 | naohirot: yeah, but pxe_irmc driver already uses a boot interface PXEBoot | 05:11 |
rameshg87 | yeah, it was applicable for every pxe* driver | 05:11 |
naohirot | rameshg87: Aha, most of changes have been done in common code. | 05:12 |
rameshg87 | NOTICE: wsme patch is merged, so issue rechecks on your patches | 05:13 |
naohirot | rameshg87: So I'm going to take a look how to adapt the new I/F, give me some time | 05:13 |
rameshg87 | naohirot: sure, thanks a lot | 05:13 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add new method validate_raid_config to RAIDInterface https://review.openstack.org/216946 | 05:14 |
*** davideagnello has quit IRC | 05:14 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add RPC APIs for RAID configuration https://review.openstack.org/196006 | 05:14 |
naohirot | rameshg87: what time range are you expecting for irmc to adapt it? | 05:14 |
rameshg87 | naohirot: as early as possible, but no hurries. take your time. | 05:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/216562 | 05:15 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/216966 | 05:15 |
rameshg87 | naohirot: it will be good if we can complete refactoring of upstream drivers by next release | 05:15 |
PradeepV | I am trying to deploy ironic ramdisk, followed the steps given in http://docs.openstack.org/developer/ironic/deploy/install-guide.html#building-or-downloading-a-deploy-ramdisk-image, and it created two images coreos_production_pxe_image-oem.cpio.gz and coreos_production_pxe.vmlinuz | 05:16 |
naohirot | rameshg87: Okay, right now I'm concentrating on soft power off and inject nmi things as you know | 05:16 |
PradeepV | are these the deploy images for ironic agent | 05:16 |
rameshg87 | naohirot: okay | 05:16 |
naohirot | rameshg87: can I ask you some advice? | 05:16 |
rameshg87 | naohirot: sure | 05:17 |
naohirot | rameshg87: I haven't write test code, because I didn't get feed back from devanada and jim. | 05:18 |
naohirot | rameshg87: thank, I'd like to get your feedback about cancel. | 05:19 |
naohirot | rameshg87: I'd like to know if current way is okay or not. | 05:20 |
naohirot | rameshg87: here is the implementation https://review.openstack.org/#/c/216730/ | 05:21 |
rameshg87 | naohirot: I will take a look at it. I remember the last time I had some comments, but looked mostly okay to me. probably the spec should be merged first. | 05:21 |
naohirot | rameshg87: Yes, I agree. the reason I presented the code is that code is concrete, but spec is a little bit abstract. | 05:22 |
naohirot | rameshg87: review comments are very welcome to the spec https://review.openstack.org/#/c/186700/ | 05:23 |
rameshg87 | naohirot: sure, will take a look at it | 05:24 |
naohirot | rameshg87: thanks. | 05:24 |
Haomeng | PradeepV: morning | 05:24 |
PradeepV | Hi Haomeng, i followed the above steps to create deploy images for agent driver, it created two images it created two images, coreos_production_pxe_image-oem.cpio.gz and coreos_production_pxe.vmlinuz | 05:31 |
PradeepV | can we use the same whole disk image for the OS | 05:32 |
*** amotoki has joined #openstack-ironic | 05:33 | |
Haomeng | PradeepV: this is agent, not the user os, so suggest to run dib to create user os | 05:34 |
*** asd112z has quit IRC | 05:35 | |
*** dims has joined #openstack-ironic | 05:36 | |
PradeepV | Haomeng: my question was can we use the same wholedisk image which we created for pxe_driver as the OS to bring up the instance | 05:38 |
*** dims has quit IRC | 05:42 | |
Haomeng | PradeepV: I remember IPA image support pxe_driver also, let me check the documents | 05:43 |
Haomeng | PradeepV: but suggest run with agent driver | 05:44 |
Haomeng | PradeepV: pxe driver will be not supportted for long time | 05:45 |
*** VikasC has quit IRC | 05:45 | |
PradeepV | ok Haomeng: how can we create the OS for agent driver | 05:46 |
*** Marga_ has joined #openstack-ironic | 05:46 | |
Haomeng | PradeepV: you mean user image? | 05:46 |
PradeepV | yes Haomeng | 05:47 |
Haomeng | PradeepV: user image did not depend on deploy images | 05:47 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor agent driver with pxe boot interface https://review.openstack.org/166521 | 05:47 |
Haomeng | PradeepV: so maybe confusing here, so you can read the guide and get idea | 05:48 |
Haomeng | PradeepV: this one - http://docs.openstack.org/developer/ironic/deploy/install-guide.html | 05:48 |
PradeepV | ok Haomeng: thank you :) | 05:49 |
Haomeng | PradeepV: selcome | 05:49 |
Haomeng | PradeepV: welcome | 05:50 |
*** Marga_ has quit IRC | 05:50 | |
*** VikasC has joined #openstack-ironic | 05:53 | |
*** ramineni has joined #openstack-ironic | 05:55 | |
*** stendulker has joined #openstack-ironic | 05:55 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: add links for UEFI secure boot support to iLO driver documentation https://review.openstack.org/216528 | 05:58 |
*** ramineni1 has joined #openstack-ironic | 05:58 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Fix ensures pass_deploy_info() always calls boot.prepare_instance() https://review.openstack.org/216756 | 05:59 |
*** ramineni has quit IRC | 05:59 | |
*** ramineni has joined #openstack-ironic | 06:01 | |
*** ramineni2 has joined #openstack-ironic | 06:02 | |
*** ramineni1 has quit IRC | 06:02 | |
*** ramineni1 has joined #openstack-ironic | 06:03 | |
*** lazy_prince has joined #openstack-ironic | 06:03 | |
*** ramineni has quit IRC | 06:05 | |
*** ramineni has joined #openstack-ironic | 06:06 | |
*** ramineni2 has quit IRC | 06:06 | |
*** ramineni2 has joined #openstack-ironic | 06:07 | |
*** ramineni1 has quit IRC | 06:08 | |
smoriya | naohirot: hi | 06:10 |
*** ramineni has quit IRC | 06:10 | |
naohirot | smoriya: Hi satoru! | 06:11 |
smoriya | naohirot: Thank you for reviewing. I replied your comment. | 06:11 |
*** ramineni2 has quit IRC | 06:11 | |
smoriya | naohirot: https://review.openstack.org/#/c/200496/17 | 06:12 |
smoriya | naohirot: ^^^^^^ | 06:12 |
naohirot | smoriya: Yes, I'm just about to reply to the latest your reply.:) | 06:12 |
naohirot | smoriya: I just push the button. | 06:13 |
smoriya | naohirot: Thanks. I got it just now :) | 06:13 |
naohirot | smoriya: s/push/pushded/ :) | 06:13 |
*** ramineni has joined #openstack-ironic | 06:14 | |
naohirot | sambetts: Yeah, I think #20 comments about standalone is much better than details steps. | 06:14 |
naohirot | sorry s/ sambetts/ smoriya / | 06:15 |
smoriya | naohirot: no problem ;) | 06:16 |
*** ramineni has quit IRC | 06:18 | |
*** saripurigopi has joined #openstack-ironic | 06:29 | |
*** Marga_ has joined #openstack-ironic | 06:29 | |
*** Marga_ has quit IRC | 06:29 | |
*** Marga_ has joined #openstack-ironic | 06:30 | |
*** achanda has quit IRC | 06:40 | |
openstackgerrit | Tan Lin proposed openstack/ironic: Use oslo.versionedobjects remotable decorators https://review.openstack.org/217000 | 06:44 |
*** rameshg87 has quit IRC | 06:47 | |
*** Sukhdev has quit IRC | 06:51 | |
*** e0ne has joined #openstack-ironic | 06:52 | |
openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 07:03 |
*** ifarkas has joined #openstack-ironic | 07:11 | |
sonuk | Hi all . kindly review https://review.openstack.org/#/c/213654/ | 07:18 |
*** davideagnello has joined #openstack-ironic | 07:22 | |
*** dims has joined #openstack-ironic | 07:24 | |
openstackgerrit | gaoxiaoyong proposed openstack/ironic: Revise "needs" to "need" in "/ironic/api/controllers/v1/types.py" Closes-Bug: #1488776 https://review.openstack.org/217007 | 07:25 |
openstack | bug 1488776 in Ironic "the content of the msg in the file "/ironic/api/controllers/v1/types.py" is incorrect" [Undecided,New] https://launchpad.net/bugs/1488776 - Assigned to gaoxiaoyong (gaoxiaoyong) | 07:25 |
*** dtantsur|afk is now known as dtantsur | 07:25 | |
dtantsur | Morning Ironic | 07:25 |
*** davideagnello has quit IRC | 07:26 | |
*** VikasC has quit IRC | 07:27 | |
*** dims has quit IRC | 07:29 | |
*** ukalifon1 has joined #openstack-ironic | 07:31 | |
*** Sukhdev has joined #openstack-ironic | 07:34 | |
*** Sukhdev_ has joined #openstack-ironic | 07:39 | |
*** Sukhdev has quit IRC | 07:39 | |
*** e0ne has quit IRC | 07:45 | |
*** ramineni has joined #openstack-ironic | 07:45 | |
*** ramineni1 has joined #openstack-ironic | 07:48 | |
*** ramineni has quit IRC | 07:50 | |
*** ramineni1 has quit IRC | 07:53 | |
*** Sukhdev_ has quit IRC | 07:55 | |
stendulker | Morning dtantsur :) | 07:58 |
dtantsur | o/ | 07:58 |
stendulker | dtantsur: Have a query... | 07:58 |
stendulker | dtantsur: We need to use parted command in config drive creation | 07:59 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Pass -w flag to iptables to make it wait for xtables lock https://review.openstack.org/216726 | 07:59 |
*** Marga_ has quit IRC | 07:59 | |
stendulker | dtantsur: There is a defect associated with the option that we want to use https://bugzilla.redhat.com/show_bug.cgi?id=217973 | 07:59 |
openstack | bugzilla.redhat.com bug 217973 in parted "Invalid mkpart size trunctations on gpt disklabels." [Low,Closed: currentrelease] - Assigned to dcantrell | 07:59 |
*** ramineni has joined #openstack-ironic | 08:01 | |
stendulker | dtantsur: It says its being fixed in RHEL-4. But fix is not available in Fedora 22 | 08:02 |
stendulker | dtantsur: Do the fixes done in RHEL gets ported to Fedora anytime? | 08:02 |
*** ramineni1 has joined #openstack-ironic | 08:03 | |
*** ramineni has left #openstack-ironic | 08:03 | |
dtantsur | stendulker, yes, it's usually expected. I think you can open/retarget bug to Fedora | 08:03 |
openstackgerrit | lei zhang proposed openstack/ironic: Add portType on port option https://review.openstack.org/217019 | 08:04 |
stendulker | dtantsur: The defect was fixed in RHEL in 2008 | 08:04 |
*** Marga_ has joined #openstack-ironic | 08:04 | |
dtantsur | stendulker, anyway you'd better ask the assignee :) | 08:05 |
*** ndipanov has quit IRC | 08:05 | |
*** ndipanov has joined #openstack-ironic | 08:06 | |
*** ndipanov has quit IRC | 08:06 | |
stendulker | dtantsur: ok. will do that | 08:06 |
stendulker | dtantsur: Thank you | 08:06 |
*** Marga_ has quit IRC | 08:06 | |
*** ramineni has joined #openstack-ironic | 08:06 | |
*** Marga_ has joined #openstack-ironic | 08:07 | |
*** ramineni1 has quit IRC | 08:07 | |
*** ramineni1 has joined #openstack-ironic | 08:09 | |
*** derekh has joined #openstack-ironic | 08:09 | |
*** VikasC has joined #openstack-ironic | 08:09 | |
*** ramineni2 has joined #openstack-ironic | 08:09 | |
*** ramineni has quit IRC | 08:10 | |
*** ramineni has joined #openstack-ironic | 08:11 | |
*** ramineni1 has quit IRC | 08:13 | |
*** VikasC has quit IRC | 08:13 | |
*** ramineni2 has quit IRC | 08:14 | |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Introduce tempest-lib to functional tests https://review.openstack.org/216701 | 08:14 |
*** ramineni1 has joined #openstack-ironic | 08:14 | |
*** ramineni has quit IRC | 08:16 | |
*** rameshg87 has joined #openstack-ironic | 08:16 | |
*** amotoki_ has joined #openstack-ironic | 08:18 | |
*** ramineni1 has quit IRC | 08:18 | |
*** lucasagomes has joined #openstack-ironic | 08:20 | |
*** dguerri` is now known as dguerri | 08:21 | |
*** VikasC has joined #openstack-ironic | 08:21 | |
*** ramineni has joined #openstack-ironic | 08:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Store and expose introspection data https://review.openstack.org/213159 | 08:25 |
*** romainh has joined #openstack-ironic | 08:27 | |
openstackgerrit | Naohiro Tamura proposed openstack/ironic: Follow up the nits in iRMC vmedia driver merged patch. https://review.openstack.org/217027 | 08:27 |
*** notq has quit IRC | 08:32 | |
*** ramineni has quit IRC | 08:32 | |
*** VikasC has quit IRC | 08:36 | |
*** jistr has joined #openstack-ironic | 08:39 | |
*** shoutm_ has quit IRC | 08:40 | |
*** yog_ has joined #openstack-ironic | 08:43 | |
*** PradeepV has quit IRC | 08:43 | |
*** pelix has joined #openstack-ironic | 08:43 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ensure that pass_deploy_info() always calls boot.prepare_instance() https://review.openstack.org/216756 | 09:04 |
*** romcheg has joined #openstack-ironic | 09:06 | |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Add cleanup in console utils tests https://review.openstack.org/216745 | 09:07 |
*** e0ne has joined #openstack-ironic | 09:08 | |
*** shoutm has joined #openstack-ironic | 09:12 | |
*** h00327910__ has joined #openstack-ironic | 09:14 | |
*** e0ne has quit IRC | 09:18 | |
*** MattMan has quit IRC | 09:19 | |
*** MattMan has joined #openstack-ironic | 09:21 | |
*** davideagnello has joined #openstack-ironic | 09:23 | |
*** e0ne has joined #openstack-ironic | 09:23 | |
lucasagomes | damn gerrit's so slow again | 09:23 |
*** daemontool_ has joined #openstack-ironic | 09:25 | |
*** zsmithnyc_ has joined #openstack-ironic | 09:25 | |
*** ndipanov has joined #openstack-ironic | 09:26 | |
dtantsur | yeah :( | 09:26 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Revise "needs" to "need" in "/ironic/api/controllers/v1/types.py" https://review.openstack.org/217007 | 09:26 |
tonyb | lucasagomes: Well while you wait for gerrit, can I pick your brains about the gate-tempest-dsvm-ironic-pxe_ipa test? | 09:27 |
lucasagomes | tonyb, sure | 09:27 |
lucasagomes | what's the story? | 09:27 |
*** davideagnello has quit IRC | 09:27 | |
sambetts | Morning all o/ | 09:27 |
rameshg87 | morning all | 09:27 |
rameshg87 | sambetts: congrats on becoming inspector-core :) | 09:28 |
tonyb | lucasagomes: well the short version is it's failing in stable/juno with "# [ERROR] /opt/stack/new/devstack/lib/ironic:656 Deploy kernel+ramdisk files don't exist and their building was disabled explicitly by IRONIC_BUILD_DEPLOY_RAMDISK" | 09:28 |
*** zsmithnyc has quit IRC | 09:28 | |
*** MattMan has quit IRC | 09:28 | |
sambetts | rameshg87: Thanks :) | 09:28 |
*** zsmithnyc_ is now known as zsmithnyc | 09:28 | |
*** daemontool__ has quit IRC | 09:28 | |
tonyb | lucasagomes: After a little digging it was failing in juno even while it was experimental | 09:28 |
Nisha | dtantsur, lucasagomes rameshg87 please could you review https://review.openstack.org/201904 (spec for inband inspection in ilo drivers) and https://review.openstack.org/198656(code patch for standalone ilo drivers ) | 09:28 |
tonyb | lucasagomes: so should stable/juno be gateing on that job? | 09:28 |
dtantsur | once I get some time (unlikely any soon)... | 09:28 |
*** MattMan has joined #openstack-ironic | 09:29 | |
dtantsur | sambetts, lucasagomes, morning; rameshg87, g'afternoon | 09:29 |
sambetts | dtantsur: changing it up eh? :-P | 09:29 |
lucasagomes | tonyb, /me thinks | 09:29 |
tonyb | lucasagomes: I stumbled across this trying the get https://review.openstack.org/#/c/173121/ to land | 09:29 |
rameshg87 | dtantsur: o/ | 09:30 |
lucasagomes | tonyb, ok no, Juno should not rely on that | 09:30 |
lucasagomes | tonyb, https://blueprints.launchpad.net/ironic/+spec/ipa-as-default-ramdisk | 09:30 |
tonyb | lucasagomes: kilo? | 09:30 |
lucasagomes | that job tests the pxe_* drivers with IPA | 09:30 |
lucasagomes | it was introduced in Kilo | 09:30 |
lucasagomes | tonyb, Kilo should do it yes | 09:30 |
lucasagomes | dtantsur, afternoon | 09:31 |
tonyb | okay I'll submit a project-config change and Cc you if that's okay? | 09:31 |
lucasagomes | tonyb, sure go for it! Thanks | 09:31 |
* dtantsur remembers he needs to disable inspector job on kilo & juno | 09:31 | |
tonyb | lucasagomes: smeet. | 09:31 |
tonyb | lucasagomes: Thanks for your help. | 09:32 |
lucasagomes | tonyb, yvw! | 09:32 |
*** ramineni has joined #openstack-ironic | 09:37 | |
*** goodygum has joined #openstack-ironic | 09:38 | |
*** dguerri is now known as dguerri` | 09:40 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add deprecation warning to periodic tasks with parallel=False https://review.openstack.org/216604 | 09:40 |
*** dtantsur is now known as dtantsur|brb | 09:45 | |
*** dims has joined #openstack-ironic | 09:48 | |
*** shoutm has quit IRC | 09:50 | |
*** marzif_ has joined #openstack-ironic | 09:55 | |
*** naohirot has quit IRC | 09:56 | |
*** chenglch has quit IRC | 10:01 | |
rameshg87 | lintan: hi | 10:01 |
rameshg87 | krotscheck: hi | 10:06 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: WIP: Impement take_over for ilo drivers https://review.openstack.org/217068 | 10:07 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Impement take_over for ilo drivers https://review.openstack.org/217068 | 10:10 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix logging and exceptions messages in ipminative driver https://review.openstack.org/217071 | 10:12 |
*** romainh has quit IRC | 10:12 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers https://review.openstack.org/217068 | 10:13 |
lintan | hi rameshg87 | 10:15 |
rameshg87 | lintan: hi | 10:16 |
rameshg87 | lintan: I had a question on https://review.openstack.org/#/c/213602/3 | 10:16 |
lintan | rameshg87, yes? | 10:16 |
rameshg87 | lintan: if there is a plan to drop access of attributes like node['key'], why modify the code to use oslo class now ? | 10:17 |
rameshg87 | lintan: can't we remove it straightaway ? | 10:17 |
*** Nisha has quit IRC | 10:19 | |
lintan | rameshg87, it's only a future plan I have to say. Because there will be a huge change in test case | 10:19 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Ensure that pass_deploy_info() always calls boot.prepare_instance() https://review.openstack.org/216756 | 10:19 |
rameshg87 | lintan: so I didn't get your reply on that review. | 10:20 |
rameshg87 | lintan: talking about https://review.openstack.org/#/c/213602/1/ironic/objects/base.py | 10:21 |
rameshg87 | lintan: in the first place, if we leave the code as it is | 10:21 |
rameshg87 | <without your patch> | 10:21 |
rameshg87 | lintan: it seems much easier and we need to change only objects/base.py to remove access to attributes like node['key'] | 10:22 |
rameshg87 | lintan: rather after your patch, we need to change 4 places to drop the support for access to attributes like node['key'] | 10:22 |
rameshg87 | lintan: so I didn't get the intention of your patch still | 10:22 |
lintan | rameshg87, I mean we don't have to change these 4 places at the same time, we can change node's behavior and it's test in one patch | 10:23 |
lintan | rameshg87, then a follow patch about port | 10:24 |
rameshg87 | lintan: but then, why not drop the support from the current state if we are dropping it's support in the future | 10:24 |
rameshg87 | lintan: that seemed much simpler the me. | 10:24 |
rameshg87 | lintan: node is the heavily used resource in our code followed by port. the rest should be trivial | 10:25 |
openstackgerrit | lei zhang proposed openstack/ironic: Add portType on port option https://review.openstack.org/217019 | 10:25 |
lintan | rameshg87, I get your point, but let me estimate the workload to see which one is better, because I want to migrate to oslo library as soon as possible to avoid yesterday's situation. | 10:27 |
*** notq has joined #openstack-ironic | 10:28 | |
*** dlpartain has joined #openstack-ironic | 10:32 | |
*** dlpartain has left #openstack-ironic | 10:33 | |
rameshg87 | lintan: yeah okay | 10:34 |
rameshg87 | lintan: even I want to not unnecessarily go somewhere if we are removing it off anyway | 10:34 |
*** pal has joined #openstack-ironic | 10:44 | |
ramineni | rameshg87: addressed your comments on https://review.openstack.org/#/c/205895/, have a look when you have time | 10:46 |
rameshg87 | ramineni: ack | 10:47 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Add links for UEFI secure boot support to iLO driver documentation https://review.openstack.org/216528 | 10:57 |
* rameshg87 will be back after a while | 11:01 | |
*** rameshg87 has quit IRC | 11:01 | |
*** ramineni has left #openstack-ironic | 11:11 | |
*** dtantsur|brb is now known as dtantsur | 11:13 | |
*** davideagnello has joined #openstack-ironic | 11:24 | |
*** dprince has joined #openstack-ironic | 11:24 | |
*** pal has quit IRC | 11:27 | |
sonuk | https://review.openstack.org/#/c/213654/ | 11:28 |
*** davideagnello has quit IRC | 11:28 | |
*** lucasagomes is now known as lucas-hungry | 11:34 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface https://review.openstack.org/216538 | 11:37 |
*** Haomeng|2 has joined #openstack-ironic | 11:37 | |
*** VikasC has joined #openstack-ironic | 11:37 | |
dtantsur | jroll, JoshNang, any of you folks every encountered anything like https://bugs.launchpad.net/nova/+bug/1341420 ? | 11:38 |
openstack | Launchpad bug 1341420 in OpenStack Compute (nova) "gap between scheduler selection and claim causes spurious failures when the instance is the last one to fit" [High,Confirmed] | 11:38 |
*** saripurigopi has quit IRC | 11:39 | |
dtantsur | maybe you didn't, as I doubt you even get close to using the last few machines... | 11:39 |
*** Haomeng has quit IRC | 11:40 | |
*** VikasC has quit IRC | 11:47 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface https://review.openstack.org/217102 | 11:47 |
*** dims_ has joined #openstack-ironic | 11:48 | |
*** romainh has joined #openstack-ironic | 11:51 | |
*** dims has quit IRC | 11:52 | |
*** h00327910__ has quit IRC | 11:53 | |
TheJulia | good morning | 11:54 |
*** amotoki has quit IRC | 11:54 | |
dtantsur | TheJulia, morning | 12:01 |
*** VikasC has joined #openstack-ironic | 12:04 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add vendor interface to ipminative driver https://review.openstack.org/215185 | 12:04 |
*** trown|outttypeww is now known as trown | 12:09 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface https://review.openstack.org/217102 | 12:09 |
*** thrash|g0ne is now known as thrash | 12:12 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent: Add support for inspection using ironic-inspector https://review.openstack.org/205587 | 12:12 |
*** stendulker has quit IRC | 12:14 | |
openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add API Discovery to Ironic Inspector https://review.openstack.org/215067 | 12:19 |
*** VikasC has quit IRC | 12:23 | |
*** shoutm has joined #openstack-ironic | 12:23 | |
*** nicodemos has joined #openstack-ironic | 12:23 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct bug with with default value where required https://review.openstack.org/217109 | 12:24 |
*** linggao has joined #openstack-ironic | 12:25 | |
*** smoriya has quit IRC | 12:28 | |
*** pal has joined #openstack-ironic | 12:36 | |
*** lucas-hungry is now known as lucasagomes | 12:37 | |
*** jjohnson2 has joined #openstack-ironic | 12:37 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Correct bug with with default value where required https://review.openstack.org/217109 | 12:39 |
*** jjohnson2 has quit IRC | 12:45 | |
*** jjohnson2 has joined #openstack-ironic | 12:46 | |
*** pal has quit IRC | 12:58 | |
*** mtanino has quit IRC | 12:59 | |
*** pal has joined #openstack-ironic | 13:01 | |
*** irenab has quit IRC | 13:03 | |
*** yog_ has quit IRC | 13:03 | |
*** irenab has joined #openstack-ironic | 13:05 | |
*** puranamr has joined #openstack-ironic | 13:08 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Fix logging and exceptions messages in ipminative driver https://review.openstack.org/217071 | 13:09 |
*** irenab has quit IRC | 13:13 | |
*** links has quit IRC | 13:15 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Add vendor interface to ipminative driver https://review.openstack.org/215185 | 13:19 |
*** coreycb has joined #openstack-ironic | 13:23 | |
*** davideagnello has joined #openstack-ironic | 13:24 | |
coreycb | I'm not seeing a tarball for the 4.0.0 release on launchpad. will that be available soon? | 13:24 |
dtantsur | I think it's still in process of getting released | 13:25 |
coreycb | dtantsur, ok I'll keep an eye out for it | 13:25 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Add introspection rules support https://review.openstack.org/208375 | 13:25 |
*** irenab has joined #openstack-ironic | 13:25 | |
*** thiagop has joined #openstack-ironic | 13:28 | |
*** davideagnello has quit IRC | 13:29 | |
*** thrash has quit IRC | 13:30 | |
*** thrash has joined #openstack-ironic | 13:31 | |
*** puranamr has quit IRC | 13:33 | |
*** irenab has quit IRC | 13:33 | |
*** rloo has joined #openstack-ironic | 13:34 | |
NobodyCam | good morning Ironicers | 13:34 |
sambetts | Morning NobodyCam o/ | 13:35 |
dtantsur | NobodyCam, morning | 13:35 |
NobodyCam | morning dtantsur sambetts lucasagomes rloo TheJulia JayF JoshNang jlvillal and everyone else not list here :) | 13:35 |
NobodyCam | hehehehe | 13:35 |
NobodyCam | :) | 13:36 |
rloo | hi and morning NobodyCam and everyone on his list :) | 13:36 |
*** cdearborn has joined #openstack-ironic | 13:36 | |
dtantsur | rloo, morning you too :) | 13:36 |
sambetts | o/ rloo | 13:36 |
lucasagomes | NobodyCam, sambetts rloo good ugt morning! | 13:37 |
thiagop | good morning Ironic | 13:37 |
rloo | looks like you've all been busy. so many emails this morning! | 13:37 |
thiagop | morning NobodyCam rloo lucasagomes dtantsur sambetts | 13:37 |
dtantsur | thiagop, o/ | 13:37 |
lucasagomes | thiagop, hi there! | 13:37 |
*** yog_ has joined #openstack-ironic | 13:38 | |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 13:38 |
NobodyCam | morning thiagop :) | 13:40 |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 13:40 |
*** irenab has joined #openstack-ironic | 13:43 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update doc about IPA image create to install grub2 https://review.openstack.org/217137 | 13:45 |
trown | good ugt morning NobodyCam sambetts dtantsur rloo lucasagomes thiagop | 13:48 |
thiagop | trown: o/ | 13:48 |
dtantsur | trown, morning | 13:48 |
*** albertoffb has joined #openstack-ironic | 13:49 | |
lucasagomes | trown, morning! | 13:49 |
trown | sambetts: good point on https://review.openstack.org/215067 | 13:50 |
trown | I am not sure the ../data link should be there, maybe it should not | 13:51 |
sambetts | trown: I think I'm going to modify it to not include it for now, and we can extend it later if we need it | 13:52 |
trown | maybe we just need to change the regex to something like "^\/%s\/.*$\/" | 13:52 |
trown | sambetts: ya +1 to that | 13:52 |
*** irenab has quit IRC | 13:52 | |
trown | oops moved the $ sign though | 13:53 |
*** puranamr has joined #openstack-ironic | 13:53 | |
trown | sambetts: I think it would still be good to include a link like that in the UT and show it is not included | 13:54 |
sambetts | trown: Yup will do :D | 13:54 |
trown | thanks! | 13:54 |
*** irenab has joined #openstack-ironic | 13:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/217150 | 13:56 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-lib: Updated from global requirements https://review.openstack.org/216966 | 13:56 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/217151 | 13:56 |
*** shoutm has quit IRC | 13:56 | |
*** irenab has quit IRC | 13:56 | |
*** irenab has joined #openstack-ironic | 13:57 | |
*** puranamr has quit IRC | 13:57 | |
*** mtanino has joined #openstack-ironic | 13:58 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Rename plugin root_device_hint -> raid_device https://review.openstack.org/217171 | 14:00 |
dtantsur | ifarkas, for your attention ^^ | 14:00 |
*** alexpilotti has joined #openstack-ironic | 14:01 | |
*** shoutm has joined #openstack-ironic | 14:03 | |
rloo | oh hey, sambetts, congrats! :) | 14:04 |
*** alexpilotti has quit IRC | 14:04 | |
*** baoli has joined #openstack-ironic | 14:04 | |
*** dlpartain has joined #openstack-ironic | 14:05 | |
*** links has joined #openstack-ironic | 14:05 | |
*** Haomeng|2 has quit IRC | 14:05 | |
ifarkas | dtantsur, thanks, will take a look | 14:09 |
*** rloo_ has joined #openstack-ironic | 14:09 | |
sambetts | rloo: Thanks :D | 14:10 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironic-inspector-client: Updated from global requirements https://review.openstack.org/217203 | 14:10 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/217204 | 14:10 |
jlvillal | Good morning dtantsur TheJulia NobodyCam sambetts rloo thiagop lucasagomes trown and anyone else I missed | 14:12 |
*** rloo has quit IRC | 14:13 | |
dtantsur | jlvillal, morning | 14:13 |
trown | o/ jlvillal | 14:13 |
thiagop | morning jlvillal | 14:13 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Periodically checks for nodes being cleaned https://review.openstack.org/213699 | 14:14 |
*** sonuk has quit IRC | 14:14 | |
lucasagomes | jlvillal, hi there | 14:15 |
*** rloo_ is now known as rloo | 14:17 | |
rloo | morning jlvillal | 14:17 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update doc about IPA image create to install grub2 https://review.openstack.org/217137 | 14:20 |
* lucasagomes forgot the commit message | 14:20 | |
*** rloo has quit IRC | 14:21 | |
*** rloo has joined #openstack-ironic | 14:21 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Update documentation to install grub2 when creating the user image https://review.openstack.org/217137 | 14:21 |
*** shoutm has quit IRC | 14:22 | |
*** lifeless1 has joined #openstack-ironic | 14:23 | |
*** lifeless has quit IRC | 14:24 | |
aarefiev | hi folks, could somebody take a look on: https://review.openstack.org/#/c/217135/ please, I'm trying to fix ironicclient functional job | 14:25 |
openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add API Discovery to Ironic Inspector https://review.openstack.org/215067 | 14:25 |
*** dlpartain has left #openstack-ironic | 14:27 | |
*** baoli has quit IRC | 14:28 | |
rloo | back in about an hour | 14:30 |
*** jcoufal has joined #openstack-ironic | 14:30 | |
*** rloo has quit IRC | 14:30 | |
*** baoli_ has joined #openstack-ironic | 14:31 | |
*** yog_ has quit IRC | 14:34 | |
jlvillal | lucasagomes, Since you setup the py34 IPA tests as non-voting. I added you to the patch to make them voting: https://review.openstack.org/#/c/216886/ | 14:35 |
jlvillal | aarefiev, I'll look at it later today when I get to work. Thanks for working on functional testing :) | 14:37 |
*** mtanino has quit IRC | 14:37 | |
aarefiev | jlvillal: thanks | 14:37 |
jlvillal | aarefiev, If you get it working, we might want to move the test job from experimental to non-voting | 14:38 |
aarefiev | jlvillal: sure, but there are only 2 tests( | 14:38 |
jlvillal | aarefiev, Even easier then :) | 14:38 |
aarefiev | jlvillal: :) | 14:39 |
jlvillal | aarefiev, If it is experimental it will be easy for the tests to break. Since they won't get run very often. | 14:39 |
aarefiev | jlvillal: I'm agree with you, since we add some tests we should make it non voting | 14:40 |
aarefiev | jlvillal: glad to see somebody interested in functional tests, thanks | 14:42 |
jlvillal | aarefiev, I am also glad to see someone interested in functional tests. krtaylor is also interested I think. | 14:43 |
dtantsur | functional tests rule :) | 14:43 |
aarefiev | dtantsur: :) | 14:43 |
*** mtanino has joined #openstack-ironic | 14:43 | |
dtantsur | we've been used them in inspector since long time | 14:44 |
* jlvillal needs to learn a lot more on them. | 14:44 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support https://review.openstack.org/208375 | 14:46 |
lucasagomes | jlvillal, ack! Thanks | 14:48 |
lucasagomes | it's been running consistently right? | 14:48 |
jlvillal | lucasagomes, From what I saw it is. It caught an error in my patch :) | 14:48 |
lucasagomes | cool | 14:48 |
lucasagomes | +1'd | 14:48 |
jlvillal | lucasagomes, WSME 0.8 = grumble, grumble from me :P | 14:48 |
*** tsekiyama has joined #openstack-ironic | 14:49 | |
dtantsur | hehe | 14:49 |
lucasagomes | jlvillal, heh | 14:49 |
lucasagomes | well at least WSME project is alive again | 14:49 |
lucasagomes | we are getting things fixed heh | 14:49 |
jlvillal | lucasagomes, That is good :) | 14:49 |
dtantsur | folks, does someone feel like overtaking https://review.openstack.org/#/c/209012/ from me? it must be very helpful for testing, but I don't have any time... | 14:50 |
dtantsur | after we land this, we can finally have $ ironic version-list call | 14:50 |
dtantsur | and something like $ ironic version-supported 1.11 | 14:51 |
dtantsur | which would allow us to write sane code in devstack (see comments on https://review.openstack.org/#/c/206055/) | 14:51 |
*** baoli_ has quit IRC | 14:57 | |
*** baoli_ has joined #openstack-ironic | 14:58 | |
*** baoli_ has quit IRC | 14:59 | |
*** links has quit IRC | 14:59 | |
*** baoli_ has joined #openstack-ironic | 15:00 | |
*** albertoffb has quit IRC | 15:00 | |
*** baoli_ has quit IRC | 15:00 | |
krotscheck | rameshg87: Thanks for your response. Let me look into what's going on there, I know that a more recent version of the config generator picks up middleware things. | 15:01 |
*** baoli_ has joined #openstack-ironic | 15:01 | |
*** asd112z has joined #openstack-ironic | 15:02 | |
*** Marga_ has quit IRC | 15:02 | |
*** r-daneel has joined #openstack-ironic | 15:07 | |
*** thrash has quit IRC | 15:08 | |
*** marzif_ has quit IRC | 15:08 | |
*** thrash has joined #openstack-ironic | 15:09 | |
*** boris-42 has quit IRC | 15:10 | |
*** asd112z has quit IRC | 15:10 | |
*** asd112z has joined #openstack-ironic | 15:11 | |
*** jistr is now known as jistr|mtg | 15:12 | |
*** bnemec has joined #openstack-ironic | 15:17 | |
*** vishwanathj has joined #openstack-ironic | 15:19 | |
*** Marga_ has joined #openstack-ironic | 15:19 | |
*** jistr|mtg is now known as jistr | 15:22 | |
*** irenab has quit IRC | 15:24 | |
*** irenab has joined #openstack-ironic | 15:24 | |
*** Marga_ has quit IRC | 15:25 | |
*** davideagnello has joined #openstack-ironic | 15:25 | |
*** Marga_ has joined #openstack-ironic | 15:25 | |
*** rloo has joined #openstack-ironic | 15:29 | |
*** Sukhdev has joined #openstack-ironic | 15:30 | |
*** davideagnello has quit IRC | 15:30 | |
devananda | g'morning, all | 15:32 |
dtantsur | devananda, morning | 15:32 |
*** Marga_ has quit IRC | 15:33 | |
NobodyCam | good morning devananda :) | 15:33 |
*** Marga_ has joined #openstack-ironic | 15:34 | |
NobodyCam | rloo: congratz :) | 15:34 |
rloo | NobodyCam: thx, I think :) | 15:34 |
NobodyCam | hehehehe :) | 15:34 |
dtantsur | :) | 15:35 |
*** pal has quit IRC | 15:36 | |
sambetts | Morning devananda o/ | 15:37 |
thiagop | mornign devananda | 15:38 |
*** ukalifon1 has quit IRC | 15:41 | |
rloo | hi dtantsur, i can take a gander at https://review.openstack.org/#/c/209012/ if you want. Although I can't recall now whether I agreed with it or not ;) | 15:41 |
*** Marga_ has quit IRC | 15:42 | |
*** lucasagomes has quit IRC | 15:42 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: New driver interface for BIOS configuration https://review.openstack.org/209612 | 15:44 |
*** lucasagomes has joined #openstack-ironic | 15:44 | |
dtantsur | rloo, yes please :) I don't care how it looks in the end, but we need some way to communicate information about versions. We probably could use only headers, but I decided to do what nova does.. | 15:44 |
-openstackstatus- NOTICE: restarting gerrit due to a slow memory leak | 15:44 | |
dtantsur | wow, wow | 15:44 |
dtantsur | probably good time to go home then :D | 15:44 |
NobodyCam | lol | 15:45 |
NobodyCam | have a good night dtantsur :) | 15:45 |
*** dtantsur is now known as dtantsur|afk | 15:45 | |
*** mtanino has quit IRC | 15:45 | |
rloo | dtantsur: ok, will look at it today. | 15:45 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: format links in the readme to work with the release notes tools https://review.openstack.org/216758 | 15:46 |
*** MattMan1 has joined #openstack-ironic | 15:47 | |
*** davideagnello has joined #openstack-ironic | 15:48 | |
*** mtanino has joined #openstack-ironic | 15:48 | |
*** davideagnello has quit IRC | 15:49 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 15:49 |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Intel Node Manager policies support https://review.openstack.org/214657 | 15:50 |
*** ionutbalutoiu has joined #openstack-ironic | 15:52 | |
krotscheck | braaaains | 15:52 |
devananda | lucasagomes: ping | 15:52 |
devananda | krotscheck: raaaaaaaawr! | 15:52 |
*** blakec has joined #openstack-ironic | 15:52 | |
lucasagomes | devananda, pong | 15:52 |
thiagop | ??? | 15:52 |
*** Marga_ has joined #openstack-ironic | 15:54 | |
devananda | rloo: you are now specs core! welcome to the dark^D^D^D^D read-all-the-things team! | 15:54 |
rloo | devananda: thx. Although I don't promise to read-all-the-things, just some-of-them :) | 15:55 |
rloo | devananda: I figure if we 'pull our own weight' I am lighter than most of the other cores, so ... | 15:55 |
devananda | heh | 15:56 |
lazy_prince | devananda: for the ironic neutron integration, there seems to be an issue with the context used for port-create call.. | 15:56 |
lazy_prince | but i believe, the ports are being created during cleaning network.. any idea on what context is being used and where is that code to use the admin context..? | 15:57 |
*** mtanino_ has joined #openstack-ironic | 15:57 | |
devananda | lazy_prince: I heard a little about that on monday from Sukhdev. can I help unstick it? | 15:57 |
devananda | lazy_prince: does the call to neutron need to be sent with an admin context, or with a user context? | 15:57 |
lazy_prince | devananda: for provisioning network and cleaning network, i believe, it should be admin context as these network would be in admin project or service project.. | 15:58 |
lazy_prince | for tenant, it could be user context.. | 15:59 |
devananda | ok - should be easy to generate an admin context for those | 15:59 |
devananda | however, at the point it needs to be put on the tenant network, yea, we might have a problem | 15:59 |
devananda | because Ironic can't generate the user context | 15:59 |
*** mtanino has quit IRC | 16:00 | |
*** dims_ has quit IRC | 16:00 | |
sambetts | The admin context should be able to do it on behalf of a user | 16:00 |
lazy_prince | I believe, its just activating the port for tenant as the port will already be generated by nova.. | 16:00 |
devananda | ooh. cool | 16:01 |
Sukhdev | lazy_prince devananda: here is an example how I use admin context to access neutron stuff (as well nova), but, had hard time dealing with Ironic - so, there is something different there - https://github.com/openstack/networking-arista/blob/master/networking_arista/common/db_lib.py#L335 | 16:01 |
*** dims has joined #openstack-ironic | 16:01 | |
*** Marga_ has quit IRC | 16:02 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: Put py34 first in the env order of tox https://review.openstack.org/217283 | 16:02 |
devananda | from oslo_context import context | 16:02 |
devananda | a_c = context.get_admin_context() | 16:02 |
lazy_prince | My understanding is that nova will still create the tenant port on nova boot and ironic will activate it on successful provisioning... and that activation can be done with admin context.. | 16:03 |
devananda | that should work anywhere inside the Ironic conductor service | 16:03 |
lazy_prince | cool.. then I will just give it a try.. | 16:03 |
*** meghal has joined #openstack-ironic | 16:06 | |
*** lucasagomes is now known as lucas-afk | 16:10 | |
*** jistr has quit IRC | 16:12 | |
*** romainh has quit IRC | 16:12 | |
krotscheck | I'm digging into the generate_sample tool right now. I see some bits where it goes through and searches for libraries, is there some way to teach it to pick up namespaces as well? | 16:16 |
*** lucas-afk is now known as lucasagomes | 16:17 | |
devananda | dtantsur|afk, lucasagomes, anyone else interested in how we will begin generating release notes -- please see this email thread http://lists.openstack.org/pipermail/openstack-dev/2015-August/072943.html | 16:19 |
* lucasagomes looks | 16:20 | |
krotscheck | Where does generate_sample come from? It seems like a vendored thing. | 16:23 |
devananda | krotscheck: context? | 16:25 |
*** albertoffb has joined #openstack-ironic | 16:26 | |
krotscheck | oslo_middleware config things do not get output into ironic.conf.sample. This seems to be a problem with ./tools/config/generate_sample.sh, because it works fine if I do oslo-config-generator --namespace oslo.middleware - I'm trying to figure out who owns that code, what a good approach to fix it would be, and whether I can get it done before I disappear. | 16:26 |
devananda | krotscheck: have you added oslo_middleware to the list of odules included in the scan? | 16:27 |
devananda | krotscheck: see oslo.config.generator.rc | 16:28 |
devananda | in tools/config/ | 16:28 |
krotscheck | devananda: No. I barely even understand how this thing works. | 16:28 |
lucasagomes | devananda, interesting, this has been discussed on the ML only? Or some TC meeting as well? | 16:28 |
lucasagomes | seems there are some parallel discussions going on outside that email thread | 16:28 |
devananda | lucasagomes: yes, TC stuff, as this affects many projects. Ironic is, once again, the first one to actually be doing it though | 16:29 |
lucasagomes | devananda, I see, I will keep an eye on the thread | 16:29 |
devananda | krotscheck: add the module you want configs written out for, to the list in that .rc file, and viola! | 16:30 |
lucasagomes | that's good we are trying something new tho, it's not that other projects have very good release cycles with all those code freezes, and spec freeze, foo freeze | 16:30 |
devananda | "foo freeze" << I like that. can we have one? | 16:31 |
*** endorphin has joined #openstack-ironic | 16:31 | |
lucasagomes | devananda, no freeze for us :-) | 16:31 |
krotscheck | devananda: "Voila, it included config options from all libraries in oslo middleware, not just the ones we're using!" | 16:33 |
devananda | krotscheck: you can specify a full module path | 16:33 |
devananda | but yes, it'll pull in all the configs for that (sub)module | 16:33 |
krotscheck | devananda: I seem to have hit a bug. | 16:36 |
devananda | awesome | 16:36 |
krotscheck | devananda: Yep. It lost the group defenition. | 16:36 |
*** Nisha_away has joined #openstack-ironic | 16:38 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Stop providing 'message' property in IronicException https://review.openstack.org/215362 | 16:42 |
jlvillal | JoshNang: Would you agree that Serializable objects are immutable? | 16:44 |
JoshNang | jlvillal: not generally, no | 16:44 |
jlvillal | JoshNang: Really? Hmmm. Do you have an example of where they get modified? | 16:45 |
JoshNang | let me look at IPA's case | 16:45 |
jlvillal | JoshNang: I'll go poke around. And yes in particular for IPA's case | 16:45 |
JoshNang | i think the original idea was to attach that to anything you wanted passed back the API. We haven't followed through on that, for sure | 16:46 |
*** derekh has quit IRC | 16:46 | |
jlvillal | JoshNang: Okay, I see an example of it changing in a derived class. Thanks for your help. | 16:47 |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Add release notes for 4.0.0 https://review.openstack.org/216843 | 16:47 |
JoshNang | jlvillal: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/hardware.py#L73 as well | 16:47 |
JoshNang | (lldp info likely wouldn't be available when creating that object) | 16:47 |
lucasagomes | folks I'm going to call it a day | 16:47 |
JoshNang | np! | 16:47 |
jlvillal | JoshNang: Thanks | 16:47 |
lucasagomes | have a great night! | 16:47 |
lucasagomes | see y'all | 16:47 |
jlvillal | lucasagomes: Ciao | 16:47 |
JoshNang | lucasagomes: o/ | 16:47 |
lucasagomes | see ya :-) | 16:48 |
*** lucasagomes is now known as lucas-dinner | 16:48 | |
jlvillal | JoshNang: I was assuming immutable after __init__. But I see cases where things are modified in derived classes. | 16:48 |
JoshNang | jlvillal: ah :/ sorry | 16:48 |
jlvillal | So I can't think of it as immutable. | 16:48 |
jlvillal | JoshNang: Trying to figure out the best way to do __hash__ Make sure I do it correctly :) | 16:49 |
* jlvillal is learning new things every day :) | 16:49 | |
JayF | `1212 | 16:49 |
JayF | sorry about that, my console had frozen | 16:49 |
JoshNang | heh | 16:49 |
JoshNang | jlvillal: me too! first time i saw the docs that said you should do __ne__ any time you do __eq__ (which makes sense) | 16:50 |
jlvillal | JoshNang: Yeah, and now I have to do __hash__ because of that.... Lucky me! :D | 16:50 |
krotscheck | Yet. Another. Hoop. | 16:51 |
*** afaranha has quit IRC | 16:51 | |
*** meghal has quit IRC | 16:51 | |
devananda | krotscheck: so you're saying it's a series of hoops? | 16:52 |
*** achanda has joined #openstack-ironic | 16:52 | |
devananda | and here, i thought it was a series of tubes ... | 16:52 |
TheJulia | if it is a series of hops, lots of documentation and memes need to be updated... pronto | 16:53 |
openstackgerrit | Merged openstack/ironic-specs: Update raid config spec to add one more RPC API https://review.openstack.org/215431 | 16:53 |
*** trown is now known as trown|lunch | 16:53 | |
*** davideagnello has joined #openstack-ironic | 16:55 | |
openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Out-of-band hardware inspection support for Cisco B/C/M-series servers https://review.openstack.org/204733 | 16:56 |
*** meghal has joined #openstack-ironic | 16:57 | |
*** MattMan1 has left #openstack-ironic | 16:58 | |
*** baoli_ has quit IRC | 16:59 | |
*** baoli has joined #openstack-ironic | 16:59 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic-python-agent: Add some comparison operators to Serializable class https://review.openstack.org/215854 | 17:04 |
*** boris-42 has joined #openstack-ironic | 17:07 | |
krotscheck | Ok, I'm done. Yet another bug, yet another "Oh I am concerned about a thing and wills tand in the way of you landing this feature" | 17:08 |
krotscheck | And now I have to go run around and go fix oslo for the upteenth time. | 17:08 |
*** meghal has quit IRC | 17:09 | |
krotscheck | I am done. | 17:09 |
*** meghal has joined #openstack-ironic | 17:09 | |
krotscheck | If you want CORS, you can do it yourself. | 17:09 |
*** meghal has quit IRC | 17:10 | |
*** meghal has joined #openstack-ironic | 17:10 | |
*** meghal has quit IRC | 17:10 | |
*** meghal has joined #openstack-ironic | 17:11 | |
*** meghal has quit IRC | 17:11 | |
*** meghal has joined #openstack-ironic | 17:12 | |
*** romcheg has quit IRC | 17:12 | |
*** romcheg has joined #openstack-ironic | 17:13 | |
*** e0ne has quit IRC | 17:15 | |
*** meghal has quit IRC | 17:17 | |
*** meghal has joined #openstack-ironic | 17:17 | |
clif_h | jlvillal: question regarding https://review.openstack.org/#/c/215362/, you say that one of the preferred ways to get a message from an exception is exc.args[0], is there a source for this? I haven't found mention of it yet in the Python 3.4.3 documentation. Is there something that explicitly states that exc.args[0] is /always/ a string description of the error? | 17:21 |
*** lazy_prince has quit IRC | 17:21 | |
jlvillal | clif_h: I'd have to go Googling to find it. But I'm pretty sure it is | 17:22 |
clif_h | Statements like this: https://docs.python.org/3/library/exceptions.html?highlight=args#BaseException.args talk about args explicitly, but don't seem to place the requirement that args[0] is the message | 17:23 |
clif_h | It may be prevailing convention, but I haven't found anything that explicitly states the requirement | 17:24 |
jlvillal | clif_h: Probably the safest way is to use str() | 17:24 |
devananda | krotscheck: you're giving me the sad. what can I do to help? | 17:24 |
devananda | because I want CORS and I want you to be happy and these shouldn't be mutually exclusive | 17:25 |
devananda | krotscheck: how about a curated sample config file specifically for the CORS options, and some documentation saying "if you want to do X, you need to append <this> file to your config" | 17:26 |
devananda | or something | 17:26 |
clif_h | jlvillal: It's kind of nitty, but I figured now would be a good time to read about differences in exception handling in Python 3, and your comment about exc.args[0] made me wonder if that is convention or actually explicitly stated by docs | 17:27 |
dims | devananda: dhellmann is taking care of splitting options in oslo_middleware into fine grained to get you what you need. | 17:27 |
jlvillal | clif_h: Please let me know what you find out! | 17:27 |
*** baoli has quit IRC | 17:27 | |
openstackgerrit | Merged openstack/ironic-python-agent: Updated from global requirements https://review.openstack.org/217151 | 17:27 |
clif_h | jlvillal: well, I'm saying I haven't been able to find any mention of args[0] being required as the message. | 17:28 |
clif_h | I think I've looked through all the relevant portions of the Python 3 docs | 17:28 |
jlvillal | clif_h: What is see is that 'usually' it is the message: https://docs.python.org/3/library/exceptions.html | 17:28 |
clif_h | It's fine if Ironic makes that a requirement for its exceptions | 17:28 |
jlvillal | clif_h: If you look at the code, Ironic is using args[0] | 17:29 |
*** davideagnello has quit IRC | 17:29 | |
clif_h | but we should probably be clear that its an Ironic convention | 17:29 |
jlvillal | clif_h: Before my patch | 17:29 |
clif_h | agreed, I see that | 17:29 |
jlvillal | I can just change the comment to drop the args[0] part as that is more an implementation detail | 17:30 |
jlvillal | s/comment/commit message/ | 17:30 |
*** Sukhdev has quit IRC | 17:30 | |
*** davideagnello has joined #openstack-ironic | 17:30 | |
clif_h | I suppose, this all feels supper nitty to me, but I wondered nonetheless | 17:30 |
clif_h | I expect this is probably the convention in the vast majority of projects | 17:30 |
jlvillal | clif_h: I think str(exc) is the way people should be doing. | 17:31 |
*** Sukhdev has joined #openstack-ironic | 17:31 | |
jlvillal | clif_h: I'm just not sure if all of our unit tests are done that way. From vague recollection of the Python 3 work. | 17:31 |
clif_h | agreed, that seems to be the preferred/safest way | 17:31 |
*** baoli_ has joined #openstack-ironic | 17:32 | |
jlvillal | clif_h: git grep 'args\[0\]' | 17:32 |
dims | devananda: https://review.openstack.org/#/c/199769/ is probably what bothered krotscheck | 17:32 |
* devananda looks | 17:32 | |
jlvillal | clif_h: Or maybe better: git grep '\.args\[0\]' | 17:32 |
dims | devananda: if other than config option its ok then should let that in and fix the config option after https://review.openstack.org/217316 lands in oslo.middleware | 17:33 |
* clif_h looks | 17:33 | |
clif_h | there is some code that reaches into args[0] explicitly outside of exceptions/test code | 17:33 |
devananda | dims: it's fine aside from the changes to ironic.conf.sample, which we can't reproduce right now | 17:33 |
devananda | dims: I've suggested putting htem in a separate file as a work around, until it can be regenerated properly | 17:34 |
devananda | suppose I'll just do that ... | 17:34 |
clif_h | jlvillal: should we patch those instances to use str(exc) instead? | 17:34 |
dims | devananda: ack | 17:34 |
jlvillal | clif_h: Probably | 17:34 |
dims | devananda: thanks | 17:35 |
* jlvillal hopes clif_h is volunteering :) | 17:35 | |
clif_h | I will, since it shouldn't take long | 17:35 |
* clif_h crosses fingers | 17:35 | |
jlvillal | clif_h: The only one that should not change is exceptions.py, since it is implementing the __str__ function. | 17:35 |
JoshNang | clif_h: famous last words | 17:35 |
clif_h | JoshNang: indeed | 17:35 |
jlvillal | JoshNang: Yeah, what I thought about the __eq__ patch for IPA :) | 17:36 |
clif_h | jlvillal: agreed | 17:36 |
*** ifarkas has quit IRC | 17:36 | |
JoshNang | jlvillal: heh | 17:36 |
jlvillal | clif_h: I wonder if should get rid or deprecate the 'format_message' call. I don't quite understand the need for it. | 17:38 |
*** achanda_ has joined #openstack-ironic | 17:39 | |
jlvillal | clif_h: There is one use of 'format_message', maybe you could also change that to use str() | 17:39 |
*** achanda has quit IRC | 17:39 | |
*** praneshp has joined #openstack-ironic | 17:39 | |
clif_h | I will take a look at that as well | 17:39 |
jlvillal | clif_h: Thanks! | 17:39 |
jlvillal | clif_h: I'll push a new patch with different commit message | 17:40 |
clif_h | jlvillal: thank you | 17:41 |
*** dims_ has joined #openstack-ironic | 17:41 | |
*** dims has quit IRC | 17:42 | |
*** meghal has quit IRC | 17:47 | |
*** baoli_ has quit IRC | 17:47 | |
*** trown|lunch is now known as trown | 17:47 | |
*** achanda_ has quit IRC | 17:48 | |
*** achanda has joined #openstack-ironic | 17:51 | |
*** pelix has quit IRC | 17:51 | |
*** meghal has joined #openstack-ironic | 18:01 | |
*** puranamr has joined #openstack-ironic | 18:04 | |
*** baoli has joined #openstack-ironic | 18:06 | |
*** absubram has joined #openstack-ironic | 18:07 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Stop providing 'message' property in IronicException https://review.openstack.org/215362 | 18:07 |
jlvillal | clif_h: ^^ | 18:08 |
*** meghal has quit IRC | 18:08 | |
*** meghal has joined #openstack-ironic | 18:09 | |
*** meghal has quit IRC | 18:09 | |
*** meghal has joined #openstack-ironic | 18:09 | |
*** meghal has quit IRC | 18:09 | |
*** meghal has joined #openstack-ironic | 18:09 | |
clif_h | jlvillal: +1 | 18:09 |
jlvillal | clif_h: Thanks. I changed it to use 'msg_fmt' as I noticed that is what Nova did. Instead of _message. | 18:10 |
clif_h | yep, seems fine | 18:10 |
*** meghal has quit IRC | 18:10 | |
*** meghal has joined #openstack-ironic | 18:11 | |
*** meghal has quit IRC | 18:11 | |
*** meghal has joined #openstack-ironic | 18:11 | |
*** baoli has quit IRC | 18:16 | |
*** achanda has quit IRC | 18:16 | |
*** openstackgerrit has quit IRC | 18:17 | |
*** openstackgerrit has joined #openstack-ironic | 18:17 | |
*** zhenguo has quit IRC | 18:23 | |
*** marzif has joined #openstack-ironic | 18:24 | |
*** meghal has quit IRC | 18:24 | |
*** e0ne has joined #openstack-ironic | 18:25 | |
*** baoli has joined #openstack-ironic | 18:26 | |
openstackgerrit | Devananda van der Veen proposed openstack/ironic: Added CORS support middleware to Ironic https://review.openstack.org/199769 | 18:26 |
devananda | dims_: ^ | 18:26 |
*** meghal has joined #openstack-ironic | 18:27 | |
devananda | i'm having issues with vagrant this morning (local env things) so i haven't tested that yet :-/ | 18:27 |
devananda | also I am just about out of time for today | 18:28 |
*** marzif has quit IRC | 18:28 | |
NobodyCam | have a great time burnning things for the next two weeks devananda :) | 18:28 |
dims_ | devananda: lgtm | 18:29 |
rloo | devananda: what's the story wrt the 4.0.0 release? has it been released? | 18:30 |
*** baoli has quit IRC | 18:36 | |
*** lifeless1 is now known as lifeless | 18:39 | |
*** ionutbalutoiu has quit IRC | 18:43 | |
*** romcheg1 has joined #openstack-ironic | 18:47 | |
*** e0ne has quit IRC | 18:50 | |
*** marzif has joined #openstack-ironic | 18:50 | |
*** baoli has joined #openstack-ironic | 18:50 | |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Support for fine-grained distro-related defaults https://review.openstack.org/217350 | 18:51 |
*** asd112z_ has joined #openstack-ironic | 18:52 | |
*** mtanino_ has quit IRC | 18:53 | |
*** tsekiyam_ has joined #openstack-ironic | 18:53 | |
*** dims_ has quit IRC | 18:53 | |
*** dims has joined #openstack-ironic | 18:54 | |
*** mtanino has joined #openstack-ironic | 18:54 | |
*** asd112z has quit IRC | 18:56 | |
*** Sukhdev has quit IRC | 18:56 | |
*** tsekiyama has quit IRC | 18:57 | |
devananda | rloo: status is on the white board | 19:01 |
devananda | rloo: tldr; there is a tarball. it is not what I would call a "release" in the sense of being complete. however, it is "released" | 19:01 |
rloo | devananda: thx. so not ready to announce to the world. | 19:01 |
devananda | rloo: the switch to post versioning has some rough edges. we'll need to polish it over the next ~ month | 19:02 |
rloo | devananda: wrt your comment about taggin 4.0.1 soon -- ?? are there some fixes in particular? | 19:02 |
devananda | well, we could announce it. it's a thing people could download and use. but they'd have to be developers (us, basically) to benefit from it | 19:02 |
devananda | most users should wait for Liberty release | 19:02 |
rloo | devananda: i'll leave it for jroll to announce or not since he wanted these releases. | 19:03 |
*** baoli has quit IRC | 19:04 | |
devananda | rloo: I think that's best | 19:04 |
*** marzif has quit IRC | 19:04 | |
rloo | devananda: i'm assuming the 4.1.0 is the version that corresponds to liberty release, and we'll do that around the liberty-rc1 timeline. | 19:05 |
*** baoli has joined #openstack-ironic | 19:05 | |
devananda | rloo: fwiw, https://review.openstack.org/#/c/216758 is in the gate now. that fixes the README so dhellmann 's release tools can pick it up | 19:05 |
rloo | devananda: yeah, i saw that. looks like the release notes is the (hopefully last) outstanding item. | 19:06 |
rloo | devananda: ah, ok, you are answering my question about 4.0.1. thx. | 19:06 |
devananda | yea | 19:06 |
devananda | i have a ptach up for those - https://review.openstack.org/#/c/216843/ | 19:06 |
devananda | both are noted on the whiteboard | 19:06 |
devananda | see the new section at L34 | 19:07 |
rloo | devananda: yup, looking at it as you were updating it :) | 19:07 |
devananda | :) | 19:07 |
rloo | devananda: do we care to release the client now? | 19:08 |
devananda | we should follow through on monday's agreement to pin the client version | 19:08 |
devananda | and we should document that in the server docs | 19:08 |
devananda | and then yes, we should release a client | 19:08 |
devananda | note that the client is now managed by dhellmann in the same way that the server is -- I can no longer just tag a client when ever I want to | 19:09 |
rloo | devananda: client was already pinned to 1.9, so nothing to do there. | 19:09 |
devananda | that also means jroll can do a client release (by working with dhellmann ) | 19:09 |
rloo | devananda: did we change the behaviour besides pinning to 1.0? | 19:09 |
devananda | rloo: except for documenting it :) | 19:09 |
rloo | devananda: oh. documenting. who needs that?! I'll bug NobodyCam about that. | 19:10 |
devananda | :-D | 19:10 |
* devananda goes afk again | 19:10 | |
NobodyCam | huh :-p | 19:10 |
NobodyCam | me | 19:10 |
NobodyCam | lol | 19:11 |
*** nicodemos has quit IRC | 19:11 | |
*** nicodemos_ has joined #openstack-ironic | 19:12 | |
*** nicodemos_ has quit IRC | 19:12 | |
*** e0ne has joined #openstack-ironic | 19:13 | |
*** tsekiyam_ has quit IRC | 19:13 | |
rloo | NobodyCam: was referring to https://review.openstack.org/#/c/209251/ | 19:13 |
NobodyCam | oh ya.. .I'll get a update to that up today :) | 19:14 |
NobodyCam | and thank you for the review :) | 19:14 |
*** krtaylor has quit IRC | 19:14 | |
*** achanda has joined #openstack-ironic | 19:16 | |
*** nicodemos has joined #openstack-ironic | 19:17 | |
*** Sukhdev has joined #openstack-ironic | 19:18 | |
thiagop | jlvillal: Your last comments on the OneView driver were addressed. Thanks again ;) | 19:21 |
SpamapS | TheJulia: before I get too far down the road.. I'm looking to add support to bifrost to have it setup a DHCP server for all the boxes in dynamic inventory. | 19:21 |
*** ionutbalutoiu has joined #openstack-ironic | 19:22 | |
SpamapS | TheJulia: this is to deal with the complexities we're having around vlans + bridges. | 19:22 |
*** ionutbalutoiu has quit IRC | 19:22 | |
SpamapS | TheJulia: simpler for us to just bake the DHCP config in than try to teach glean how to do all of this. | 19:22 |
*** meghal has quit IRC | 19:24 | |
*** achanda has quit IRC | 19:25 | |
*** meghal has joined #openstack-ironic | 19:26 | |
*** sonuk has joined #openstack-ironic | 19:27 | |
jlvillal | thiagop, Thanks. Hopefully some other people will look at it too :) | 19:32 |
thiagop | jlvillal: I'm hoping for that | 19:32 |
*** tsekiyama has joined #openstack-ironic | 19:37 | |
*** mtanino_ has joined #openstack-ironic | 19:37 | |
*** Marga_ has joined #openstack-ironic | 19:38 | |
*** mtanino has quit IRC | 19:39 | |
*** meghal has quit IRC | 19:48 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Add more info to checksum exception https://review.openstack.org/217369 | 19:49 |
TheJulia | SpamapS: I'm all for this, I just didn't do it because... well... shiny object syndrome | 19:50 |
*** irenab has quit IRC | 19:51 | |
SpamapS | TheJulia: :) | 19:53 |
SpamapS | TheJulia: I'm working on it. just working out how to handle the nics | 19:54 |
SpamapS | I think I'll just take the first NIC and deal with what that means later. | 19:54 |
openstackgerrit | Merged openstack/ironic: Create a versions.py file https://review.openstack.org/213856 | 19:57 |
*** nicodemos has quit IRC | 19:58 | |
*** marzif has joined #openstack-ironic | 20:00 | |
TheJulia | SpamapS: My only concern would be exausing the available pool | 20:00 |
*** achanda has joined #openstack-ironic | 20:02 | |
*** puranamr has quit IRC | 20:03 | |
SpamapS | TheJulia: er, no pool | 20:03 |
SpamapS | TheJulia: still static assignment | 20:03 |
SpamapS | TheJulia: just from the dynamic inventory -> dhcp-host lines | 20:04 |
* TheJulia sees caveat warnings appear in her brain | 20:06 | |
*** achanda has quit IRC | 20:08 | |
openstackgerrit | Clint 'SpamapS' Byrum proposed openstack/bifrost: Add static DHCP for dynamic inventory https://review.openstack.org/217374 | 20:08 |
SpamapS | TheJulia: ^ untested braindump | 20:08 |
*** baoli has quit IRC | 20:10 | |
*** krtaylor has joined #openstack-ironic | 20:11 | |
*** marzif has quit IRC | 20:12 | |
TheJulia | SpamapS: I'll take a look in a little bit | 20:13 |
*** puranamr has joined #openstack-ironic | 20:18 | |
*** irenab has joined #openstack-ironic | 20:18 | |
*** puranamr has quit IRC | 20:21 | |
*** Marga_ has quit IRC | 20:21 | |
*** meghal has joined #openstack-ironic | 20:22 | |
*** puranamr has joined #openstack-ironic | 20:24 | |
*** lucas-dinner has quit IRC | 20:25 | |
*** meghal has quit IRC | 20:26 | |
*** achanda has joined #openstack-ironic | 20:26 | |
*** e0ne has quit IRC | 20:28 | |
*** puranamr has quit IRC | 20:28 | |
*** [1]cdearborn has joined #openstack-ironic | 20:30 | |
*** endorphin has quit IRC | 20:35 | |
*** albertoffb has quit IRC | 20:35 | |
*** puranamr has joined #openstack-ironic | 20:36 | |
*** Nisha_away has quit IRC | 20:36 | |
*** Nisha has joined #openstack-ironic | 20:42 | |
*** cdearborn has quit IRC | 20:46 | |
*** irenab_ has joined #openstack-ironic | 20:50 | |
*** irenab has quit IRC | 20:50 | |
*** irenab_ is now known as irenab | 20:50 | |
*** asd112z_ has quit IRC | 20:51 | |
*** asd112z has joined #openstack-ironic | 20:52 | |
*** asd112z has quit IRC | 20:52 | |
*** asd112z has joined #openstack-ironic | 20:53 | |
trown | TheJulia: nice catch on the agent element, I am confused whey we need a mapping of curl:curl | 20:53 |
openstackgerrit | Stephanie Miller proposed openstack/bifrost: Systemd is not just for Red Hat derivatives https://review.openstack.org/217391 | 20:53 |
openstackgerrit | Lilia Sampaio proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 20:58 |
*** meghal has joined #openstack-ironic | 20:59 | |
*** irenab has quit IRC | 21:01 | |
*** openstackgerrit has quit IRC | 21:01 | |
*** openstackgerrit has joined #openstack-ironic | 21:02 | |
*** r-daneel has quit IRC | 21:05 | |
*** irenab has joined #openstack-ironic | 21:05 | |
*** achanda has quit IRC | 21:07 | |
*** irenab has quit IRC | 21:10 | |
rloo | hi jlvillal, we should spend a bit of time looking into failures. wrt https://review.openstack.org/215739, it is that timeout issue. | 21:10 |
*** jcoufal has quit IRC | 21:10 | |
*** meghal has quit IRC | 21:11 | |
*** romcheg1 has quit IRC | 21:13 | |
*** jjohnson2 has quit IRC | 21:14 | |
*** trown is now known as trown|outttypeww | 21:15 | |
jlvillal | rloo: I'm not sure. I don't know much about the test. My simple logic was it passed the 'test' check and then failed the 'gate' check. Try again... | 21:16 |
*** romcheg1 has joined #openstack-ironic | 21:16 | |
thiagop | good night folks | 21:17 |
openstackgerrit | Lilia Sampaio proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 21:17 |
*** thiagop has quit IRC | 21:17 | |
*** Sukhdev has quit IRC | 21:18 | |
jlvillal | When thiagop says good night, thiagop is gone | 21:18 |
jlvillal | JoshNang: I noticed the python34 job is now voting for IPA. My patch had it listed as voting :) | 21:18 |
*** jcoufal has joined #openstack-ironic | 21:18 | |
*** irenab has joined #openstack-ironic | 21:20 | |
JoshNang | jlvillal: \o/ i saw the job merged this morning | 21:20 |
TheJulia | trown|outttypeww: okay, interesting status tag.... | 21:20 |
*** puranamr has quit IRC | 21:21 | |
TheJulia | trown|outttypeww: So we may not actually need to map curl out, it was my immediate thought as to why the pkg-map was failing, then I realized it is json and... was missing commas | 21:21 |
*** irenab has quit IRC | 21:21 | |
rloo | jlvillal: i don't know much about debuggin the tests, but if everyone does a 'recheck' we'll never know if some particular bug/test is recurring. | 21:23 |
rloo | jlvillal: http://logs.openstack.org/39/215739/1/gate/gate-tempest-dsvm-ironic-pxe_ipa/bc585ef/logs/testr_results.html.gz | 21:23 |
jlvillal | rloo: agreed. I also don't know much about debugging the tests. | 21:24 |
rloo | jlvillal: I got that from clicking on the failed test, then logs, then under tempest, testr_results.html.gz | 21:24 |
jlvillal | rloo: Cool link. I never knew about that. | 21:24 |
rloo | jlvillal: or something like that | 21:24 |
jlvillal | rloo: Oh, I thought it was a history of all test runs. | 21:25 |
jlvillal | rloo: Which sounded more interesting to figure out how much a test is failing. | 21:25 |
*** absubram has quit IRC | 21:26 | |
jlvillal | rloo: I don't know if that error has anything to do with Ironic or it is a failure with the tempest job getting setup. | 21:26 |
*** puranamr has joined #openstack-ironic | 21:28 | |
*** irenab has joined #openstack-ironic | 21:28 | |
*** irenab has quit IRC | 21:29 | |
*** Sukhdev has joined #openstack-ironic | 21:30 | |
rloo | jlvillal: i don't know much about it all. I just muddle through. | 21:31 |
*** irenab has joined #openstack-ironic | 21:31 | |
jlvillal | +1 for me | 21:32 |
rloo | jlvillal: so if you do a 'recheck bug #...', that'll get counted. and if the plumbing is in place, will show up in elastic something. | 21:32 |
rloo | jlvillal: elastic recheck: http://status.openstack.org/elastic-recheck/ | 21:33 |
*** alex_xu has quit IRC | 21:33 | |
jlvillal | rloo: Okay. Do you want to use the bug listed in the whiteboard? | 21:33 |
jlvillal | rloo: https://bugs.launchpad.net/ironic/+bug/1408067 | 21:34 |
openstack | Launchpad bug 1408067 in Ironic "ironic jobs sometimes fail with node callback timeout" [High,Triaged] | 21:34 |
jlvillal | This failure seems different, I think | 21:34 |
jlvillal | tempest.exceptions.BuildErrorException: Server 96d6c1ba-4b75-49e9-9aef-132a9889111a failed to build and is in ERROR status | 21:35 |
*** alex_xu has joined #openstack-ironic | 21:35 | |
*** irenab has quit IRC | 21:36 | |
*** jamielennox has quit IRC | 21:36 | |
rloo | jlvillal: there's a bug. i have it noted in the etherpad. | 21:37 |
jlvillal | rloo: The one for the SSH job? | 21:37 |
rloo | jlvillal: it might just be that we have to increase the timeout. i don't know. | 21:38 |
*** jamielennox has joined #openstack-ironic | 21:38 | |
rloo | jlvillal: the point being that if it is failing fairly often, it might be something that someone should look into. | 21:38 |
rloo | jlvillal: but if folks just do 'recheck' and don't spend a few min looking into wh something failed, we'll never know. | 21:39 |
*** natorious has quit IRC | 21:39 | |
jlvillal | Okay. I'm not sure how often it is occurring. Intermittent errors are the hard ones to fix, of course. | 21:40 |
rloo | jlvillal: don't know. i got rid of all the test failure links cuz i have too many windows open. don't want to look into it. | 21:41 |
*** natorious has joined #openstack-ironic | 21:41 | |
openstackgerrit | Merged openstack/ironic: Follow up the nits in iRMC vmedia driver merged patch. https://review.openstack.org/217027 | 21:43 |
*** meghal has joined #openstack-ironic | 21:43 | |
*** puranamr has quit IRC | 21:46 | |
mrda | Morning Ironic | 21:46 |
*** krtaylor has quit IRC | 21:48 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers https://review.openstack.org/217068 | 21:48 |
*** dprince has quit IRC | 21:50 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Expose versioning information on GET / endpoint https://review.openstack.org/209012 | 21:53 |
NobodyCam | good morning mrda | 21:55 |
NobodyCam | :) | 21:55 |
mrda | o/ | 21:56 |
NobodyCam | :) | 21:56 |
*** dims_ has joined #openstack-ironic | 21:56 | |
*** dims has quit IRC | 21:59 | |
*** romcheg1 has left #openstack-ironic | 22:04 | |
*** achanda has joined #openstack-ironic | 22:06 | |
*** linggao has quit IRC | 22:06 | |
*** meghal has quit IRC | 22:08 | |
*** meghal has joined #openstack-ironic | 22:08 | |
*** meghal has quit IRC | 22:08 | |
*** meghal has joined #openstack-ironic | 22:08 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic-python-agent: Add command metrics to IPA API https://review.openstack.org/119981 | 22:09 |
openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add volume_connector table to save connector information https://review.openstack.org/200983 | 22:10 |
openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce VolumeConnector object https://review.openstack.org/214584 | 22:10 |
openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Introduce new RPCs to support volume connector operation https://review.openstack.org/214585 | 22:10 |
openstackgerrit | Merged openstack/ironic: Add cleanup in console utils tests https://review.openstack.org/216745 | 22:11 |
*** shoutm has joined #openstack-ironic | 22:15 | |
*** [1]cdearborn has quit IRC | 22:15 | |
*** meghal has quit IRC | 22:21 | |
*** meghal has joined #openstack-ironic | 22:21 | |
*** dims_ has quit IRC | 22:24 | |
*** puranamr has joined #openstack-ironic | 22:24 | |
*** dims has joined #openstack-ironic | 22:24 | |
*** achanda_ has joined #openstack-ironic | 22:28 | |
*** achanda has quit IRC | 22:29 | |
*** jcoufal has quit IRC | 22:31 | |
*** meghal has quit IRC | 22:32 | |
*** shoutm has quit IRC | 22:32 | |
*** dims has quit IRC | 22:34 | |
*** shoutm has joined #openstack-ironic | 22:36 | |
*** meghal has joined #openstack-ironic | 22:38 | |
*** krtaylor has joined #openstack-ironic | 22:39 | |
*** meghal has quit IRC | 22:39 | |
*** meghal has joined #openstack-ironic | 22:45 | |
*** Nisha_away has joined #openstack-ironic | 22:49 | |
*** Nisha has quit IRC | 22:49 | |
openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: Add REST API for volume connector operation https://review.openstack.org/214586 | 22:51 |
openstackgerrit | Tomoki Sekiyama proposed openstack/ironic: [POC] pxe: support boot from Cinder iSCSI volume using iPXE https://review.openstack.org/215849 | 22:51 |
JayF | rloo: congrats, you were buried in +1s by the time I got to the email, but you get my full support :) | 22:55 |
rloo | JayF: thx! | 23:01 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 23:02 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 23:04 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers https://review.openstack.org/217068 | 23:07 |
Nisha_away | rloo, JayF NobodyCam devananda Please could you review above ^^^ | 23:10 |
Nisha_away | (ofcourse at your convinience :)) | 23:11 |
*** Nisha_away has quit IRC | 23:16 | |
*** achanda has joined #openstack-ironic | 23:18 | |
*** achanda_ has quit IRC | 23:18 | |
*** arif-ali has quit IRC | 23:19 | |
*** arif-ali has joined #openstack-ironic | 23:19 | |
*** thrash is now known as thrash|g0ne | 23:20 | |
* mrda just tells everyone about the nova ironic driver via email | 23:29 | |
*** Sukhdev has quit IRC | 23:30 | |
openstackgerrit | Clif Houck proposed openstack/ironic: Standardize use of str(exc) to get Exception descriptions https://review.openstack.org/217434 | 23:33 |
clif_h | https://review.openstack.org/#/c/217434/ <- jlvillal finally got around to the exception standardization patch | 23:33 |
jlvillal | clif_h: Thanks. I'll take a look. | 23:34 |
jlvillal | clif_h: +1 from me. tox tests worked for me. | 23:37 |
clif_h | thanks | 23:37 |
*** achanda has quit IRC | 23:38 | |
*** meghal has quit IRC | 23:44 | |
*** naohirot has joined #openstack-ironic | 23:53 | |
*** shoutm_ has joined #openstack-ironic | 23:54 | |
*** blakec has quit IRC | 23:57 | |
*** asd112z has quit IRC | 23:57 | |
*** shoutm has quit IRC | 23:57 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!