naohirot | jroll: as you mentioned in the last irc meeting log. | 00:00 |
---|---|---|
*** absubram has joined #openstack-ironic | 00:01 | |
*** alexpilotti has joined #openstack-ironic | 00:01 | |
JoshNang | naohirot: he just left | 00:01 |
naohirot | JoshNang: Hi good evening, Ok, thank you for letting me know. | 00:02 |
*** Haomeng has joined #openstack-ironic | 00:02 | |
JoshNang | naohirot: good evening! np | 00:03 |
naohirot | jroll: 17:36:51 <jroll> I may miss gerrit things, I may miss IRC scrollback, but I won't miss mailing list posts or meeting logs. | 00:04 |
naohirot | jroll: ^^^^ http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-08-31-17.00.log.html | 00:04 |
*** Haomeng|2 has quit IRC | 00:05 | |
*** zz_natorious is now known as natorious | 00:08 | |
*** alexpilotti has quit IRC | 00:10 | |
*** garthb__ has quit IRC | 00:18 | |
*** goodygum has quit IRC | 00:19 | |
*** mtanino_ has quit IRC | 00:21 | |
*** meghal has quit IRC | 00:21 | |
*** jcoufal has joined #openstack-ironic | 00:21 | |
*** goodygum has joined #openstack-ironic | 00:21 | |
openstackgerrit | Merged openstack/ironic-python-agent: Split functional and py27 tests https://review.openstack.org/214735 | 00:22 |
*** shadower has quit IRC | 00:23 | |
*** shadower has joined #openstack-ironic | 00:23 | |
*** achanda has joined #openstack-ironic | 00:28 | |
*** natorious is now known as zz_natorious | 00:28 | |
*** alexpilotti has joined #openstack-ironic | 00:32 | |
*** Sukhdev_ has joined #openstack-ironic | 00:37 | |
*** meghal has joined #openstack-ironic | 00:38 | |
*** achanda has quit IRC | 00:51 | |
*** achanda has joined #openstack-ironic | 00:52 | |
*** achanda has quit IRC | 00:52 | |
*** stendulker has joined #openstack-ironic | 00:55 | |
*** shoutm_ has joined #openstack-ironic | 00:56 | |
*** shoutm has quit IRC | 00:56 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface https://review.openstack.org/216538 | 00:57 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface https://review.openstack.org/217102 | 00:58 |
Sukhdev_ | NobodyCam: Finally, I am up and running - what a day!!! | 01:00 |
*** jiangfei has quit IRC | 01:01 | |
*** stendulker has quit IRC | 01:08 | |
*** saripurigopi has joined #openstack-ironic | 01:08 | |
*** meghal has quit IRC | 01:11 | |
jroll | naohirot: I'll try to look at it soon, my main priority right now is figuring out liberty release things | 01:17 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments https://review.openstack.org/219390 | 01:17 |
*** ijw__ has quit IRC | 01:20 | |
naohirot | jroll: Oh, hi jim | 01:20 |
jroll | hi :) | 01:20 |
naohirot | jroll: I know you are busy for many thing:) | 01:20 |
naohirot | jroll: but what I'd like to know is if the direction I proposed is okay or not. | 01:21 |
naohirot | jroll: there are two concerns as I described in | 01:22 |
jroll | naohirot: I understand, it's on my todo list, I will try to get to it | 01:23 |
jroll | sorry I kind of dropped this, been busy with downstream + releases + vacation lately | 01:23 |
naohirot | https://review.openstack.org/#/c/207897/ | 01:24 |
naohirot | jroll: the last comment of ^^^ | 01:24 |
jroll | naohirot: are you asking if I'm okay with splitting the patches into three pieces like that? | 01:25 |
jroll | naohirot: or the approach of each patch? | 01:26 |
naohirot | jroll: 2 things I'd like to confirm are 1) cancel implementation 2) drop NMI from ipmidriver | 01:26 |
*** saripurigopi has quit IRC | 01:26 | |
jroll | oh, so not the last comment | 01:26 |
jroll | rather the one from Aug 14 8:04 AM | 01:26 |
naohirot | jroll: Oh, I put wrong url. | 01:27 |
naohirot | jroll: Yeah, URL is correct, but not the last my comment :) | 01:27 |
jroll | right, ok | 01:28 |
jroll | naohirot: I did see that comment - I haven't had time to process my thoughts on it yet | 01:28 |
naohirot | jroll: the 5th my comment from the last, | 01:28 |
naohirot | jroll: on Naohiro TamuraAug 15 12:04 AM | 01:28 |
jroll | naohirot: right, I still need to review it further | 01:29 |
naohirot | jroll: Okay, In terms of dropping NMI from IPMItool, I considered muti tenant | 01:29 |
naohirot | jroll: after switching to deploy network to tenant network, conductor cannot reach to Instance, | 01:30 |
naohirot | jroll: so conductor only can communicate bmc | 01:31 |
naohirot | jroll: however bmc cannot detect the status of OS | 01:31 |
naohirot | jroll: so ipmitool doesn't have a way if nmi has been completed or not. that's the reason I propose to drop NMI from ipmitool. | 01:32 |
naohirot | jroll: anyway have a good evening, thanks for your consideration :) | 01:33 |
jroll | naohirot: I'm not sure that's ipmitool specific, but I'll think about it more and leave comments in the review | 01:37 |
jroll | and you're welcome, have a good day today :) | 01:38 |
naohirot | jroll: Okay, I'll check your comment in the gerrit :) | 01:39 |
*** jcoufal has quit IRC | 01:41 | |
*** achanda has joined #openstack-ironic | 01:44 | |
*** saripurigopi has joined #openstack-ironic | 01:46 | |
*** boris-42 has quit IRC | 01:50 | |
*** jcoufal has joined #openstack-ironic | 01:52 | |
*** saripurigopi has quit IRC | 01:52 | |
*** baoli has quit IRC | 02:06 | |
*** baoli has joined #openstack-ironic | 02:06 | |
*** baoli has quit IRC | 02:07 | |
*** baoli has joined #openstack-ironic | 02:08 | |
openstackgerrit | Merged openstack/ironic: Removes unused posix-ipc requirement https://review.openstack.org/219003 | 02:11 |
*** jcoufal has quit IRC | 02:16 | |
*** saripurigopi has joined #openstack-ironic | 02:17 | |
saripurigopi | Morning Ironic | 02:17 |
*** spandhe has quit IRC | 02:17 | |
*** Sukhdev_ has quit IRC | 02:32 | |
openstackgerrit | Merged openstack/ironic: Remove unused dep discover from test reqs https://review.openstack.org/218313 | 02:33 |
*** hakimo_ has quit IRC | 02:54 | |
*** hakimo has joined #openstack-ironic | 02:57 | |
*** Nisha has joined #openstack-ironic | 03:00 | |
*** ramineni has joined #openstack-ironic | 03:03 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/218906 | 03:06 |
*** harshs has joined #openstack-ironic | 03:07 | |
*** meghal has joined #openstack-ironic | 03:08 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers https://review.openstack.org/217068 | 03:14 |
*** vvb1 has joined #openstack-ironic | 03:15 | |
*** shoutm_ has quit IRC | 03:20 | |
*** meghal1 has joined #openstack-ironic | 03:22 | |
*** meghal has quit IRC | 03:23 | |
*** Sukhdev has quit IRC | 03:26 | |
*** Sukhdev has joined #openstack-ironic | 03:27 | |
*** BadCub has quit IRC | 03:27 | |
*** eglute has quit IRC | 03:27 | |
*** zz_natorious has quit IRC | 03:27 | |
*** BadCub has joined #openstack-ironic | 03:27 | |
*** vishwanathj has quit IRC | 03:28 | |
*** achanda has quit IRC | 03:28 | |
*** davidlenwell has quit IRC | 03:29 | |
*** dguerri` has quit IRC | 03:30 | |
*** jroll has quit IRC | 03:30 | |
*** zz_natorious has joined #openstack-ironic | 03:31 | |
*** zz_natorious is now known as natorious | 03:32 | |
*** eglute has joined #openstack-ironic | 03:32 | |
*** davidlenwell has joined #openstack-ironic | 03:32 | |
*** dguerri` has joined #openstack-ironic | 03:32 | |
*** dguerri` is now known as dguerri | 03:32 | |
*** dguerri has joined #openstack-ironic | 03:32 | |
*** jroll has joined #openstack-ironic | 03:33 | |
*** adam_g has quit IRC | 03:33 | |
*** adam_g has joined #openstack-ironic | 03:35 | |
*** adam_g has quit IRC | 03:35 | |
*** adam_g has joined #openstack-ironic | 03:35 | |
*** dims has joined #openstack-ironic | 03:35 | |
*** shoutm has joined #openstack-ironic | 03:41 | |
*** boris-42 has joined #openstack-ironic | 03:59 | |
*** dims has quit IRC | 04:00 | |
*** links has joined #openstack-ironic | 04:02 | |
*** Sukhdev_ has joined #openstack-ironic | 04:06 | |
*** amotoki has joined #openstack-ironic | 04:11 | |
*** baoli has quit IRC | 04:14 | |
*** vishwanathj has joined #openstack-ironic | 04:22 | |
*** shoutm has quit IRC | 04:38 | |
*** rameshg87 has joined #openstack-ironic | 04:42 | |
*** puranamr has joined #openstack-ironic | 04:46 | |
*** shoutm has joined #openstack-ironic | 04:47 | |
*** PradeepV has joined #openstack-ironic | 04:51 | |
PradeepV | Good morning Ironic | 04:52 |
*** dims has joined #openstack-ironic | 04:59 | |
*** puranamr has quit IRC | 05:04 | |
*** dims has quit IRC | 05:06 | |
*** puranamr has joined #openstack-ironic | 05:11 | |
*** puranamr has quit IRC | 05:11 | |
*** ukalifon has joined #openstack-ironic | 05:15 | |
*** puranamr has joined #openstack-ironic | 05:16 | |
*** puranamr has quit IRC | 05:18 | |
*** puranamr has joined #openstack-ironic | 05:24 | |
*** derekh has joined #openstack-ironic | 05:26 | |
*** achanda has joined #openstack-ironic | 05:34 | |
*** shoutm has quit IRC | 05:36 | |
*** shoutm has joined #openstack-ironic | 05:37 | |
*** achanda has quit IRC | 05:40 | |
*** ishant has joined #openstack-ironic | 05:43 | |
*** puranamr has quit IRC | 05:47 | |
*** VikasC has joined #openstack-ironic | 05:49 | |
*** shoutm has quit IRC | 05:58 | |
*** shoutm has joined #openstack-ironic | 05:58 | |
*** achanda has joined #openstack-ironic | 06:00 | |
*** Sukhdev_ has quit IRC | 06:06 | |
*** baoli has joined #openstack-ironic | 06:14 | |
*** natorious is now known as zz_natorious | 06:18 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: agent_ilo doesnt boot up the image https://review.openstack.org/220003 | 06:18 |
*** baoli has quit IRC | 06:19 | |
*** achanda has quit IRC | 06:19 | |
*** getvasanth has joined #openstack-ironic | 06:23 | |
*** shoutm_ has joined #openstack-ironic | 06:25 | |
*** shoutm has quit IRC | 06:26 | |
*** harshs has quit IRC | 06:29 | |
*** achanda has joined #openstack-ironic | 06:48 | |
*** meghal1 has quit IRC | 06:53 | |
*** dtantsur|afk is now known as dtantsur | 06:57 | |
dtantsur | Morning Ironic | 06:57 |
Nisha | dtantsur, o/ | 07:01 |
Nisha | dtantsur, i responded to ur comments on https://review.openstack.org/#/c/201904/. Wanted to discuss on the inspector config variable for ports | 07:04 |
dtantsur | sure | 07:05 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 07:10 |
getvasanth | Good morning all, any pointers to create a bootable image with DIB | 07:10 |
getvasanth | created a baremetal and a vm image both failed with "No or empty root= argument error" | 07:11 |
*** Marga_ has joined #openstack-ironic | 07:15 | |
*** Marga_ has quit IRC | 07:15 | |
Nisha | dtantsur, does the current config variable add_ports=pxe, keep_ports=all, will create the port only for the MAC which is being used for inspecting? | 07:15 |
*** Marga_ has joined #openstack-ironic | 07:16 | |
Nisha | dtantsur, or for all connected MACs | 07:16 |
dtantsur | Nisha, only that one (keep_ports means it won't delete any existing ports) | 07:16 |
Nisha | dtantsur, ilo drivers requirements is to create ports for all connected MACs | 07:17 |
Nisha | and not only one | 07:17 |
dtantsur | Nisha, you can add setting add_ports=active to your documentation | 07:18 |
Nisha | this will be a new option correct? | 07:20 |
Nisha | and is not existing currently | 07:20 |
*** ifarkas has joined #openstack-ironic | 07:22 | |
dtantsur | Nisha, it exists | 07:28 |
Nisha | dtantsur, ok. could you give me pointer for this setting | 07:29 |
dtantsur | Nisha, https://github.com/openstack/ironic-inspector/blob/master/example.conf#L538-L545 | 07:29 |
Nisha | dtantsur, but if IP is not assigned? say 2 MACs are physically connected but IPs are not provided will this setting work? | 07:32 |
dtantsur | Nisha, no. that's how we define "active" currently | 07:32 |
Nisha | dtantsur, but during inspection we dont expect the neutron and ironic ports created for the MACs, correct? | 07:33 |
dtantsur | Nisha, right | 07:34 |
Nisha | then how does active helps/differ from pxe? | 07:34 |
Nisha | add_ports=pxe, add_ports=all, add_ports=active | 07:35 |
Nisha | getvasanth, what command line did u use? | 07:36 |
*** derekh has quit IRC | 07:36 | |
dtantsur | Nisha, it depends on whether you have a DHCP server in your environment.. FWIW I'm open to redefining "active" there, or adding a new option | 07:36 |
Nisha | dtantsur, i would prefer a new option to add_ports | 07:37 |
Nisha | because active seems to be dependent on IP address | 07:38 |
Nisha | dtantsur, how about add_ports=connected | 07:38 |
dtantsur | looks good. or we can change what "active" means, I don't care much. | 07:38 |
Nisha | will it not be a backward compatibility break for inspector | 07:41 |
Nisha | ? | 07:41 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: agent_ilo fails to bring up instance https://review.openstack.org/220003 | 07:41 |
Nisha | dtantsur, ^^^ | 07:43 |
Nisha | dtantsur, will it not be a compatibility break for inspector | 07:44 |
Nisha | dtantsur, one more query apart from this....you have suggested following : "Enhance IPA inspection support to discover the above capabilities" | 07:44 |
Nisha | dtantsur, now the inspection is supposed to be working only with IPA? | 07:45 |
openstackgerrit | Merged openstack/ironic: Revert "Do not overwrite the iPXE boot script on every deployment" https://review.openstack.org/219749 | 07:46 |
Nisha | dtantsur, the bash ramdisk is deprecated in liberty but not removed, so IMO it should be allowed in both ramdisk | 07:46 |
openstackgerrit | Merged openstack/ironic: Add deprecation warning to periodic tasks with parallel=False https://review.openstack.org/216604 | 07:47 |
*** Haomeng|2 has joined #openstack-ironic | 07:47 | |
dtantsur | Nisha, well, I personally don't want (and can't) spend time on the bash ramdisk, so yeah, I'd prefer us working with IPA first | 07:47 |
dtantsur | Nisha, the IPA patch for inspector is awaiting reviews: https://review.openstack.org/#/c/205587/ | 07:48 |
*** Haomeng has quit IRC | 07:50 | |
*** vvb1 has quit IRC | 07:51 | |
Nisha | dtantsur, if we just enhance the ironic-discoverd-ramdisk will it not work for both? | 07:52 |
*** Marga_ has quit IRC | 07:52 | |
Nisha | and then in ironic-inspector and ironic/driver/modules/inspector.py we can simply consume it? | 07:53 |
Nisha | dtantsur, this was my thought for retrieving the data inband | 07:53 |
Nisha | dtantsur, for IPA ramdisk the IPA needs to be enhanced additionally. | 07:54 |
Nisha | dtantsur, am i missing some flow in between | 07:55 |
*** Marga_ has joined #openstack-ironic | 07:56 | |
dtantsur | Nisha, IPA inspection code is unrelated to the old ramdisk. Actually IPA is a reimplementation in Python | 07:57 |
dtantsur | Nisha, so the flow will be the same, but we'll be writing Python code :) | 07:57 |
Nisha | dtantsur, how do u build inspection ramdisk for agent? | 07:58 |
Nisha | is it not thru DIB? | 07:58 |
dtantsur | Nisha, it's also through DIB via ironic-agent element. But right now it's a bit trickier, because 2 required patches didn't merge: https://review.openstack.org/#/c/205587/ and https://review.openstack.org/#/c/211521/ | 07:59 |
Nisha | dtantsur, or if i understand you mean that the DIB element functionaility is IPA instead of being in DIB | 08:00 |
Nisha | hmmm i see | 08:00 |
*** ukalifon1 has joined #openstack-ironic | 08:02 | |
*** links has quit IRC | 08:02 | |
*** ukalifon has quit IRC | 08:04 | |
*** ndipanov has joined #openstack-ironic | 08:06 | |
*** pelix has joined #openstack-ironic | 08:12 | |
*** romainh has joined #openstack-ironic | 08:13 | |
*** lucasagomes has joined #openstack-ironic | 08:18 | |
*** mgoddard has joined #openstack-ironic | 08:22 | |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 08:23 |
getvasanth | Nisha: disk-image-create -u -o ubuntu1 -a amd64 ubuntu | 08:25 |
*** alexpilotti has quit IRC | 08:28 | |
*** dlpartain has joined #openstack-ironic | 08:33 | |
*** achanda has quit IRC | 08:36 | |
*** jistr has joined #openstack-ironic | 08:37 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers https://review.openstack.org/198656 | 08:43 |
Nisha | getvasanth, what is -u for? | 08:44 |
getvasanth | uncompressed | 08:44 |
getvasanth | it wont compressed the image | 08:45 |
getvasanth | Nisha: do you have any specific steps to create a image? | 08:45 |
Nisha | getvasanth, no same comand but i dont use -u | 08:46 |
getvasanth | Nisha: wanted to get the image boot | 08:46 |
Nisha | getvasanth, "disk-image-create ubuntu -o image " will create partition image | 08:47 |
Nisha | getvasanth, "disk-image-create ubuntu vm -o image " will create whole disk image | 08:47 |
getvasanth | Nisha: Ok, but this one fails with the error no or empty argument for root= error | 08:47 |
Nisha | getvasanth, i have never hit this issue | 08:48 |
*** alexpilotti has joined #openstack-ironic | 08:49 | |
getvasanth | Nisha: Ok, this is the actual error message "dracut: FATAL: No or empty root= argument" | 08:50 |
getvasanth | Nisha: then how do u create the ramdisk? | 08:50 |
Nisha | ramdisk or image? | 08:50 |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers https://review.openstack.org/217068 | 08:51 |
getvasanth | Nisha: there 2 steps right, 1. to create the image, 2. to create the ramdisk and the kernel | 08:51 |
Nisha | yes | 08:51 |
getvasanth | Nisha: how do you create the 2 one? | 08:51 |
getvasanth | Nisha: 1 question in your command, were u able to boot both the ways, partition & whole disk image? | 08:53 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Revisit required provision states https://review.openstack.org/219247 | 08:53 |
Nisha | for creating ramdisk, "ramdisk-image-create ubuntu agent-ironic -o image" | 08:53 |
*** alexpilotti has quit IRC | 08:54 | |
*** derekh has joined #openstack-ironic | 08:55 | |
lucasagomes | Nisha, I think you have to use disk-image-creat for "agent-ironic" element | 08:55 |
lucasagomes | not ramdisk-image-create | 08:55 |
lucasagomes | ironic-agent* | 08:56 |
lucasagomes | https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/cleanup.d/99-ramdisk-create | 08:56 |
getvasanth | lucasagomes: hi | 08:56 |
lucasagomes | it will create the ramdisk by itself, the problem with the ramdisk-image-create is that it just builds a filesytem with busybox and replace the init with a custom script. The agent needs a real init system | 08:57 |
lucasagomes | getvasanth, hi | 08:57 |
Nisha | lucasagomes, :) | 08:57 |
getvasanth | lucasagomes: will be back soon | 08:57 |
Nisha | i had always used bash ramdisk till now | 08:57 |
Nisha | for which ramdisk-image-create works | 08:57 |
lucasagomes | Nisha, ok... | 08:58 |
lucasagomes | I thought it would break at some part when building (/me haven't tested it) | 08:58 |
Nisha | i was under the impression the same shud work with agent-ironic also. i will try this once and see | 08:58 |
dtantsur | speaking of ramdisks, lucasagomes could you review my IPA patch for inspection please? https://review.openstack.org/#/c/205587/ | 08:58 |
Nisha | lucasagomes, no it doesnt break | 08:58 |
dtantsur | lucasagomes, and g'morning :) | 08:59 |
lucasagomes | dtantsur, ack, morning | 08:59 |
lucasagomes | Nisha, ironic-agent* | 08:59 |
Nisha | lucasagomes, dtantsur please review these also https://review.openstack.org/198656 , https://review.openstack.org/217068 | 08:59 |
Nisha | lucasagomes, :) | 09:00 |
lucasagomes | ack | 09:00 |
Nisha | lucasagomes, thanks | 09:00 |
*** alexpilotti has joined #openstack-ironic | 09:01 | |
*** dlpartain has left #openstack-ironic | 09:06 | |
*** vvb1 has joined #openstack-ironic | 09:10 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection https://review.openstack.org/201904 | 09:13 |
*** romcheg has joined #openstack-ironic | 09:14 | |
*** siva_ has joined #openstack-ironic | 09:14 | |
sambetts | Morning all o/ | 09:14 |
*** siva_ is now known as Guest43408 | 09:15 | |
*** alexpilotti has quit IRC | 09:15 | |
*** alexpilotti has joined #openstack-ironic | 09:15 | |
*** alexpilotti has quit IRC | 09:15 | |
dtantsur | sambetts, morning | 09:16 |
*** alexpilotti has joined #openstack-ironic | 09:16 | |
*** alexpilotti has quit IRC | 09:16 | |
sambetts | dtantsur o/ | 09:16 |
*** Marga_ has quit IRC | 09:17 | |
*** Marga_ has joined #openstack-ironic | 09:17 | |
*** Marga_ has quit IRC | 09:24 | |
*** Marga_ has joined #openstack-ironic | 09:24 | |
*** vvb2 has joined #openstack-ironic | 09:26 | |
*** vvb2 has left #openstack-ironic | 09:26 | |
openstackgerrit | Merged openstack/ironic: Prevent ilo drivers powering off active nodes during take over https://review.openstack.org/218785 | 09:28 |
*** vvb1 has quit IRC | 09:29 | |
*** Marga_ has quit IRC | 09:29 | |
openstackgerrit | Michael Davies proposed openstack/ironic: Switch default partition type to gpt https://review.openstack.org/126625 | 09:32 |
*** e0ne has joined #openstack-ironic | 09:32 | |
*** alexpilotti has joined #openstack-ironic | 09:37 | |
*** priteau has quit IRC | 09:37 | |
*** alexpilotti has quit IRC | 09:41 | |
*** marzif has joined #openstack-ironic | 09:42 | |
*** Marga_ has joined #openstack-ironic | 09:42 | |
aarefiev | lucasagomes: hi, are u around? | 09:46 |
lucasagomes | aarefiev, hi there, yes | 09:46 |
aarefiev | lucasagomes: I'm trying to fix ironicclient functional job | 09:47 |
aarefiev | could you take a look https://review.openstack.org/#/c/217135/ please | 09:47 |
lucasagomes | aarefiev, sure | 09:48 |
* lucasagomes clicks | 09:48 | |
aarefiev | lucasagomes: thanks | 09:49 |
*** alexpilotti has joined #openstack-ironic | 09:49 | |
*** vvb1 has joined #openstack-ironic | 09:53 | |
aarefiev | lucasagomes: infra patch https://review.openstack.org/#/c/217022/ | 09:54 |
*** naohirot has quit IRC | 09:55 | |
openstackgerrit | Michael Davies proposed openstack/ironic: Switch default partition type to gpt https://review.openstack.org/126625 | 09:58 |
*** priteau has joined #openstack-ironic | 10:02 | |
*** Marga_ has quit IRC | 10:02 | |
*** Marga_ has joined #openstack-ironic | 10:02 | |
lucasagomes | aarefiev, reviewed, overall it LGTM. I had some questions that I put inline | 10:02 |
*** e0ne has quit IRC | 10:04 | |
aarefiev | lucasagomes: thanks, I'll update with your comments, about bug, should I file it in ironicclient? Also my first idea was to get landed infra patch first and then run job on ironic patch, to see how it works | 10:07 |
*** e0ne has joined #openstack-ironic | 10:08 | |
*** marzif has quit IRC | 10:08 | |
*** karimb has joined #openstack-ironic | 10:09 | |
*** kbyrne has quit IRC | 10:12 | |
lucasagomes | aarefiev, I think it would be nice. It just looks a bit odd to say it's broken but the patch has no bug associated with it | 10:13 |
lucasagomes | it's good to have a bug so it's shows in the release notes | 10:13 |
aarefiev | lucasagomes: sure, just I wasn't sure where bug should be | 10:14 |
aarefiev | lucasagomes: thanks for help | 10:14 |
lucasagomes | aarefiev, no problem at all! Thank you for fixing it! | 10:14 |
*** kbyrne has joined #openstack-ironic | 10:15 | |
*** MattMan has quit IRC | 10:23 | |
*** MattMan has joined #openstack-ironic | 10:24 | |
*** alexpilotti has quit IRC | 10:24 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints https://review.openstack.org/218271 | 10:25 |
*** dtantsur is now known as dtantsur|brb | 10:30 | |
*** links has joined #openstack-ironic | 10:31 | |
Nisha | lucasagomes, hi | 10:33 |
lucasagomes | Nisha, hi there | 10:33 |
Nisha | lucasagomes, regarding your comments on https://review.openstack.org/#/c/220003/2/ironic/drivers/modules/agent.py | 10:33 |
Nisha | lucasagomes, i dont see any difference between "if task.driver.boot is not None" and "if task.driver.boot:" | 10:34 |
* lucasagomes relooks | 10:35 | |
lucasagomes | Nisha, oh yeah I think that we don't need to test the placeholder there | 10:36 |
lucasagomes | I think I misread that code | 10:36 |
Nisha | lucasagomes, we need to have the check "if task.driver.boot" | 10:36 |
lucasagomes | Nisha, indeed | 10:37 |
Nisha | else any driver which doesnt has that implemented yet will fail | 10:37 |
lucasagomes | changed my vote there | 10:37 |
lucasagomes | thanks for pointing it out | 10:37 |
Nisha | wow thanks | 10:37 |
lucasagomes | thank you for the fix! | 10:38 |
Nisha | :) thanks for reviewing. | 10:38 |
lucasagomes | np | 10:39 |
* Nisha hope to see my other patches also reviewed | 10:39 | |
Nisha | lucasagomes, do u know how do we run tests for inspector | 10:40 |
Nisha | dtantsur|brb, ^^^ | 10:40 |
lucasagomes | Nisha, which types of tests? (yeah dtantsur|brb will def know the internals better) | 10:40 |
Nisha | lucasagomes, i got some results just now for the tests | 10:41 |
Nisha | will look into them :) | 10:41 |
*** vvb1 has quit IRC | 10:41 | |
lucasagomes | ack | 10:41 |
*** ukalifon1 has quit IRC | 10:42 | |
*** alexpilotti has joined #openstack-ironic | 10:42 | |
*** thrash|g0ne is now known as thrash | 10:44 | |
*** romcheg has quit IRC | 10:45 | |
*** dtantsur|brb is now known as dtantsur | 10:45 | |
*** Nisha has quit IRC | 10:46 | |
*** alexpilotti has quit IRC | 10:47 | |
*** marzif has joined #openstack-ironic | 10:47 | |
*** jcoufal has joined #openstack-ironic | 10:51 | |
PradeepV | Hi Ironic, how can i generate whole disk image in ISO format | 10:51 |
*** alexpilotti has joined #openstack-ironic | 10:54 | |
openstackgerrit | Anton Arefiev proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 10:56 |
*** romcheg has joined #openstack-ironic | 10:57 | |
*** Guest43408 has quit IRC | 10:57 | |
lucasagomes | PradeepV, in DIB I think you can use the "vm" and "iso" elements | 11:01 |
lucasagomes | I've never tested, maybe the folks in #tripleo can tell you better about it | 11:01 |
*** lazy_prince has joined #openstack-ironic | 11:02 | |
PradeepV | ok lucasagomes | 11:03 |
lucasagomes | PradeepV, tho, I don't know how it's going to play out because to have a bootable ISO you will probably need the kernel and ramdisk | 11:03 |
lucasagomes | it won't use the bootloader of the image AFAICT | 11:04 |
PradeepV | i tried creating iso in DIB but it failed | 11:06 |
lucasagomes | PradeepV, with full disk image? Or just partition image? | 11:07 |
PradeepV | full disk image | 11:07 |
lucasagomes | right yeah, I don't know how those would play out | 11:08 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Update iLO driver documentation for configuring Swift https://review.openstack.org/182736 | 11:08 |
lucasagomes | PradeepV, why you need that? Just out of curisity | 11:08 |
rameshg87 | PradeepV: lucasagomes: vm + iso doesn't yield a bootable iso of whole disk image | 11:10 |
rameshg87 | PradeepV: lucasagomes: iso element can be used only with some elements for bundling a kernel and ramdisk to form a bootable element | 11:10 |
lucasagomes | rameshg87, yeah that's my thought as well# | 11:11 |
*** Marga_ has quit IRC | 11:12 | |
* rameshg87 goes home | 11:13 | |
*** romcheg1 has joined #openstack-ironic | 11:13 | |
*** rameshg87 has quit IRC | 11:13 | |
*** romcheg has quit IRC | 11:14 | |
*** shoutm has joined #openstack-ironic | 11:17 | |
*** shoutm_ has quit IRC | 11:18 | |
*** ishant has quit IRC | 11:19 | |
*** ramineni has quit IRC | 11:26 | |
*** dlpartain has joined #openstack-ironic | 11:26 | |
*** alexpilotti has quit IRC | 11:27 | |
getvasanth | lucasagomes: I am back :) | 11:32 |
*** dlpartain has left #openstack-ironic | 11:35 | |
getvasanth | lucasagomes: On te image creation "disk-image-create ubuntu vm -o image" and "ramdisk-image-create -o ramdisk ubuntu" | 11:35 |
*** Marga_ has joined #openstack-ironic | 11:35 | |
getvasanth | lucasagomes: is that correct or am i missing any parameters | 11:36 |
lucasagomes | getvasanth, I think it's missing the deploy element for the ramdisk one | 11:36 |
lucasagomes | getvasanth, which driver you are going to use? | 11:36 |
lucasagomes | getvasanth, for something generic I recommend creating the ramdisk as "disk-image-create -o ramdisk ironic-agent ubuntu" (or fedora which is the most tested one) | 11:37 |
getvasanth | lucasagomes: pxe_ipmitool | 11:37 |
lucasagomes | getvasanth, right that works for pxe_ipmitool ^ | 11:37 |
lucasagomes | getvasanth, btw http://docs.openstack.org/developer/ironic/deploy/install-guide.html#building-or-downloading-a-deploy-ramdisk-image | 11:38 |
getvasanth | lucasagomes: "disk-image-create -o ramdisk ironic-agent ubuntu " this will build both the image and the ramdisk right? | 11:38 |
lucasagomes | getvasanth, no it will only build the ramdisk | 11:39 |
lucasagomes | getvasanth, problem is that the ironic-agent element (which is IPA) needs a real init system | 11:39 |
lucasagomes | the ramdisk-image-create command for DIB replaces the init system with a custom script | 11:39 |
lucasagomes | which is not idea | 11:39 |
lucasagomes | ideal* | 11:39 |
getvasanth | lucasagomes: oh! i am confused now | 11:40 |
lucasagomes | getvasanth, the ironic-agent element workaround that by building the ramdisk itself at the end of the image creation process: https://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/cleanup.d/99-ramdisk-create | 11:40 |
lucasagomes | yeah I know | 11:40 |
lucasagomes | it's confusing as hell | 11:40 |
openstackgerrit | Merged openstack/ironic: Fix docker package name in Ubuntu 14.04 in Install Guide https://review.openstack.org/218694 | 11:40 |
getvasanth | lucasagomes: this is what i am going to do now, 1. disk-image-create ubuntu -o image 2. disk-image-create -o ramdisk ironic-agent ubuntu. is that correct what i am doing? | 11:41 |
lucasagomes | getvasanth, yes | 11:43 |
lucasagomes | getvasanth, tho note there's a different in the first command that you showed me before | 11:43 |
lucasagomes | you were using the "vm" element to create a full disk image | 11:43 |
lucasagomes | the new 1) is creating a partition image (no "vm" element) | 11:43 |
lucasagomes | difference* | 11:44 |
* lucasagomes 1 sec will grab a sandwich with the rest of the folks in the office | 11:44 | |
getvasanth | lucasagomes: yes, understood that, it should boot. | 11:45 |
getvasanth | lucasagomes: thanks for confirming, then next step include the kernel & ramdisk and and upload a 3 part image and boot it | 11:45 |
lucasagomes | getvasanth, ack... yeah so if you want a partition image make sure you have the kernel and ramdisk for that image too | 11:45 |
lucasagomes | brb quickly | 11:46 |
*** lucasagomes is now known as lucas-hungry | 11:46 | |
getvasanth | lucasagomes: yes!!, thanks a lot again, will be back after building it | 11:46 |
*** amotoki has quit IRC | 11:47 | |
*** alexpilotti has joined #openstack-ironic | 11:53 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers https://review.openstack.org/220097 | 11:54 |
*** alexpilotti_ has joined #openstack-ironic | 11:58 | |
*** Marga_ has quit IRC | 12:01 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints https://review.openstack.org/218271 | 12:02 |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers https://review.openstack.org/220097 | 12:03 |
sinval | morning Ironicers | 12:06 |
sinval | does anyone knows if the current version of Ironic should work with OpenStack Kilo (Glance, Nova, Keystone and etc?) | 12:06 |
getvasanth | sinval: I am trying to get it working ;) | 12:08 |
sinval | getvasanth: Oh, great, let me know when you have some news about it, thanks | 12:13 |
*** romcheg1 has quit IRC | 12:14 | |
*** marzif has quit IRC | 12:14 | |
*** marzif has joined #openstack-ironic | 12:15 | |
getvasanth | sinval: sure | 12:15 |
TheJulia | good morning | 12:17 |
*** absubram has quit IRC | 12:24 | |
sinval | TheJulia: morning | 12:25 |
*** VikasC has quit IRC | 12:28 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Improve logging for VMs to identify IPA issues https://review.openstack.org/220112 | 12:30 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix HACKING.rst and canary test for merges https://review.openstack.org/220113 | 12:30 |
*** lucas-hungry is now known as lucasagomes | 12:30 | |
*** baoli has joined #openstack-ironic | 12:33 | |
*** trown|outttypeww is now known as trown | 12:36 | |
*** baoli_ has joined #openstack-ironic | 12:37 | |
*** baoli has quit IRC | 12:38 | |
*** dims has joined #openstack-ironic | 12:41 | |
trown | TheJulia: would you mind looking at https://review.openstack.org/#/c/211521/ again, and see if my comment clears things up? | 12:46 |
*** shoutm_ has joined #openstack-ironic | 12:47 | |
*** jjohnson2 has joined #openstack-ironic | 12:48 | |
*** shoutm has quit IRC | 12:48 | |
*** jcoufal has quit IRC | 12:49 | |
TheJulia | trown: I must have been on another planet earlier this week. Actually, I know I was | 12:53 |
trown | :), thanks for looking | 12:53 |
trown | I pinged in #tripleo to try to get folks to look at the actual fix patch | 12:54 |
*** PradeepV has quit IRC | 12:54 | |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Improve logging for VMs to identify IPA issues https://review.openstack.org/220112 | 12:55 |
TheJulia | thanks, I'd like to see it land at some point | 12:56 |
TheJulia | it will make me much happier and a CI job very happy | 12:56 |
openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix HACKING.rst and canary test for merges https://review.openstack.org/220113 | 12:56 |
*** lazy_prince has quit IRC | 12:57 | |
trown | TheJulia: ya...I broke that at a really bad time...CI has been pretty awful this week | 12:58 |
TheJulia | c'est la vie | 13:01 |
*** jcoufal has joined #openstack-ironic | 13:02 | |
*** rloo has joined #openstack-ironic | 13:04 | |
lucasagomes | jroll, around? | 13:08 |
*** lazy_prince has joined #openstack-ironic | 13:13 | |
rloo | hi lucasagomes, just looking at your comment: https://review.openstack.org/#/c/220003/2/ironic/drivers/modules/agent.py | 13:24 |
rloo | lucasagomes: I think we should change it to what you suggested | 13:24 |
rloo | hi everyone and lucasagomes :) | 13:24 |
lucasagomes | rloo, is not None? | 13:25 |
rloo | lucasagomes: yes. | 13:25 |
rloo | lucasagomes: otherwise, like you said, it isn't backwards compat | 13:25 |
lucasagomes | that's the placeholder, but just checking if the boot is set to something seems fair enuff | 13:26 |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers https://review.openstack.org/220097 | 13:26 |
lucasagomes | rloo, but peole running that code will have the base.py seeting "boot = None" | 13:26 |
lucasagomes | even if they driver doesn't specify an boot interface | 13:26 |
lucasagomes | it will be None | 13:26 |
lucasagomes | so the "if task.driver.boot:" checking if it's set will work | 13:27 |
rloo | lucasagomes: sorry. i didn't look at the code change, i was reading your comments only. yeah, it is ok as is. | 13:27 |
lucasagomes | rloo, yeah I got confused too and changed my vote later | 13:28 |
dtantsur | morning TheJulia, trown, rloo :) | 13:28 |
rloo | lucasagomes: yeah, you confused me. it's good now :) | 13:29 |
*** amotoki has joined #openstack-ironic | 13:29 | |
*** vvb1 has joined #openstack-ironic | 13:30 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers https://review.openstack.org/220097 | 13:31 |
lucasagomes | rloo, :-) | 13:31 |
rloo | lucasagomes: I put that patch on the list of stuff to land before cutting 4.1.0 this week | 13:32 |
lucasagomes | rloo, btw are you working on https://bugs.launchpad.net/ironic/+bug/1461140 ? | 13:32 |
openstack | Launchpad bug 1461140 in Ironic "conflict (HTTP 409) incorrect for some cases" [Medium,Confirmed] - Assigned to Ruby Loo (rloo) | 13:32 |
rloo | lucasagomes: no. i thought you and dmitry were looking into it. we discussed at some meeting awhile ago? | 13:32 |
dtantsur | trown, mind having a look at https://review.openstack.org/#/c/208375/ ? that's the basis for the whole profile matching work (and yeah, it's super long :) | 13:32 |
trown | dtantsur: ya, I have been trying to get to that one for a bit...sorry | 13:33 |
dtantsur | np at all | 13:33 |
lucasagomes | rloo, right... yeah perhaps I forgot :-( | 13:34 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints https://review.openstack.org/218271 | 13:34 |
lucasagomes | will take a look | 13:34 |
rloo | lucasagomes: i don't recall the details. Oh, I hope it wasn't that we shouldn't retry on 409. that would be bad. | 13:34 |
lucasagomes | rloo, did we decide anything whether use diff exceptions or retry-after? | 13:34 |
trown | dtantsur: it is a tough one to get interrupted during. if I do not get to it today, I will make sure to get up early and review it before anyone has a chance to bother me :) | 13:34 |
dtantsur | hehe, thanks! | 13:34 |
lucasagomes | rloo, yeah no idea... I will give it some thought | 13:34 |
dtantsur | yeah, it's way too huge, I just didn't figure out how to meaningfully split it | 13:34 |
rloo | lucasagomes: let me try to find the meeting where we discussed it | 13:35 |
lucasagomes | rloo, no bothers | 13:35 |
*** afaranha has joined #openstack-ironic | 13:35 | |
lucasagomes | rloo, we released an openstack installer not long ago | 13:35 |
lucasagomes | and since I've been pretty busy with it so I'm sure I'm the one who forgot stuff | 13:35 |
rloo | lucasagomes: we did? oh, RH. | 13:35 |
lucasagomes | rloo, I can find it, no bothers | 13:35 |
lucasagomes | yeah | 13:35 |
lucasagomes | RH | 13:35 |
*** afaranha has quit IRC | 13:36 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support https://review.openstack.org/208375 | 13:36 |
*** afaranha has joined #openstack-ironic | 13:37 | |
*** afaranha has joined #openstack-ironic | 13:37 | |
rloo | lucasagomes: http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-07-13-17.00.log.html | 13:39 |
lucasagomes | rloo, thanks! | 13:40 |
*** links has quit IRC | 13:40 | |
*** alexpilotti has quit IRC | 13:53 | |
*** alexpilotti_ is now known as alexpilotti | 13:53 | |
*** jroll has quit IRC | 13:54 | |
*** jroll has joined #openstack-ironic | 13:54 | |
jroll | lucasagomes: rloo dtantsur trown morning :) | 14:01 |
trown | morning jroll | 14:01 |
dtantsur | jroll, morning | 14:01 |
*** bnemec has quit IRC | 14:02 | |
trown | jroll: my fedora package review for IPA was accepted yesterday, so should have an IPA package in fedora rawhide today :) | 14:02 |
*** bnemec has joined #openstack-ironic | 14:02 | |
rloo | morning jroll | 14:02 |
jlvillal | Good morning Ironic | 14:03 |
jroll | trown: yay | 14:03 |
trown | less exciting for you than me :P, but it does mean that we are one step closer to destroying the bash ramdisk | 14:03 |
lucasagomes | jroll, morning! | 14:03 |
lucasagomes | jroll, btw re node reservation. I remember you said you had another idea in mind? Which wouldn't use the /reservation endpoint | 14:04 |
lucasagomes | jroll, you wanna talk about it? | 14:04 |
lucasagomes | just as a context, we have been hitting some races in the nova-scheduler and Ironic | 14:05 |
lucasagomes | so I would like to think about a solution for that, it could be in Nova or in Ironic | 14:05 |
trown | ya, nova-scheduler is really really bad at scheduling N instances to N nodes | 14:05 |
trown | N available nodes that is | 14:06 |
jroll | lucasagomes: I had ideas for the multi-compute things, I'm not sure if it will help scheduler races at all | 14:06 |
lucasagomes | fwiw there's a spec from Nikola to address the problem (or some of it) in nova (http://specs.openstack.org/openstack/nova-specs/specs/liberty/approved/host-state-level-locking.html ) | 14:06 |
jroll | I don't think it will, the scheduler in general is just racy as hell | 14:06 |
lucasagomes | right | 14:06 |
lucasagomes | jroll, the reservation actually would solve it AFAIUI | 14:06 |
jroll | the only thing that can actually solve problems is a distributed lock manager | 14:07 |
jroll | ... which I just now noticed, the reservation thing would be using ironic as the DLM, lol | 14:07 |
lucasagomes | yea | 14:08 |
lucasagomes | and the spec by nikola wouldn't solve it for multiple n-cpu | 14:08 |
lucasagomes | only for 1 n-cpu (the spec says it) | 14:08 |
jroll | lucasagomes: we could just as easily require zookeeper for the ironic scheduler bits and accomplish the same thing :P | 14:08 |
lucasagomes | jroll, yeah, food for thought | 14:09 |
lucasagomes | jroll, one thing that would help as well is fail fast when the node is already taken | 14:09 |
jroll | lucasagomes: I don't see where that spec says one n-cpu? | 14:09 |
lucasagomes | usually in nova it keeps retrying the request to add the instance_uuid in the node | 14:09 |
lucasagomes | which is already populated | 14:09 |
jroll | sounds more like one scheduler | 14:09 |
lucasagomes | so that's why I'm looking at the retry-after thing as well, so we don't retry on certain exceptions | 14:10 |
lucasagomes | and it would make it to fail faster and the retry filter would then get a new node | 14:10 |
*** harshs has joined #openstack-ironic | 14:10 | |
jroll | yeah, so to be clear you still get a reschedule yeah? it's not 'broken' just suboptimal? | 14:10 |
lucasagomes | single n-sche* | 14:10 |
lucasagomes | sorry | 14:10 |
lucasagomes | "It is worth noting that this proposal only looks at fixing data consistency among threads of a single nova-scheduler process." | 14:11 |
jroll | yeah | 14:11 |
jroll | which is really the only supported method | 14:11 |
lucasagomes | yeah | 14:11 |
jroll | now that said, I wish nova would just make this use a DLM | 14:11 |
jroll | and then you can run all the schedulers you want | 14:11 |
lucasagomes | jroll, yeah eventually it get reschedule'd | 14:11 |
jroll | even if it uses mysql as the DLM, which is horrible and bad and something we do | 14:11 |
dtantsur | ... or fails to get rescheduled | 14:12 |
jroll | that's still an improvement | 14:12 |
lucasagomes | yeah it will try 3 times (I think thta's the default for the retry filter) | 14:12 |
lucasagomes | and yeah it's possible to fail 3x | 14:12 |
jroll | so the way this multi-compute thing works is less racy, to be clear (I think) | 14:12 |
jroll | depending how hard things are cached | 14:12 |
jroll | but we could validate after choosing | 14:13 |
jroll | I need to write that up :( | 14:13 |
* jroll is a bad person | 14:13 | |
lucasagomes | heh happens | 14:14 |
*** saripurigopi has quit IRC | 14:14 | |
*** shoutm_ has quit IRC | 14:18 | |
*** alexpilo_ has joined #openstack-ironic | 14:20 | |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Fix minor grammar/comment description errors https://review.openstack.org/219551 | 14:24 |
openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-python-agent: Use a tiny connection wrapper in the heartbeat thread https://review.openstack.org/219905 | 14:26 |
getvasanth | lucasagomes: hi, 1 quick question, in ironic for a whole disk image, it boots with deploy_ramdisk & deploy_kernel and restarts the system and boots again, is this how it works? | 14:33 |
*** mgoddard1 has joined #openstack-ironic | 14:33 | |
*** devlaps has quit IRC | 14:33 | |
*** vvb1 has quit IRC | 14:34 | |
*** mtanino has joined #openstack-ironic | 14:35 | |
*** mgoddard has quit IRC | 14:35 | |
lucasagomes | getvasanth, yeah. You can use local boot so you don't need to PXE boot again for the second boot | 14:36 |
lucasagomes | or if you already deployed you probably can just set the boot device to disk | 14:37 |
lucasagomes | (make sure the full disk image has a bootloader) | 14:37 |
*** thrash is now known as thrash|biab | 14:38 | |
*** getvasanth has quit IRC | 14:39 | |
*** getvasanth has joined #openstack-ironic | 14:40 | |
getvasanth | lucasagomes: hi, 1 quick question, in ironic for a whole disk image, it boots with deploy_ramdisk & deploy_kernel and restarts the system and boots again, is this how it works? | 14:40 |
lucasagomes | getvasanth, heh network problems? | 14:40 |
*** garthb has joined #openstack-ironic | 14:40 | |
lucasagomes | <lucasagomes> getvasanth, yeah. You can use local boot so you don't need to PXE boot again for the second boot | 14:40 |
lucasagomes | <lucasagomes> or if you already deployed you probably can just set the boot device to disk | 14:40 |
lucasagomes | <lucasagomes> (make sure the full disk image has a bootloader) | 14:40 |
lucasagomes | jroll, dtantsur python-ironicclient 0.8.0 is now release, I wonder if we could pin it in the g-r | 14:41 |
lucasagomes | so that we can use things like native rety in the nova ironic driver etc | 14:41 |
getvasanth | lucasagomes: yes, network trouble :( | 14:41 |
lucasagomes | getvasanth, happens :-) | 14:41 |
dtantsur | lucasagomes ++ | 14:42 |
jroll | lucasagomes: I'm cool with that | 14:42 |
lucasagomes | jroll, I want to use custom fields as well to minimize the logs in nova | 14:42 |
lucasagomes | ok, will put a patch to pin it | 14:42 |
lucasagomes | today is the last day right? To ping things in the g-r | 14:43 |
*** Marga_ has joined #openstack-ironic | 14:43 | |
jroll | yes | 14:43 |
jroll | afaik | 14:43 |
lucasagomes | ok | 14:43 |
*** mgoddard1 has quit IRC | 14:43 | |
jroll | lucasagomes: if you bug people for reviews maybe add in https://review.openstack.org/#/c/219011/ :P | 14:44 |
lucasagomes | jroll, will do! hah | 14:44 |
*** mgoddard has joined #openstack-ironic | 14:44 | |
*** romainh has left #openstack-ironic | 14:44 | |
getvasanth | lucasagomes: thanks :) | 14:46 |
lucasagomes | np | 14:47 |
*** zz_natorious is now known as natorious | 14:47 | |
lucasagomes | https://review.openstack.org/220174 | 14:50 |
lucasagomes | for those interested | 14:50 |
*** vvb1 has joined #openstack-ironic | 14:51 | |
yuriyz | morning/evening Ironic, glad to see ironic-lib on pypi, but Programming Language :: Python :: 2.6 looks wrong | 14:52 |
*** mtanino has quit IRC | 14:53 | |
lucasagomes | yuriyz, morning | 14:58 |
lucasagomes | yuriyz, heh indeed | 14:58 |
dtantsur | good catch! somehow I missed it when cleaning up setup.cfg... | 14:58 |
*** jistr is now known as jistr|call | 15:02 | |
*** Marga_ has quit IRC | 15:05 | |
openstackgerrit | Merged openstack/ironic-python-agent: Three attempts message is incorrect https://review.openstack.org/219913 | 15:07 |
*** thrash|biab is now known as thrash | 15:09 | |
*** alexpilotti_ has joined #openstack-ironic | 15:16 | |
*** alexpilotti has quit IRC | 15:17 | |
*** alexpilotti_ is now known as alexpilotti | 15:17 | |
*** harshs has quit IRC | 15:17 | |
*** alexpilo_ has quit IRC | 15:20 | |
NobodyCam | good late morning Ironicers | 15:22 |
NobodyCam | :-p | 15:22 |
jroll | morning NobodyCam :) | 15:24 |
*** dims has quit IRC | 15:26 | |
lucasagomes | NobodyCam, hi there | 15:26 |
NobodyCam | good morning lucasagomes & jroll | 15:31 |
NobodyCam | I'm moving slow this mornign | 15:31 |
NobodyCam | :-p | 15:31 |
NobodyCam | ruff night with the kids :) | 15:31 |
dtantsur | NobodyCam, morning | 15:31 |
NobodyCam | morning dtantsur :) | 15:31 |
*** dims has joined #openstack-ironic | 15:36 | |
*** mtanino has joined #openstack-ironic | 15:36 | |
*** amotoki has quit IRC | 15:40 | |
rloo | morning NobodyCam. I hope the kids are ok now. | 15:41 |
rloo | hi lucasagomes, was wondering what the status is for deprecating the bash ramdisk. | 15:43 |
rloo | lucasagomes: https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit?pli=1#gid=1338037272 | 15:43 |
rloo | lucasagomes: still need code/patch in ironic? | 15:43 |
*** jistr|call is now known as jistr | 15:44 | |
*** alexpilo_ has joined #openstack-ironic | 15:46 | |
lucasagomes | rloo, I think in Ironic is grand | 15:46 |
lucasagomes | I just rebased two patches in the tripleo-incubator | 15:46 |
lucasagomes | there's a patch in DIB as well | 15:46 |
lucasagomes | <openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Drop USE_IRONIC from tripleo-incubator https://review.openstack.org<openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Drop USE_IRONIC from tripleo-incubator https://review.openstack.org/194161 | 15:46 |
lucasagomes | <openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Use ironic-python-agent as the default deploy ramdisk https://review.openstack.org/210461/194161 | 15:46 |
lucasagomes | <openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Use ironic-python-agent as the default deploy ramdisk https://review.openstack.org/210461 | 15:46 |
lucasagomes | damn that's messy | 15:47 |
lucasagomes | https://review.openstack.org/194161 and https://review.openstack.org/210461 | 15:47 |
rloo | lucasagomes: ok good. even w/o those non-ironic patches landing, we can still announce that we're deprecating the ramdisk starting in M or N, and removing it in N or O? | 15:47 |
rloo | lucasagomes: or deprecating it starting in L? I'm losing track of time. | 15:48 |
lucasagomes | rloo, well, we need to discuss it. Cause it would be important to have those patches in the other projects | 15:48 |
lucasagomes | rloo, we can't just remove the endpoints if the tripleo-ci is not updated to use IPA for e.g | 15:48 |
lucasagomes | or we are going to break it | 15:48 |
lucasagomes | remove the endpoints in the Ironic API I mean | 15:48 |
lucasagomes | rloo, yeah probably in L :-/ | 15:48 |
rloo | lucasagomes: ok. was just wondering what the priority was for getting those non-ironic patches landed in L. cuz there are ironic patches that we want to land in L and there's only so much time... | 15:48 |
rloo | lucasagomes: well, deprecating and removing are two different things/times. | 15:49 |
lucasagomes | rloo, well I'm kinda confident we can get those patches merged in this cycle | 15:49 |
lucasagomes | I will ping people | 15:49 |
rloo | lucasagomes: ok thx. can i update the google doc then, that it doesn't require any patches to ironic? | 15:49 |
lucasagomes | rloo, yeah we are probably removing only in N | 15:49 |
lucasagomes | rloo, not that I'm aware off | 15:50 |
lucasagomes | of* | 15:50 |
* lucasagomes thinks | 15:50 | |
rloo | lucasagomes: do we 'deprecate' log msgs in ironic code, where the ramdisk stuff is handled? | 15:50 |
NobodyCam | mornign rloo | 15:50 |
NobodyCam | rloo: there doing okay... just really restless lastnight | 15:50 |
lucasagomes | rloo, yeah https://review.openstack.org/#/c/209188/ | 15:50 |
lucasagomes | docs seems fine too (I will double check) | 15:51 |
lucasagomes | all the docs now are building IPA ramdisks instead of bash ones | 15:51 |
rloo | lucasagomes: awesome. if it isn't fine, then it is a bug, we can deal with that in 2+ weeks ;) | 15:51 |
*** absubram has joined #openstack-ironic | 15:51 | |
lucasagomes | https://review.openstack.org/#/c/209189/ | 15:51 |
rloo | NobodyCam: good to hear. I don't think it was a full moon last night either. | 15:51 |
lucasagomes | rloo, ack | 15:51 |
lucasagomes | rloo, yeah it seems just the non-ironic patches needs to land now | 15:52 |
rloo | thx lucasagomes. will update google doc. | 15:52 |
lucasagomes | rloo, ok thank you | 15:52 |
dtantsur | and I need to make one of J* review inspector IPA patch :) | 15:52 |
* dtantsur also tries to get rid of bash ramdisk | 15:52 | |
* lucasagomes was just testing nova ironic driver with ironiclcient 0.8.0 to minimize the logs | 15:54 | |
lucasagomes | so much better, damn | 15:54 |
lucasagomes | just to not have that configdrive string there makes a lot of difference | 15:54 |
dtantsur | I can imagine | 15:54 |
*** mgoddard has quit IRC | 16:01 | |
*** mgoddard has joined #openstack-ironic | 16:02 | |
JoshNang | dtantsur: i'll take a look. https://review.openstack.org/#/c/205587/ right? | 16:02 |
dtantsur | JoshNang, correct, thanks a lot! | 16:03 |
*** marzif has quit IRC | 16:04 | |
*** getvasanth has quit IRC | 16:06 | |
*** jistr has quit IRC | 16:06 | |
*** jaypipes has joined #openstack-ironic | 16:07 | |
*** ijw_ has joined #openstack-ironic | 16:08 | |
jroll | lucasagomes: rloo I think we have everything merged for 4.1 now yes? | 16:12 |
rloo | jroll: not yet. sec, let me check. | 16:12 |
* lucasagomes checking | 16:12 | |
*** VikasC has joined #openstack-ironic | 16:12 | |
jroll | I had three on my list: enroll docs, release notes, and lucas' revert | 16:12 |
lucasagomes | https://review.openstack.org/#/c/220003/ < not merged yet | 16:13 |
jroll | oh, new ilo | 16:13 |
jroll | yeah | 16:13 |
rloo | jroll: the enroll docs hasn't merged yet either | 16:13 |
jroll | oh heh | 16:13 |
jroll | slow gate | 16:13 |
jroll | so, here's my plan: | 16:13 |
lucasagomes | where's the enroll docs one? | 16:13 |
jroll | intel/rackspace are having a bug bash today, includes some ironic work (y'all are welcome to join) | 16:14 |
lucasagomes | oh found it | 16:14 |
jroll | and friday is bad for releasing | 16:14 |
jroll | so I'm thinking 4.1 tuesday | 16:14 |
lucasagomes | jroll, I'm in the office today and will go home soon :-( | 16:14 |
jroll | soft freeze beginning thurs 9/17 | 16:14 |
rloo | jroll: i'm fine with tuesday. | 16:14 |
lucasagomes | before it closes | 16:14 |
jroll | 4.2 9/24 | 16:14 |
lucasagomes | but yeah tuesday seems ok | 16:14 |
jroll | lucasagomes: yeah no worries :) | 16:14 |
rloo | jroll: IF the gate is fast and those two land later today, you're welcome to cut a release if you want. | 16:15 |
jroll | so anyway if y'all have bugs that aren't fixed yet that you'd like fixed for 4.1 let me know and I'll make sure they're on the list :) | 16:15 |
jroll | rloo: nod, going to focus on bugs today though | 16:15 |
NobodyCam | nice https://review.openstack.org/#/c/218879 (disk image builder ipa patch) landed this mornign :_ | 16:15 |
NobodyCam | s/_/)/ | 16:15 |
rloo | jroll: wrt the bug bash, awesome. I'm going to bow out; I think i should focus on reviewing patches for features. not that this is something i love to do but ... | 16:15 |
jroll | rloo: yeah, no worries, I'm also going to try to review some feature patches today | 16:16 |
*** puranamr has joined #openstack-ironic | 16:16 | |
lucasagomes | rloo, I agree with ur comments on the enroll docs | 16:17 |
*** karimb has quit IRC | 16:17 | |
lucasagomes | perhaps we can update it ? | 16:17 |
lucasagomes | or NobodyCam can take a look so we land it soonish? | 16:18 |
*** derekh has quit IRC | 16:19 | |
rloo | lucasagomes: isn't the enroll doc approved? | 16:19 |
lucasagomes | rloo, nop, https://review.openstack.org/#/c/209251/ | 16:19 |
jroll | lucasagomes: see her last comment | 16:19 |
lucasagomes | ohh | 16:20 |
lucasagomes | damn | 16:20 |
* lucasagomes was reviewing the wrong thing | 16:20 | |
rloo | lucasagomes: crap. sorry about that. maybe i forgot to update the link in the etherpad. | 16:20 |
lucasagomes | ok I will abandon that one then, ack? | 16:20 |
*** ndipanov has quit IRC | 16:21 | |
jroll | yeah +1 | 16:21 |
rloo | lucasagomes: yeah, thx. i forgot that we could abandon them. | 16:22 |
lucasagomes | done :-) no bothers | 16:23 |
dtantsur | see you tomorrow folks, happy bug bash for some of you :) | 16:29 |
rloo | bye dtantsur | 16:29 |
NobodyCam | have a good night dtantsur | 16:29 |
*** dtantsur is now known as dtantsur|afk | 16:31 | |
lucasagomes | dtantsur|afk, see ya | 16:31 |
jroll | \o dtantsur|afk | 16:32 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC https://review.openstack.org/206243 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API https://review.openstack.org/206244 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net https://review.openstack.org/206245 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic https://review.openstack.org/213262 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB https://review.openstack.org/206232 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs https://review.openstack.org/206238 | 16:34 |
openstackgerrit | Om Kumar proposed openstack/ironic: Add network provider interface and implementations https://review.openstack.org/139687 | 16:34 |
*** e0ne has quit IRC | 16:34 | |
lazy_prince | Sukhdev: changes pushed ^^ | 16:34 |
jroll | wheeee | 16:36 |
openstackgerrit | Sam Betts proposed openstack/ironic: Add Cisco IMC PXE Driver https://review.openstack.org/219253 | 16:36 |
lucasagomes | folks I'm heading home | 16:37 |
lucasagomes | have a good night everyone! | 16:37 |
lucasagomes | see y'all later | 16:37 |
jroll | nn \o | 16:37 |
*** lucasagomes has quit IRC | 16:39 | |
*** ifarkas has quit IRC | 16:40 | |
NobodyCam | have a good niht lucas | 16:41 |
*** alexpilo_ has quit IRC | 16:41 | |
NobodyCam | night even | 16:41 |
* NobodyCam cann't type today | 16:41 | |
*** meghal has joined #openstack-ironic | 16:42 | |
*** meghal1 has joined #openstack-ironic | 16:45 | |
*** meghal has quit IRC | 16:48 | |
*** liliars has joined #openstack-ironic | 16:54 | |
*** meghal1 has quit IRC | 16:58 | |
*** priteau has quit IRC | 16:58 | |
*** harshs has joined #openstack-ironic | 16:59 | |
*** jrist-afk is now known as JRIST | 17:00 | |
*** JRIST is now known as jrist | 17:00 | |
*** puranamr has quit IRC | 17:02 | |
*** lazy_prince has quit IRC | 17:04 | |
sambetts | I'm heading off too, cy'all tomorrow | 17:04 |
*** deray has joined #openstack-ironic | 17:05 | |
*** alexpilo_ has joined #openstack-ironic | 17:07 | |
*** priteau has joined #openstack-ironic | 17:10 | |
*** puranamr has joined #openstack-ironic | 17:13 | |
*** achanda has joined #openstack-ironic | 17:14 | |
*** meghal has joined #openstack-ironic | 17:17 | |
jroll | node.properties[local_gb] can be a float, right? | 17:18 |
jroll | riiiiiight? | 17:18 |
jroll | maybe that's a nova question | 17:18 |
*** puranamr has quit IRC | 17:19 | |
jroll | int it is | 17:20 |
*** achanda has quit IRC | 17:21 | |
*** Nisha has joined #openstack-ironic | 17:21 | |
*** achanda has joined #openstack-ironic | 17:22 | |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments https://review.openstack.org/219390 | 17:27 |
openstackgerrit | John L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments https://review.openstack.org/219390 | 17:28 |
*** penick has joined #openstack-ironic | 17:32 | |
*** Nisha_away has joined #openstack-ironic | 17:33 | |
*** Nisha has quit IRC | 17:33 | |
*** meghal has quit IRC | 17:33 | |
*** meghal has joined #openstack-ironic | 17:34 | |
jroll | does anyone mind me closing https://bugs.launchpad.net/ironic/+bug/1390432 as a dupe of https://bugs.launchpad.net/ironic/+bug/1461239 | 17:35 |
openstack | Launchpad bug 1390432 in Ironic "Changes in mutable fields (e.g. dicts) of Objects are not detected" [Wishlist,Confirmed] - Assigned to Tan Lin (tan-lin-good) | 17:35 |
jroll | rloo: jlvillal ^ | 17:35 |
openstack | Launchpad bug 1461239 in Ironic "replace objects with oslo.versionedobjects" [Low,In progress] - Assigned to Tan Lin (tan-lin-good) | 17:35 |
jroll | in theory could be fixed earlier but meh | 17:35 |
jlvillal | jroll: lintan might object. I think 1390432 is a portion of the 1461239 work. He might be breaking it into smaller pieces to keep track of everything. | 17:37 |
jroll | jlvillal: this bug is already fixed in o.vo | 17:37 |
*** meghal has quit IRC | 17:37 | |
*** meghal has joined #openstack-ironic | 17:38 | |
jroll | jlvillal: let me rephrase, it was fixed in nova at some point, and that code was migrated to o.vo. I haven't confirmed there's no regression, but I assume there is not | 17:38 |
jlvillal | jroll: Ah. Then I think you are right that it is a dupe. | 17:38 |
jroll | k, thanks | 17:38 |
jlvillal | jroll: It would be nice to get that in. With the weird having to create a temporary variable in order to modify a dict hack-around. | 17:39 |
jlvillal | And then me cluelessly saying, why is there this temporary variable here. Doesn't seem needed :) | 17:39 |
jroll | jlvillal: of course, I hate that bug so much :) | 17:39 |
jroll | when you forget it while coding it's really hard to debug | 17:39 |
jlvillal | Yeah :( | 17:40 |
*** absubram has quit IRC | 17:43 | |
*** vishwanathj has quit IRC | 17:43 | |
*** e0ne has joined #openstack-ironic | 17:46 | |
*** aarefiev22 has joined #openstack-ironic | 17:46 | |
*** ijw_ has quit IRC | 17:48 | |
*** bradjones has joined #openstack-ironic | 17:53 | |
*** bradjones has quit IRC | 17:53 | |
*** bradjones has joined #openstack-ironic | 17:53 | |
*** Nisha has joined #openstack-ironic | 17:54 | |
*** Nisha_away has quit IRC | 17:54 | |
rloo | jroll: if you're sure the fix is in oslo.versionedobjects, then I'm fine. if you're not, we should leave it open for someone to verify. | 17:54 |
*** VikasC has quit IRC | 17:55 | |
rloo | jroll: actually, after thinking for 30 secs more, i think it is worth keeping separate. it'd be good to have a separate bug to track fixing that; otherwise it gets hidden in 'migrate to use oslo.versionedobjects' | 17:56 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add APIs for RAID configuration https://review.openstack.org/196007 | 17:56 |
*** VikasC has joined #openstack-ironic | 17:57 | |
*** alexpilo_ has quit IRC | 18:00 | |
*** deray has quit IRC | 18:04 | |
JayF | To make sure I didn't just tell someone wrong: test-requirements don't have to be in global-requirements, right? | 18:11 |
*** Nisha has quit IRC | 18:13 | |
*** pelix has quit IRC | 18:17 | |
jroll | JayF: they do | 18:17 |
jroll | rloo: idk, maybe you're right hm | 18:18 |
rloo | jroll, JayF: if you mean the GLOBAL requirements, they do. If you mean in our 'requirements.txt', they don't. | 18:18 |
JayF | rloo: I did mean in global requirements :( damn | 18:19 |
JayF | lekha: ^ | 18:19 |
rloo | JayF: better get it in quick then! | 18:19 |
*** garthb has quit IRC | 18:19 | |
JayF | jlvillal: should https://review.openstack.org/#/c/219390/ have a bug associated? | 18:22 |
jlvillal | JayF: I'm not sure. I can create one if desired. | 18:23 |
JayF | jlvillal: I was just asking b/c it was on the bug bash etherpad as a review | 18:23 |
jlvillal | JayF: It is more of an enhancement than bug fix. | 18:24 |
*** alexpilo_ has joined #openstack-ironic | 18:26 | |
jlvillal | JayF: And for sure not a high priority | 18:27 |
*** marzif has joined #openstack-ironic | 18:28 | |
JayF | jlvillal: no problem, still going to review it :) | 18:29 |
JayF | jlvillal: just wanted to make sure | 18:30 |
jlvillal | JayF: Thanks | 18:30 |
*** harlowja has quit IRC | 18:31 | |
lekha | jlvillal: I was thinking of picking up the task of adding functional tests for the pythin-ironicclient using mimic, just like I did for arsenal. | 18:32 |
*** meghal has quit IRC | 18:32 | |
*** meghal has joined #openstack-ironic | 18:32 | |
*** harlowja has joined #openstack-ironic | 18:32 | |
jlvillal | lekha: That would be great. How does that compare to how the python-novaclient does it? | 18:32 |
jlvillal | Similar? Very different? | 18:32 |
lekha | jlvillal: I have not looked at the python-novaclient's functional tests, but I would think very different cause its going to be using mimic. | 18:33 |
jlvillal | lekha: Okay. I like the idea of the mimic stuff. Sound great to me. | 18:34 |
lekha | jlvillal: awesome :) | 18:34 |
*** Nisha has joined #openstack-ironic | 18:34 | |
jlvillal | lekha: Is mimic already supported in the Jenkins test jobs? | 18:34 |
JayF | jlvillal: I'm about to push a PR to add it to global requirements | 18:35 |
*** Nisha has quit IRC | 18:35 | |
JayF | jlvillal: as far as running it is concerned, it's all hidden behind "tox" just like you'd expect | 18:35 |
jlvillal | JayF: Ah great. I was looking and didn't see it. | 18:35 |
*** Nisha has joined #openstack-ironic | 18:35 | |
JayF | jlvillal: so no devstack or gate changes needed | 18:35 |
lekha | jvillal: what JayF said | 18:35 |
jlvillal | lekha: JayF: Thanks | 18:36 |
JayF | https://review.openstack.org/#/c/220268/ <-- adding mimic to g-r | 18:36 |
*** marzif has quit IRC | 18:38 | |
*** Nisha has quit IRC | 18:40 | |
*** ijw_ has joined #openstack-ironic | 18:50 | |
*** ijw_ has quit IRC | 18:55 | |
openstackgerrit | Juliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers https://review.openstack.org/220097 | 18:59 |
*** VikasC has quit IRC | 19:00 | |
jroll | JayF: left some commentary -- I'm set up to do constraints generation if you want me to grab that at some point | 19:02 |
JayF | jroll: please do fix it | 19:03 |
jroll | JayF: booo :) | 19:04 |
JayF | jroll: lekha said 1.6.0 is the right version | 19:04 |
jroll | JayF: lekha thanks | 19:04 |
*** meghal has quit IRC | 19:08 | |
*** alexpilo_ has quit IRC | 19:08 | |
*** meghal has joined #openstack-ironic | 19:10 | |
*** MattMan1 has joined #openstack-ironic | 19:17 | |
aarefiev22 | jroll: hi, are u around? | 19:17 |
*** achanda has quit IRC | 19:18 | |
*** priteau has quit IRC | 19:18 | |
jroll | aarefiev22: hi | 19:19 |
aarefiev22 | jroll: could you take a look at https://review.openstack.org/#/c/217135/ please, it blocks further work on functional tests | 19:19 |
jroll | aarefiev22: sure, I'll add it to my list :) | 19:20 |
aarefiev22 | jroll: thanks | 19:20 |
JayF | I'll take a look too, although I'm not core | 19:20 |
lekha | jroll: I have add/delete nodes for ironic in https://github.com/rackerlabs/mimic/pull/387 | 19:20 |
lekha | jroll: will release the next version of mimic with it, possibly today | 19:20 |
jroll | lekha: /me adds to list | 19:21 |
*** meghal has quit IRC | 19:23 | |
jroll | aarefiev22: two questions | 19:25 |
jroll | 1) we don't need __init__.py, right? | 19:25 |
jroll | since there's no python code in hooks/ | 19:25 |
jroll | 2) why not something like tox -e functional? | 19:26 |
*** alexpilotti has quit IRC | 19:26 | |
*** alexpilotti has joined #openstack-ironic | 19:26 | |
jroll | or does this just fix tox -e functional? | 19:26 |
* jroll not sure he gets it | 19:26 | |
*** alexpilotti has quit IRC | 19:28 | |
aarefiev22 | jroll: yes about __init__ you are right , fail( | 19:28 |
*** alexpilotti has joined #openstack-ironic | 19:28 | |
jroll | it happens :) | 19:28 |
aarefiev22 | jroll: Many Eyes Make All Bugs Shallow) | 19:28 |
*** Marga_ has joined #openstack-ironic | 19:29 | |
*** dguerri is now known as dguerri` | 19:29 | |
jroll | aarefiev22: so if you remove this file, I'm happy with the change | 19:30 |
jroll | aarefiev22: though I'm not sure I fully understand why that fixes it | 19:30 |
*** puranamr has joined #openstack-ironic | 19:31 | |
*** alexpilotti has quit IRC | 19:33 | |
rloo | jroll: is this a 'valid' patch, or has it been superceded by other patches? | 19:34 |
*** ijw_ has joined #openstack-ironic | 19:34 | |
rloo | jroll: I got that link via links in our priority googledoc | 19:34 |
jroll | rloo: which one? | 19:35 |
rloo | jroll: https://review.openstack.org/#/c/155848/ | 19:35 |
rloo | jroll: sorry, guess that would help | 19:35 |
jroll | rloo: heh, I believe that's been superceded, need to abandon | 19:35 |
*** puranamr has quit IRC | 19:35 | |
jroll | and done, sorry about that | 19:35 |
rloo | jroll: thx. i'll see if i can find the patches for ironic-neutron integration. | 19:36 |
jroll | rloo: should be https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bp/network_provider,n,z | 19:36 |
jroll | jenkinsssss why | 19:36 |
jroll | oh, that's still the wsme update problem | 19:36 |
rloo | jroll: ok thx. i'll update our spreadsheet | 19:36 |
jroll | thanks! | 19:37 |
aarefiev22 | jroll: thanks, I'll update it tomorrow(in ~10h), the problem user stack can't get os credentials w/o preserve environment variables, I thought it would be nice to have script on ironic side, we can change it and see job results | 19:37 |
*** alexpilotti has joined #openstack-ironic | 19:39 | |
jroll | aarefiev22: ok, I can just update it for you today if that's ok :) | 19:39 |
aarefiev22 | jroll: sure, thanks a lot, I'm writting from phone | 19:40 |
rloo | NobodyCam, BadCub, or jroll do you have powers? Need to approve this BP: https://blueprints.launchpad.net/ironic/+spec/ironic-ml2-integration | 19:40 |
aarefiev22 | jroll: it would be my next level - update paches from smartphone) | 19:40 |
rloo | NobodyCam, BadCub, jroll: and set url to http://specs.openstack.org/openstack/ironic-specs/specs/approved/ironic-ml2-integration.html | 19:41 |
jroll | rloo: I haz no power | 19:41 |
jroll | aarefiev22: lol, yes will update today :) | 19:41 |
rloo | jroll: so sad. too much power in hp hands me thinks :) | 19:41 |
jroll | rloo: :/ | 19:42 |
jroll | rloo: only deva has power to update that group, too | 19:42 |
rloo | jroll: of course, we should be able to automate all this. oh well, it isn't a blocker. | 19:42 |
rloo | jroll: will let deva clean it up when he's back | 19:42 |
jroll | rloo: indeed. | 19:43 |
jroll | actually... | 19:43 |
jroll | dhellmann: do you have the power to add me to ironic-drivers on launchpad? | 19:43 |
dhellmann | jroll: maybe, let's see | 19:43 |
dhellmann | jroll: is your lp id "jroll" | 19:43 |
dhellmann | ? | 19:44 |
dhellmann | nm, I found you | 19:44 |
dhellmann | jroll: done | 19:44 |
rloo | ooo, dhellmann has lots of power. good person to know :D thx dhellmann | 19:45 |
dhellmann | rloo: :-) | 19:45 |
jroll | dhellmann: yay, thanks! | 19:45 |
jroll | rloo: approved and url set | 19:46 |
rloo | thx jroll | 19:46 |
jroll | np | 19:46 |
rloo | NobodyCam, BadCub: ^^ never mind; we got it done w/o you. | 19:47 |
BadCub | cool. Didn't mean to ignore, been heads down on internal things that required immediate attention | 19:48 |
*** aarefiev22 has quit IRC | 19:48 | |
rloo | BadCub: no worries | 19:48 |
rloo | jroll: qq. does the ironic-neutron work need the pluggable network providers to be done? It is mentioned in the spec sort of in passing. want to know what i should be reviewing first. | 19:51 |
jroll | rloo: they're kind of the same thing, they're in the same patch chain | 19:51 |
jroll | rloo: the "ml2 integration" spec covers the data model so those are first | 19:52 |
rloo | jroll: oh. ok, | 19:52 |
jroll | the "add portgroups" patches | 19:52 |
rloo | jroll: got it. thx. | 19:53 |
jroll | np | 19:53 |
NobodyCam | rloo: lol :) | 19:54 |
*** meghal has joined #openstack-ironic | 19:55 | |
openstackgerrit | Jim Rollenhagen proposed openstack/python-ironicclient: Fix functional tests job https://review.openstack.org/217135 | 19:55 |
jroll | ^ just removed unnecessary __init__.py, easy +2 otherwise | 19:56 |
*** Nisha has joined #openstack-ironic | 19:56 | |
JoshNang | jroll: i'm just gonna fix up 215505, looks like a minor change to make it landable | 19:57 |
jroll | JoshNang: by all means | 19:57 |
*** Nisha has quit IRC | 19:57 | |
*** Nisha has joined #openstack-ironic | 19:57 | |
openstackgerrit | Lindsay Kusmierek proposed openstack/ironic-python-agent: Fix plural agreement grammar error https://review.openstack.org/220293 | 19:58 |
*** Nisha has quit IRC | 19:58 | |
*** Nisha has joined #openstack-ironic | 19:59 | |
*** Marga_ has quit IRC | 20:01 | |
*** derekh has joined #openstack-ironic | 20:02 | |
*** Nisha has quit IRC | 20:08 | |
*** e0ne has quit IRC | 20:12 | |
jroll | lekha: hmmmmm | 20:12 |
jroll | Collecting mimic>=1.6.0 (from -r global-requirements.txt (line 248)) | 20:12 |
jroll | Could not find a version that satisfies the requirement mimic>=1.6.0 (from -r global-requirements.txt (line 248)) (from versions: 1.0.0, 1.4.0, 1.5.0) | 20:12 |
jroll | Some externally hosted files were ignored as access to them may be unreliable (use --allow-external mimic to allow). | 20:12 |
jroll | No matching distribution found for mimic>=1.6.0 (from -r global-requirements.txt (line 248)) | 20:12 |
*** baoli_ has quit IRC | 20:13 | |
jroll | lekha: 1) allow-external is going away in 3 months, 2) this is weird and I'm not using a cache, I'm wondering if the allow-external thing is what's breaking thatr | 20:13 |
jroll | maybe a glyph question, idk, I don't immediately see external things in setup.py | 20:15 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Remove ordereddict as 2.6 is no longer supported https://review.openstack.org/220298 | 20:16 |
lekha | jroll: hmm, not sure why that would be. Looking | 20:16 |
jroll | lekha: no rush but it's going to block this thing | 20:17 |
jroll | also people are going to yell about twisted getting pulled in, heh | 20:17 |
jroll | (global-requirements thing) | 20:17 |
*** dguerri` is now known as dguerri | 20:18 | |
*** achanda has joined #openstack-ironic | 20:18 | |
*** alexpilotti has quit IRC | 20:19 | |
*** dguerri is now known as dguerri` | 20:20 | |
JayF | joshnang: jroll: ^ 220298 is another easy ipa review | 20:20 |
jroll | JayF: refresh | 20:20 |
*** derekh has quit IRC | 20:20 | |
JoshNang | +A'ed | 20:23 |
*** achanda has quit IRC | 20:24 | |
*** e0ne has joined #openstack-ironic | 20:25 | |
*** e0ne has quit IRC | 20:26 | |
*** Marga_ has joined #openstack-ironic | 20:27 | |
*** achanda has joined #openstack-ironic | 20:29 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Validate the input of properties https://review.openstack.org/215505 | 20:31 |
JoshNang | jroll: ^ | 20:33 |
jroll | nice, thanks | 20:33 |
*** meghal has quit IRC | 20:34 | |
*** achanda has quit IRC | 20:38 | |
*** thiagop has joined #openstack-ironic | 20:42 | |
*** priteau has joined #openstack-ironic | 20:42 | |
thiagop | morning Ironicers | 20:42 |
*** priteau has quit IRC | 20:43 | |
thiagop | quick question: you guys that work with external libs for drivers, do you log anything inside your libs using oslo_log? | 20:43 |
*** alexpilotti has joined #openstack-ironic | 20:45 | |
jroll | lekha: found the bug :/ | 20:46 |
*** achanda has joined #openstack-ironic | 20:47 | |
rloo | jroll, NobodyCam, BadCub: the first one who gets a chance, please update the URL for https://blueprints.launchpad.net/ironic/+spec/inband-raid-configuration to be http://specs.openstack.org/openstack/ironic-specs/specs/approved/inband-raid-configuration.html | 20:47 |
* BadCub looks | 20:49 | |
BadCub | rloo: ^^ done | 20:50 |
lekha | jroll: anything I ought to do in mimic | 20:50 |
rloo | thx BadCub. NobodyCam, jroll: ^^ don't bother, it's been done | 20:50 |
NobodyCam | hehehe :) | 20:51 |
*** trown is now known as trown|outttypeww | 20:51 | |
lekha | I am looking at the existing functional tests for the python-ironicclient, and it seems to run against ironic and I am trying to write to tests that can be run against mimic and not need devstack | 20:52 |
lekha | any thoughts on whether I should be doing that as another functional test | 20:53 |
lekha | jlvillal: ^ | 20:53 |
jroll | lekha: so for mimic... you're only publishing wheels, and only for python 2, so when it tries to install on py3 it just hard fails | 20:55 |
jroll | lekha: I think if you publish an sdist it should just work | 20:55 |
lekha | jroll: ok | 20:56 |
jroll | (I'm not sure if it runs on py3 or not, but we can solve that problem... eventually) | 20:56 |
jroll | stepping away for a while | 20:56 |
*** harshs has quit IRC | 20:59 | |
*** harshs has joined #openstack-ironic | 21:01 | |
lekha | jroll: I just release 1.7.0 with the add/delete node capabilities. And that has the sdist. | 21:02 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Ensure all methods in util.py have docstrings. https://review.openstack.org/220313 | 21:03 |
*** harshs has quit IRC | 21:03 | |
*** jjohnson2 has quit IRC | 21:03 | |
jroll | lekha: thanks I'll try it | 21:03 |
lekha | jroll: thanks :) | 21:03 |
*** meghal has joined #openstack-ironic | 21:03 | |
*** greghaynes has quit IRC | 21:10 | |
jroll | lekha: aaaand updated https://review.openstack.org/#/c/220268/ | 21:12 |
jroll | thanks for the assist | 21:13 |
jroll | actually stepping away for a while now | 21:13 |
jroll | ... maybe after infra convo, heh | 21:17 |
jlvillal | lekha: I am not sure what is the best idea. We really have very little in there right now. So I would be okay with getting rid of it, if you felt it was necessary. Not sure how others would feel. | 21:22 |
JoshNang | jlvillal: it looks like what's in the functional testing for the client runs against devstack, right? | 21:23 |
jlvillal | JoshNang: I think so. adam_g would know for sure. | 21:23 |
jlvillal | JoshNang: Are there test jobs that don't run against devstack? | 21:24 |
* jlvillal doesn't know how all the test jobs are configured. | 21:24 | |
JoshNang | jlvillal: unittests and ipa's functional jobs | 21:24 |
jlvillal | JoshNang: Ah yeah. Duh! | 21:25 |
adam_g | the python-ironicclient functiional test runs against a minimal devstack that has only ironic/rabbit/db running | 21:25 |
adam_g | (at least it did last time i looked) | 21:25 |
JoshNang | adam_g: thanks! i think it still does, or it's doing some magic i can't figure out | 21:25 |
jlvillal | adam_g: Thanks! I doubt it has changed. lekha is looking into using mimic. | 21:25 |
adam_g | whats mimic? | 21:25 |
adam_g | it should also be able to run against a venv'd ironic, a mocked ironic server, etc.. it runs against devstack b/c that was the easiest way to bootstrap a minimal ironic env to run against in the gate | 21:26 |
jlvillal | adam_g: I am not the expert. mimic pretends to be a service that responds to REST API calls. | 21:26 |
JoshNang | basically a mock for api services (as i understand it) | 21:26 |
jlvillal | I think you tell it, if you get this HTTP request, respond like this. lekha is the expert on it. | 21:27 |
adam_g | ok ya, so in theory that should work too assuming mimic is doing what its supposed to | 21:27 |
JoshNang | yeah, seems like they're both valuable in different ways | 21:27 |
JoshNang | i still like the 'actually talk to a running ironic instance' jobs | 21:28 |
adam_g | ya mimic sounds interesting. like i said, the gate job runs against devstack b/c setting up a devstack there with only a few services was easier than teaching infra how to run something else | 21:28 |
jlvillal | +1, good value in that. | 21:28 |
openstackgerrit | Merged openstack/ironic: Remove 'is_valid_event' method https://review.openstack.org/219918 | 21:29 |
* jlvillal wonders how we will know if an event is valid now... | 21:29 | |
*** penick has quit IRC | 21:29 | |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/218906 | 21:30 |
openstackgerrit | Merged openstack/ironic: agent_ilo fails to bring up instance https://review.openstack.org/220003 | 21:30 |
*** meghal has quit IRC | 21:30 | |
*** penick has joined #openstack-ironic | 21:32 | |
*** jcoufal has quit IRC | 21:32 | |
*** meghal has joined #openstack-ironic | 21:36 | |
lekha | jroll: thanks | 21:36 |
lekha | adam_g: mimic will allow for running the functional tests locally | 21:36 |
adam_g | lekha, in python-ironicclient, tools/run_functional.sh writes out a test.conf that tells the client used for testing where to talk to ironic. you could modify that to point to the mimic server and you should be good | 21:37 |
*** harshs has joined #openstack-ironic | 21:38 | |
lekha | adam_g: ha thanks! I have been staring at the functional tests and trying to figure where the conf file was coming from. | 21:38 |
adam_g | lekha, :) | 21:38 |
*** marzif has joined #openstack-ironic | 21:39 | |
*** harshs has quit IRC | 21:46 | |
*** vvb1 has quit IRC | 21:59 | |
*** harshs has joined #openstack-ironic | 22:01 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 22:08 |
*** meghal has quit IRC | 22:10 | |
*** penick has quit IRC | 22:14 | |
openstackgerrit | Thiago Paiva Brito proposed openstack/python-oneviewclient: Sending OneView Client implementation https://review.openstack.org/212796 | 22:26 |
thiagop | Good night folks | 22:26 |
thiagop | see you tomorrow | 22:26 |
jroll | nn thiagop :) | 22:26 |
thiagop | jroll: ? | 22:26 |
jroll | "night night" | 22:27 |
jroll | :P | 22:27 |
thiagop | jroll: awn, I understood as "no no", like "WAAIT!" | 22:27 |
jroll | WAIT WE LIKE YOU | 22:27 |
jroll | how's that? | 22:27 |
* thiagop is flattered | 22:28 | |
thiagop | have a good night, sir | 22:28 |
thiagop | ;) | 22:28 |
*** thiagop has quit IRC | 22:28 | |
jroll | you too | 22:28 |
*** penick has joined #openstack-ironic | 22:31 | |
*** greghaynes has joined #openstack-ironic | 22:32 | |
*** Marga_ has quit IRC | 22:35 | |
*** natorious is now known as zz_natorious | 22:37 | |
*** Marga_ has joined #openstack-ironic | 22:37 | |
*** MattMan1 has left #openstack-ironic | 22:42 | |
*** Marga_ has quit IRC | 22:43 | |
openstackgerrit | Merged openstack/ironic-python-agent: Fix plural agreement grammar error https://review.openstack.org/220293 | 22:43 |
*** Marga_ has joined #openstack-ironic | 22:44 | |
*** penick has quit IRC | 22:45 | |
*** harshs has quit IRC | 22:45 | |
*** penick has joined #openstack-ironic | 22:47 | |
openstackgerrit | Lindsay Kusmierek proposed openstack/ironic-python-agent: Add docstrings to the IPA api.controllers tree https://review.openstack.org/220339 | 22:52 |
jroll | JoshNang: pep8 on 215505 :( | 22:52 |
jroll | change looks fine though | 22:53 |
JoshNang | jroll: yeah, just saw that :/ my editor stopped doing the line at 80 chars for some reason | 22:54 |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Validate the input of properties https://review.openstack.org/215505 | 22:54 |
jroll | lol | 22:54 |
* JoshNang shakes fist at pycharm | 22:54 | |
jroll | set colorcolumn=80 | 22:55 |
jroll | ftw | 22:55 |
jroll | +2 btw | 22:55 |
JoshNang | \o/ | 22:56 |
JoshNang | thanks! | 22:56 |
jroll | thank you sir | 22:56 |
*** meghal has joined #openstack-ironic | 23:09 | |
jlvillal | JoshNang: I'll admit that patch confuses me :) | 23:09 |
*** Marga_ has quit IRC | 23:12 | |
*** meghal has quit IRC | 23:13 | |
*** zz_natorious is now known as natorious | 23:15 | |
JoshNang | jlvillal: how so? | 23:16 |
jlvillal | JoshNang: Mainly because I hadn't read the code about IronicObject. It makes sense now... | 23:16 |
openstackgerrit | Jay Faulkner proposed openstack/ironic-python-agent: Add docstrings for cmd/agent and extensions/clean https://review.openstack.org/220346 | 23:16 |
JoshNang | jlvillal: no worries. i started at the end of the chain earlier and was quite confused | 23:18 |
jlvillal | JoshNang: I was wondering if _validate_property() should just call self.obj_what_changed() | 23:19 |
* jlvillal is probably bikeshedding... | 23:19 | |
* jlvillal slaps himself lightly and tells himself to stop it. | 23:20 | |
*** puranamr has joined #openstack-ironic | 23:21 | |
*** marzif has quit IRC | 23:21 | |
JoshNang | jlvillal: i'd tell you to fight with tan lin about it ;) i was just fixing an error message so it would be landable | 23:21 |
jlvillal | :) | 23:21 |
*** Sukhdev_ has joined #openstack-ironic | 23:23 | |
jlvillal | JoshNang: +1 | 23:23 |
*** meghal has joined #openstack-ironic | 23:24 | |
jroll | JoshNang: are you actually working on https://bugs.launchpad.net/ironic/+bug/1483828 | 23:27 |
openstack | Launchpad bug 1483828 in Ironic "IPA exceptions are not printed properly" [High,In progress] - Assigned to Josh Gachnang (joshnang) | 23:27 |
JoshNang | jroll: that's done, i believe | 23:28 |
jroll | orly | 23:28 |
jroll | got a patch? | 23:28 |
JoshNang | https://review.openstack.org/#/c/211692/ | 23:28 |
JoshNang | boo, forgot to do closes-bug | 23:28 |
*** puranamr has quit IRC | 23:29 | |
*** penick has quit IRC | 23:29 | |
jroll | thanks | 23:29 |
*** bradjones has quit IRC | 23:31 | |
*** puranamr has joined #openstack-ironic | 23:31 | |
*** achanda has quit IRC | 23:32 | |
*** dims has quit IRC | 23:44 | |
*** ijw_ has quit IRC | 23:45 | |
*** puranamr has quit IRC | 23:48 | |
*** shoutm has joined #openstack-ironic | 23:50 | |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Add python3-dev[el] to list of prereqs for devs https://review.openstack.org/220352 | 23:50 |
*** dims has joined #openstack-ironic | 23:53 | |
*** derekh has joined #openstack-ironic | 23:53 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!