Thursday, 2015-09-03

naohirotjroll: as you mentioned in the last irc meeting log.00:00
*** absubram has joined #openstack-ironic00:01
*** alexpilotti has joined #openstack-ironic00:01
JoshNangnaohirot: he just left00:01
naohirotJoshNang: Hi good evening, Ok, thank you for letting me know.00:02
*** Haomeng has joined #openstack-ironic00:02
JoshNangnaohirot: good evening! np00:03
naohirotjroll: 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
naohirotjroll: ^^^^ http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-08-31-17.00.log.html00:04
*** Haomeng|2 has quit IRC00:05
*** zz_natorious is now known as natorious00:08
*** alexpilotti has quit IRC00:10
*** garthb__ has quit IRC00:18
*** goodygum has quit IRC00:19
*** mtanino_ has quit IRC00:21
*** meghal has quit IRC00:21
*** jcoufal has joined #openstack-ironic00:21
*** goodygum has joined #openstack-ironic00:21
openstackgerritMerged openstack/ironic-python-agent: Split functional and py27 tests  https://review.openstack.org/21473500:22
*** shadower has quit IRC00:23
*** shadower has joined #openstack-ironic00:23
*** achanda has joined #openstack-ironic00:28
*** natorious is now known as zz_natorious00:28
*** alexpilotti has joined #openstack-ironic00:32
*** Sukhdev_ has joined #openstack-ironic00:37
*** meghal has joined #openstack-ironic00:38
*** achanda has quit IRC00:51
*** achanda has joined #openstack-ironic00:52
*** achanda has quit IRC00:52
*** stendulker has joined #openstack-ironic00:55
*** shoutm_ has joined #openstack-ironic00:56
*** shoutm has quit IRC00:56
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor iscsi_ilo driver to use new boot interface  https://review.openstack.org/21653800:57
openstackgerritShivanand Tendulker proposed openstack/ironic: Refactor agent_ilo driver to use new boot interface  https://review.openstack.org/21710200:58
Sukhdev_NobodyCam: Finally, I am up and running - what a day!!!01:00
*** jiangfei has quit IRC01:01
*** stendulker has quit IRC01:08
*** saripurigopi has joined #openstack-ironic01:08
*** meghal has quit IRC01:11
jrollnaohirot: I'll try to look at it soon, my main priority right now is figuring out liberty release things01:17
openstackgerritJohn L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments  https://review.openstack.org/21939001:17
*** ijw__ has quit IRC01:20
naohirotjroll: Oh, hi jim01:20
jrollhi :)01:20
naohirotjroll: I know you are busy for many thing:)01:20
naohirotjroll: but what I'd like to know is if the direction I proposed is okay or not.01:21
naohirotjroll: there are two concerns as I described in01:22
jrollnaohirot: I understand, it's on my todo list, I will try to get to it01:23
jrollsorry I kind of dropped this, been busy with downstream + releases + vacation lately01:23
naohirothttps://review.openstack.org/#/c/207897/01:24
naohirotjroll: the last comment of ^^^01:24
jrollnaohirot: are you asking if I'm okay with splitting the patches into three pieces like that?01:25
jrollnaohirot: or the approach of each patch?01:26
naohirotjroll: 2 things I'd like to confirm are 1) cancel implementation 2) drop NMI from ipmidriver01:26
*** saripurigopi has quit IRC01:26
jrolloh, so not the last comment01:26
jrollrather the one from Aug 14 8:04 AM01:26
naohirotjroll: Oh, I put wrong url.01:27
naohirotjroll: Yeah, URL is correct, but not the last my comment :)01:27
jrollright, ok01:28
jrollnaohirot: I did see that comment - I haven't had time to process my thoughts on it yet01:28
naohirotjroll: the 5th my comment from the last,01:28
naohirotjroll: on Naohiro TamuraAug 15 12:04 AM01:28
jrollnaohirot: right, I still need to review it further01:29
naohirotjroll: Okay, In terms of dropping NMI from IPMItool, I considered muti tenant01:29
naohirotjroll: after switching to deploy network to tenant network, conductor cannot reach to Instance,01:30
naohirotjroll: so conductor only can communicate bmc01:31
naohirotjroll: however bmc cannot detect the status of OS01:31
naohirotjroll: 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
naohirotjroll: anyway have a good evening, thanks for your consideration :)01:33
jrollnaohirot: I'm not sure that's ipmitool specific, but I'll think about it more and leave comments in the review01:37
jrolland you're welcome, have a good day today :)01:38
naohirotjroll: Okay, I'll check your comment in the gerrit :)01:39
*** jcoufal has quit IRC01:41
*** achanda has joined #openstack-ironic01:44
*** saripurigopi has joined #openstack-ironic01:46
*** boris-42 has quit IRC01:50
*** jcoufal has joined #openstack-ironic01:52
*** saripurigopi has quit IRC01:52
*** baoli has quit IRC02:06
*** baoli has joined #openstack-ironic02:06
*** baoli has quit IRC02:07
*** baoli has joined #openstack-ironic02:08
openstackgerritMerged openstack/ironic: Removes unused posix-ipc requirement  https://review.openstack.org/21900302:11
*** jcoufal has quit IRC02:16
*** saripurigopi has joined #openstack-ironic02:17
saripurigopiMorning Ironic02:17
*** spandhe has quit IRC02:17
*** Sukhdev_ has quit IRC02:32
openstackgerritMerged openstack/ironic: Remove unused dep discover from test reqs  https://review.openstack.org/21831302:33
*** hakimo_ has quit IRC02:54
*** hakimo has joined #openstack-ironic02:57
*** Nisha has joined #openstack-ironic03:00
*** ramineni has joined #openstack-ironic03:03
openstackgerritOpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements  https://review.openstack.org/21890603:06
*** harshs has joined #openstack-ironic03:07
*** meghal has joined #openstack-ironic03:08
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706803:14
*** vvb1 has joined #openstack-ironic03:15
*** shoutm_ has quit IRC03:20
*** meghal1 has joined #openstack-ironic03:22
*** meghal has quit IRC03:23
*** Sukhdev has quit IRC03:26
*** Sukhdev has joined #openstack-ironic03:27
*** BadCub has quit IRC03:27
*** eglute has quit IRC03:27
*** zz_natorious has quit IRC03:27
*** BadCub has joined #openstack-ironic03:27
*** vishwanathj has quit IRC03:28
*** achanda has quit IRC03:28
*** davidlenwell has quit IRC03:29
*** dguerri` has quit IRC03:30
*** jroll has quit IRC03:30
*** zz_natorious has joined #openstack-ironic03:31
*** zz_natorious is now known as natorious03:32
*** eglute has joined #openstack-ironic03:32
*** davidlenwell has joined #openstack-ironic03:32
*** dguerri` has joined #openstack-ironic03:32
*** dguerri` is now known as dguerri03:32
*** dguerri has joined #openstack-ironic03:32
*** jroll has joined #openstack-ironic03:33
*** adam_g has quit IRC03:33
*** adam_g has joined #openstack-ironic03:35
*** adam_g has quit IRC03:35
*** adam_g has joined #openstack-ironic03:35
*** dims has joined #openstack-ironic03:35
*** shoutm has joined #openstack-ironic03:41
*** boris-42 has joined #openstack-ironic03:59
*** dims has quit IRC04:00
*** links has joined #openstack-ironic04:02
*** Sukhdev_ has joined #openstack-ironic04:06
*** amotoki has joined #openstack-ironic04:11
*** baoli has quit IRC04:14
*** vishwanathj has joined #openstack-ironic04:22
*** shoutm has quit IRC04:38
*** rameshg87 has joined #openstack-ironic04:42
*** puranamr has joined #openstack-ironic04:46
*** shoutm has joined #openstack-ironic04:47
*** PradeepV has joined #openstack-ironic04:51
PradeepVGood morning Ironic04:52
*** dims has joined #openstack-ironic04:59
*** puranamr has quit IRC05:04
*** dims has quit IRC05:06
*** puranamr has joined #openstack-ironic05:11
*** puranamr has quit IRC05:11
*** ukalifon has joined #openstack-ironic05:15
*** puranamr has joined #openstack-ironic05:16
*** puranamr has quit IRC05:18
*** puranamr has joined #openstack-ironic05:24
*** derekh has joined #openstack-ironic05:26
*** achanda has joined #openstack-ironic05:34
*** shoutm has quit IRC05:36
*** shoutm has joined #openstack-ironic05:37
*** achanda has quit IRC05:40
*** ishant has joined #openstack-ironic05:43
*** puranamr has quit IRC05:47
*** VikasC has joined #openstack-ironic05:49
*** shoutm has quit IRC05:58
*** shoutm has joined #openstack-ironic05:58
*** achanda has joined #openstack-ironic06:00
*** Sukhdev_ has quit IRC06:06
*** baoli has joined #openstack-ironic06:14
*** natorious is now known as zz_natorious06:18
openstackgerritNisha Agarwal proposed openstack/ironic: agent_ilo doesnt boot up the image  https://review.openstack.org/22000306:18
*** baoli has quit IRC06:19
*** achanda has quit IRC06:19
*** getvasanth has joined #openstack-ironic06:23
*** shoutm_ has joined #openstack-ironic06:25
*** shoutm has quit IRC06:26
*** harshs has quit IRC06:29
*** achanda has joined #openstack-ironic06:48
*** meghal1 has quit IRC06:53
*** dtantsur|afk is now known as dtantsur06:57
dtantsurMorning Ironic06:57
Nishadtantsur, o/07:01
Nishadtantsur, i responded to ur comments on https://review.openstack.org/#/c/201904/. Wanted to discuss on the inspector config variable for ports07:04
dtantsursure07:05
openstackgerritNisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers  https://review.openstack.org/19865607:10
getvasanthGood morning all, any pointers to create a bootable image with DIB07:10
getvasanthcreated a baremetal and a vm image both failed with "No or empty root= argument error"07:11
*** Marga_ has joined #openstack-ironic07:15
*** Marga_ has quit IRC07:15
Nishadtantsur, 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-ironic07:16
Nishadtantsur, or for all connected MACs07:16
dtantsurNisha, only that one (keep_ports means it won't delete any existing ports)07:16
Nishadtantsur, ilo drivers requirements is to create ports for all connected MACs07:17
Nishaand not only one07:17
dtantsurNisha, you can add setting add_ports=active to your documentation07:18
Nishathis will be a new option correct?07:20
Nishaand is not existing currently07:20
*** ifarkas has joined #openstack-ironic07:22
dtantsurNisha, it exists07:28
Nishadtantsur, ok. could you give me pointer for this setting07:29
dtantsurNisha, https://github.com/openstack/ironic-inspector/blob/master/example.conf#L538-L54507:29
Nishadtantsur, but if IP is not assigned? say 2 MACs are physically connected but IPs are not provided will this setting work?07:32
dtantsurNisha, no. that's how we define "active" currently07:32
Nishadtantsur, but during inspection we dont expect the neutron and ironic ports created for the MACs, correct?07:33
dtantsurNisha, right07:34
Nishathen how does active helps/differ from pxe?07:34
Nishaadd_ports=pxe, add_ports=all, add_ports=active07:35
Nishagetvasanth, what command line did u use?07:36
*** derekh has quit IRC07:36
dtantsurNisha, it depends on whether you have a DHCP server in your environment.. FWIW I'm open to redefining "active" there, or adding a new option07:36
Nishadtantsur, i would prefer a new option to add_ports07:37
Nishabecause active seems to be dependent on IP address07:38
Nishadtantsur, how about add_ports=connected07:38
dtantsurlooks good. or we can change what "active" means, I don't care much.07:38
Nishawill it not be a backward compatibility break for inspector07:41
Nisha?07:41
openstackgerritNisha Agarwal proposed openstack/ironic: agent_ilo fails to bring up instance  https://review.openstack.org/22000307:41
Nishadtantsur, ^^^07:43
Nishadtantsur, will it not be a compatibility break for inspector07:44
Nishadtantsur, one more query apart from this....you have suggested following : "Enhance IPA inspection support to discover the above capabilities"07:44
Nishadtantsur, now the inspection is supposed to be working only with IPA?07:45
openstackgerritMerged openstack/ironic: Revert "Do not overwrite the iPXE boot script on every deployment"  https://review.openstack.org/21974907:46
Nishadtantsur, the bash ramdisk is deprecated in liberty but not removed, so IMO it should be allowed in both ramdisk07:46
openstackgerritMerged openstack/ironic: Add deprecation warning to periodic tasks with parallel=False  https://review.openstack.org/21660407:47
*** Haomeng|2 has joined #openstack-ironic07:47
dtantsurNisha, well, I personally don't want (and can't) spend time on the bash ramdisk, so yeah, I'd prefer us working with IPA first07:47
dtantsurNisha, the IPA patch for inspector is awaiting reviews: https://review.openstack.org/#/c/205587/07:48
*** Haomeng has quit IRC07:50
*** vvb1 has quit IRC07:51
Nishadtantsur, if we just enhance the ironic-discoverd-ramdisk will it not work for both?07:52
*** Marga_ has quit IRC07:52
Nishaand then in ironic-inspector and ironic/driver/modules/inspector.py we can simply consume it?07:53
Nishadtantsur, this was my thought for retrieving the data inband07:53
Nishadtantsur, for IPA ramdisk the IPA needs to be enhanced additionally.07:54
Nishadtantsur, am i missing some flow in between07:55
*** Marga_ has joined #openstack-ironic07:56
dtantsurNisha, IPA inspection code is unrelated to the old ramdisk. Actually IPA is a reimplementation in Python07:57
dtantsurNisha, so the flow will be the same, but we'll be writing Python code :)07:57
Nishadtantsur, how do u build inspection ramdisk for agent?07:58
Nishais it not thru DIB?07:58
dtantsurNisha, 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
Nishadtantsur, or if i understand you mean that the DIB element functionaility is IPA instead of being in DIB08:00
Nishahmmm i see08:00
*** ukalifon1 has joined #openstack-ironic08:02
*** links has quit IRC08:02
*** ukalifon has quit IRC08:04
*** ndipanov has joined #openstack-ironic08:06
*** pelix has joined #openstack-ironic08:12
*** romainh has joined #openstack-ironic08:13
*** lucasagomes has joined #openstack-ironic08:18
*** mgoddard has joined #openstack-ironic08:22
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Fix functional tests job  https://review.openstack.org/21713508:23
getvasanthNisha: disk-image-create -u -o ubuntu1 -a amd64 ubuntu08:25
*** alexpilotti has quit IRC08:28
*** dlpartain has joined #openstack-ironic08:33
*** achanda has quit IRC08:36
*** jistr has joined #openstack-ironic08:37
openstackgerritNisha Agarwal proposed openstack/ironic: Remove the hard dependency of swift from ilo drivers  https://review.openstack.org/19865608:43
Nishagetvasanth, what is -u for?08:44
getvasanthuncompressed08:44
getvasanthit wont compressed the image08:45
getvasanthNisha: do you have any specific steps to create a image?08:45
Nishagetvasanth, no same comand but i dont use -u08:46
getvasanthNisha: wanted to get the image boot08:46
Nishagetvasanth, "disk-image-create ubuntu -o image " will create partition image08:47
Nishagetvasanth, "disk-image-create ubuntu vm -o image " will create whole disk image08:47
getvasanthNisha: Ok, but this one fails with the error no or empty argument for root= error08:47
Nishagetvasanth, i have never hit this issue08:48
*** alexpilotti has joined #openstack-ironic08:49
getvasanthNisha: Ok, this is the actual error message "dracut: FATAL: No or empty root= argument"08:50
getvasanthNisha: then how do u create the ramdisk?08:50
Nisharamdisk or image?08:50
openstackgerritNisha Agarwal proposed openstack/ironic: Implements take_over() for ilo drivers  https://review.openstack.org/21706808:51
getvasanthNisha: there 2 steps right, 1. to create the image, 2. to create the ramdisk and the kernel08:51
Nishayes08:51
getvasanthNisha: how do you create the 2 one?08:51
getvasanthNisha: 1 question in your command, were u able to boot both the ways, partition & whole disk image?08:53
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Revisit required provision states  https://review.openstack.org/21924708:53
Nishafor creating ramdisk, "ramdisk-image-create ubuntu agent-ironic -o image"08:53
*** alexpilotti has quit IRC08:54
*** derekh has joined #openstack-ironic08:55
lucasagomesNisha, I think you have to use disk-image-creat for "agent-ironic" element08:55
lucasagomesnot ramdisk-image-create08:55
lucasagomesironic-agent*08:56
lucasagomeshttps://github.com/openstack/diskimage-builder/blob/master/elements/ironic-agent/cleanup.d/99-ramdisk-create08:56
getvasanthlucasagomes: hi08:56
lucasagomesit 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 system08:57
lucasagomesgetvasanth, hi08:57
Nishalucasagomes, :)08:57
getvasanthlucasagomes: will be back soon08:57
Nishai had always used bash ramdisk till now08:57
Nishafor which ramdisk-image-create works08:57
lucasagomesNisha, ok...08:58
lucasagomesI thought it would break at some part when building (/me haven't tested it)08:58
Nishai was under the impression the same shud work with agent-ironic also. i will try this once and see08:58
dtantsurspeaking of ramdisks, lucasagomes could you review my IPA patch for inspection please? https://review.openstack.org/#/c/205587/08:58
Nishalucasagomes, no it doesnt break08:58
dtantsurlucasagomes, and g'morning :)08:59
lucasagomesdtantsur, ack, morning08:59
lucasagomesNisha, ironic-agent*08:59
Nishalucasagomes, dtantsur please review these also https://review.openstack.org/198656 , https://review.openstack.org/21706808:59
Nishalucasagomes, :)09:00
lucasagomesack09:00
Nishalucasagomes, thanks09:00
*** alexpilotti has joined #openstack-ironic09:01
*** dlpartain has left #openstack-ironic09:06
*** vvb1 has joined #openstack-ironic09:10
openstackgerritNisha Agarwal proposed openstack/ironic-specs: Enhance ilo drivers to do inband inspection  https://review.openstack.org/20190409:13
*** romcheg has joined #openstack-ironic09:14
*** siva_ has joined #openstack-ironic09:14
sambettsMorning all o/09:14
*** siva_ is now known as Guest4340809:15
*** alexpilotti has quit IRC09:15
*** alexpilotti has joined #openstack-ironic09:15
*** alexpilotti has quit IRC09:15
dtantsursambetts, morning09:16
*** alexpilotti has joined #openstack-ironic09:16
*** alexpilotti has quit IRC09:16
sambettsdtantsur o/09:16
*** Marga_ has quit IRC09:17
*** Marga_ has joined #openstack-ironic09:17
*** Marga_ has quit IRC09:24
*** Marga_ has joined #openstack-ironic09:24
*** vvb2 has joined #openstack-ironic09:26
*** vvb2 has left #openstack-ironic09:26
openstackgerritMerged openstack/ironic: Prevent ilo drivers powering off active nodes during take over  https://review.openstack.org/21878509:28
*** vvb1 has quit IRC09:29
*** Marga_ has quit IRC09:29
openstackgerritMichael Davies proposed openstack/ironic: Switch default partition type to gpt  https://review.openstack.org/12662509:32
*** e0ne has joined #openstack-ironic09:32
*** alexpilotti has joined #openstack-ironic09:37
*** priteau has quit IRC09:37
*** alexpilotti has quit IRC09:41
*** marzif has joined #openstack-ironic09:42
*** Marga_ has joined #openstack-ironic09:42
aarefievlucasagomes: hi, are u around?09:46
lucasagomesaarefiev, hi there, yes09:46
aarefievlucasagomes: I'm trying  to fix ironicclient functional job09:47
aarefievcould you take a look  https://review.openstack.org/#/c/217135/  please09:47
lucasagomesaarefiev, sure09:48
* lucasagomes clicks09:48
aarefievlucasagomes: thanks09:49
*** alexpilotti has joined #openstack-ironic09:49
*** vvb1 has joined #openstack-ironic09:53
aarefievlucasagomes: infra patch https://review.openstack.org/#/c/217022/09:54
*** naohirot has quit IRC09:55
openstackgerritMichael Davies proposed openstack/ironic: Switch default partition type to gpt  https://review.openstack.org/12662509:58
*** priteau has joined #openstack-ironic10:02
*** Marga_ has quit IRC10:02
*** Marga_ has joined #openstack-ironic10:02
lucasagomesaarefiev, reviewed, overall it LGTM. I had some questions that I put inline10:02
*** e0ne has quit IRC10:04
aarefievlucasagomes: 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 works10:07
*** e0ne has joined #openstack-ironic10:08
*** marzif has quit IRC10:08
*** karimb has joined #openstack-ironic10:09
*** kbyrne has quit IRC10:12
lucasagomesaarefiev, 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 it10:13
lucasagomesit's good to have a bug so it's shows in the release notes10:13
aarefievlucasagomes: sure, just I wasn't sure where bug should be10:14
aarefievlucasagomes: thanks for help10:14
lucasagomesaarefiev, no problem at all! Thank you for fixing it!10:14
*** kbyrne has joined #openstack-ironic10:15
*** MattMan has quit IRC10:23
*** MattMan has joined #openstack-ironic10:24
*** alexpilotti has quit IRC10:24
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints  https://review.openstack.org/21827110:25
*** dtantsur is now known as dtantsur|brb10:30
*** links has joined #openstack-ironic10:31
Nishalucasagomes, hi10:33
lucasagomesNisha, hi there10:33
Nishalucasagomes, regarding your comments on https://review.openstack.org/#/c/220003/2/ironic/drivers/modules/agent.py10:33
Nishalucasagomes, i dont see any difference between "if task.driver.boot is not None" and "if task.driver.boot:"10:34
* lucasagomes relooks10:35
lucasagomesNisha, oh yeah I think that we don't need to test the placeholder there10:36
lucasagomesI think I misread that code10:36
Nishalucasagomes, we need to have the check "if task.driver.boot"10:36
lucasagomesNisha, indeed10:37
Nishaelse any driver which doesnt has that implemented yet will fail10:37
lucasagomeschanged my vote there10:37
lucasagomesthanks for pointing it out10:37
Nishawow thanks10:37
lucasagomesthank you for the fix!10:38
Nisha:) thanks for reviewing.10:38
lucasagomesnp10:39
* Nisha hope to see my other patches also reviewed10:39
Nishalucasagomes, do u know how do we run tests for inspector10:40
Nishadtantsur|brb, ^^^10:40
lucasagomesNisha, which types of tests? (yeah dtantsur|brb will def know the internals better)10:40
Nishalucasagomes, i got some results just now for the tests10:41
Nishawill look into them :)10:41
*** vvb1 has quit IRC10:41
lucasagomesack10:41
*** ukalifon1 has quit IRC10:42
*** alexpilotti has joined #openstack-ironic10:42
*** thrash|g0ne is now known as thrash10:44
*** romcheg has quit IRC10:45
*** dtantsur|brb is now known as dtantsur10:45
*** Nisha has quit IRC10:46
*** alexpilotti has quit IRC10:47
*** marzif has joined #openstack-ironic10:47
*** jcoufal has joined #openstack-ironic10:51
PradeepVHi Ironic, how can i generate whole disk image in ISO format10:51
*** alexpilotti has joined #openstack-ironic10:54
openstackgerritAnton Arefiev proposed openstack/python-ironicclient: Fix functional tests job  https://review.openstack.org/21713510:56
*** romcheg has joined #openstack-ironic10:57
*** Guest43408 has quit IRC10:57
lucasagomesPradeepV, in DIB I think you can use the "vm" and "iso" elements11:01
lucasagomesI've never tested, maybe the folks in #tripleo can tell you better about it11:01
*** lazy_prince has joined #openstack-ironic11:02
PradeepVok lucasagomes11:03
lucasagomesPradeepV, 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 ramdisk11:03
lucasagomesit won't use the bootloader of the image AFAICT11:04
PradeepVi tried creating iso in DIB but it failed11:06
lucasagomesPradeepV, with full disk image? Or just partition image?11:07
PradeepVfull disk image11:07
lucasagomesright yeah, I don't know how those would play out11:08
openstackgerritRamakrishnan G proposed openstack/ironic: Update iLO driver documentation for configuring Swift  https://review.openstack.org/18273611:08
lucasagomesPradeepV, why you need that? Just out of curisity11:08
rameshg87PradeepV: lucasagomes: vm + iso doesn't yield a bootable iso of whole disk image11:10
rameshg87PradeepV: lucasagomes: iso element can be used only with some elements for bundling a kernel and ramdisk to form a bootable element11:10
lucasagomesrameshg87, yeah that's my thought as well#11:11
*** Marga_ has quit IRC11:12
* rameshg87 goes home11:13
*** romcheg1 has joined #openstack-ironic11:13
*** rameshg87 has quit IRC11:13
*** romcheg has quit IRC11:14
*** shoutm has joined #openstack-ironic11:17
*** shoutm_ has quit IRC11:18
*** ishant has quit IRC11:19
*** ramineni has quit IRC11:26
*** dlpartain has joined #openstack-ironic11:26
*** alexpilotti has quit IRC11:27
getvasanthlucasagomes: I am back :)11:32
*** dlpartain has left #openstack-ironic11:35
getvasanthlucasagomes: On te image creation "disk-image-create ubuntu vm -o image" and "ramdisk-image-create -o ramdisk ubuntu"11:35
*** Marga_ has joined #openstack-ironic11:35
getvasanthlucasagomes: is that correct or am i missing any parameters11:36
lucasagomesgetvasanth, I think it's missing the deploy element for the ramdisk one11:36
lucasagomesgetvasanth, which driver you are going to use?11:36
lucasagomesgetvasanth, 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
getvasanthlucasagomes: pxe_ipmitool11:37
lucasagomesgetvasanth, right that works for pxe_ipmitool ^11:37
lucasagomesgetvasanth, btw http://docs.openstack.org/developer/ironic/deploy/install-guide.html#building-or-downloading-a-deploy-ramdisk-image11:38
getvasanthlucasagomes: "disk-image-create -o ramdisk ironic-agent ubuntu " this will build both the image and the ramdisk right?11:38
lucasagomesgetvasanth, no it will only build the ramdisk11:39
lucasagomesgetvasanth, problem is that the ironic-agent element (which is IPA) needs a real init system11:39
lucasagomesthe ramdisk-image-create command for DIB replaces the init system with a custom script11:39
lucasagomeswhich is not idea11:39
lucasagomesideal*11:39
getvasanthlucasagomes: oh! i am confused now11:40
lucasagomesgetvasanth, 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-create11:40
lucasagomesyeah I know11:40
lucasagomesit's confusing as hell11:40
openstackgerritMerged openstack/ironic: Fix docker package name in Ubuntu 14.04 in Install Guide  https://review.openstack.org/21869411:40
getvasanthlucasagomes: 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
lucasagomesgetvasanth, yes11:43
lucasagomesgetvasanth, tho note there's a different in the first command that you showed me before11:43
lucasagomesyou were using the "vm" element to create a full disk image11:43
lucasagomesthe new 1) is creating a partition image (no "vm" element)11:43
lucasagomesdifference*11:44
* lucasagomes 1 sec will grab a sandwich with the rest of the folks in the office11:44
getvasanthlucasagomes: yes, understood that, it should boot.11:45
getvasanthlucasagomes: thanks for confirming, then next step include the kernel & ramdisk and and upload a 3 part image and boot it11:45
lucasagomesgetvasanth, ack... yeah so if you want a partition image make sure you have the kernel and ramdisk for that image too11:45
lucasagomesbrb quickly11:46
*** lucasagomes is now known as lucas-hungry11:46
getvasanthlucasagomes: yes!!, thanks a lot again, will be back after building it11:46
*** amotoki has quit IRC11:47
*** alexpilotti has joined #openstack-ironic11:53
openstackgerritJuliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers  https://review.openstack.org/22009711:54
*** alexpilotti_ has joined #openstack-ironic11:58
*** Marga_ has quit IRC12:01
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints  https://review.openstack.org/21827112:02
openstackgerritJuliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers  https://review.openstack.org/22009712:03
sinvalmorning Ironicers12:06
sinvaldoes anyone knows if the current version of Ironic should work with OpenStack Kilo (Glance, Nova, Keystone and etc?)12:06
getvasanthsinval: I am trying to get it working ;)12:08
sinvalgetvasanth: Oh, great, let me know when you have some news about it, thanks12:13
*** romcheg1 has quit IRC12:14
*** marzif has quit IRC12:14
*** marzif has joined #openstack-ironic12:15
getvasanthsinval: sure12:15
TheJuliagood morning12:17
*** absubram has quit IRC12:24
sinvalTheJulia: morning12:25
*** VikasC has quit IRC12:28
openstackgerritJulia Kreger proposed openstack/bifrost: Improve logging for VMs to identify IPA issues  https://review.openstack.org/22011212:30
openstackgerritJulia Kreger proposed openstack/bifrost: Fix HACKING.rst and canary test for merges  https://review.openstack.org/22011312:30
*** lucas-hungry is now known as lucasagomes12:30
*** baoli has joined #openstack-ironic12:33
*** trown|outttypeww is now known as trown12:36
*** baoli_ has joined #openstack-ironic12:37
*** baoli has quit IRC12:38
*** dims has joined #openstack-ironic12:41
trownTheJulia: 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-ironic12:47
*** jjohnson2 has joined #openstack-ironic12:48
*** shoutm has quit IRC12:48
*** jcoufal has quit IRC12:49
TheJuliatrown: I must have been on another planet earlier this week.  Actually, I know I was12:53
trown:), thanks for looking12:53
trownI pinged in #tripleo to try to get folks to look at the actual fix patch12:54
*** PradeepV has quit IRC12:54
openstackgerritJulia Kreger proposed openstack/bifrost: Improve logging for VMs to identify IPA issues  https://review.openstack.org/22011212:55
TheJuliathanks, I'd like to see it land at some point12:56
TheJuliait will make me much happier and a CI job very happy12:56
openstackgerritJulia Kreger proposed openstack/bifrost: Fix HACKING.rst and canary test for merges  https://review.openstack.org/22011312:56
*** lazy_prince has quit IRC12:57
trownTheJulia: ya...I broke that at a really bad time...CI has been pretty awful this week12:58
TheJuliac'est la vie13:01
*** jcoufal has joined #openstack-ironic13:02
*** rloo has joined #openstack-ironic13:04
lucasagomesjroll, around?13:08
*** lazy_prince has joined #openstack-ironic13:13
rloohi lucasagomes, just looking at your comment: https://review.openstack.org/#/c/220003/2/ironic/drivers/modules/agent.py13:24
rloolucasagomes: I think we should change it to what you suggested13:24
rloohi everyone and lucasagomes :)13:24
lucasagomesrloo, is not None?13:25
rloolucasagomes: yes.13:25
rloolucasagomes: otherwise, like you said, it isn't backwards compat13:25
lucasagomesthat's the placeholder, but just checking if the boot is set to something seems fair enuff13:26
openstackgerritJuliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers  https://review.openstack.org/22009713:26
lucasagomesrloo, but peole running that code will have the base.py seeting "boot = None"13:26
lucasagomeseven if they driver doesn't specify an boot interface13:26
lucasagomesit will be None13:26
lucasagomesso the "if task.driver.boot:" checking if it's set will work13:27
rloolucasagomes: sorry. i didn't look at the code change, i was reading your comments only. yeah, it is ok as is.13:27
lucasagomesrloo, yeah I got confused too and changed my vote later13:28
dtantsurmorning TheJulia, trown, rloo :)13:28
rloolucasagomes: yeah, you confused me. it's good now :)13:29
*** amotoki has joined #openstack-ironic13:29
*** vvb1 has joined #openstack-ironic13:30
openstackgerritJuliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers  https://review.openstack.org/22009713:31
lucasagomesrloo, :-)13:31
rloolucasagomes: I put that patch on the list of stuff to land before cutting 4.1.0 this week13:32
lucasagomesrloo, btw are you working on https://bugs.launchpad.net/ironic/+bug/1461140 ?13:32
openstackLaunchpad bug 1461140 in Ironic "conflict (HTTP 409) incorrect for some cases" [Medium,Confirmed] - Assigned to Ruby Loo (rloo)13:32
rloolucasagomes: no. i thought you and dmitry were looking into it. we discussed at some meeting awhile ago?13:32
dtantsurtrown, 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
trowndtantsur: ya, I have been trying to get to that one for a bit...sorry13:33
dtantsurnp at all13:33
lucasagomesrloo, right... yeah perhaps I forgot :-(13:34
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: [WIP] Smart root disk selection including support for root device hints  https://review.openstack.org/21827113:34
lucasagomeswill take a look13:34
rloolucasagomes: 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
lucasagomesrloo, did we decide anything whether use diff exceptions or retry-after?13:34
trowndtantsur: 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
dtantsurhehe, thanks!13:34
lucasagomesrloo, yeah no idea... I will give it some thought13:34
dtantsuryeah, it's way too huge, I just didn't figure out how to meaningfully split it13:34
rloolucasagomes: let me try to find the meeting where we discussed it13:35
lucasagomesrloo, no bothers13:35
*** afaranha has joined #openstack-ironic13:35
lucasagomesrloo, we released an openstack installer not long ago13:35
lucasagomesand since I've been pretty busy with it so I'm sure I'm the one who forgot stuff13:35
rloolucasagomes: we did? oh, RH.13:35
lucasagomesrloo, I can find it, no bothers13:35
lucasagomesyeah13:35
lucasagomesRH13:35
*** afaranha has quit IRC13:36
openstackgerritDmitry Tantsur proposed openstack/ironic-inspector: Add introspection rules support  https://review.openstack.org/20837513:36
*** afaranha has joined #openstack-ironic13:37
*** afaranha has joined #openstack-ironic13:37
rloolucasagomes: http://eavesdrop.openstack.org/meetings/ironic/2015/ironic.2015-07-13-17.00.log.html13:39
lucasagomesrloo, thanks!13:40
*** links has quit IRC13:40
*** alexpilotti has quit IRC13:53
*** alexpilotti_ is now known as alexpilotti13:53
*** jroll has quit IRC13:54
*** jroll has joined #openstack-ironic13:54
jrolllucasagomes: rloo dtantsur trown morning :)14:01
trownmorning jroll14:01
dtantsurjroll, morning14:01
*** bnemec has quit IRC14:02
trownjroll: 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-ironic14:02
rloomorning jroll14:02
jlvillalGood morning Ironic14:03
jrolltrown: yay14:03
trownless exciting for you than me :P, but it does mean that we are one step closer to destroying the bash ramdisk14:03
lucasagomesjroll, morning!14:03
lucasagomesjroll, btw re node reservation. I remember you said you had another idea in mind? Which wouldn't use the /reservation endpoint14:04
lucasagomesjroll, you wanna talk about it?14:04
lucasagomesjust as a context, we have been hitting some races in the nova-scheduler and Ironic14:05
lucasagomesso I would like to think about a solution for that, it could be in Nova or in Ironic14:05
trownya, nova-scheduler is really really bad at scheduling N instances to N nodes14:05
trownN available nodes that is14:06
jrolllucasagomes: I had ideas for the multi-compute things, I'm not sure if it will help scheduler races at all14:06
lucasagomesfwiw 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
jrollI don't think it will, the scheduler in general is just racy as hell14:06
lucasagomesright14:06
lucasagomesjroll, the reservation actually would solve it AFAIUI14:06
jrollthe only thing that can actually solve problems is a distributed lock manager14:07
jroll... which I just now noticed, the reservation thing would be using ironic as the DLM, lol14:07
lucasagomesyea14:08
lucasagomesand the spec by nikola wouldn't solve it for multiple n-cpu14:08
lucasagomesonly for 1 n-cpu (the spec says it)14:08
jrolllucasagomes: we could just as easily require zookeeper for the ironic scheduler bits and accomplish the same thing :P14:08
lucasagomesjroll, yeah, food for thought14:09
lucasagomesjroll, one thing that would help as well is fail fast when the node is already taken14:09
jrolllucasagomes: I don't see where that spec says one n-cpu?14:09
lucasagomesusually in nova it keeps retrying the request to add the instance_uuid in the node14:09
lucasagomeswhich is already populated14:09
jrollsounds more like one scheduler14:09
lucasagomesso that's why I'm looking at the retry-after thing as well, so we don't retry on certain exceptions14:10
lucasagomesand it would make it to fail faster and the retry filter would then get a new node14:10
*** harshs has joined #openstack-ironic14:10
jrollyeah, so to be clear you still get a reschedule yeah? it's not 'broken' just suboptimal?14:10
lucasagomessingle n-sche*14:10
lucasagomessorry14: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
jrollyeah14:11
jrollwhich is really the only supported method14:11
lucasagomesyeah14:11
jrollnow that said, I wish nova would just make this use a DLM14:11
jrolland then you can run all the schedulers you want14:11
lucasagomesjroll, yeah eventually it get reschedule'd14:11
jrolleven if it uses mysql as the DLM, which is horrible and bad and something we do14:11
dtantsur... or fails to get rescheduled14:12
jrollthat's still an improvement14:12
lucasagomesyeah it will try 3 times (I think thta's the default for the retry filter)14:12
lucasagomesand yeah it's possible to fail 3x14:12
jrollso the way this multi-compute thing works is less racy, to be clear (I think)14:12
jrolldepending how hard things are cached14:12
jrollbut we could validate after choosing14:13
jrollI need to write that up :(14:13
* jroll is a bad person14:13
lucasagomesheh happens14:14
*** saripurigopi has quit IRC14:14
*** shoutm_ has quit IRC14:18
*** alexpilo_ has joined #openstack-ironic14:20
openstackgerritSHIGEMATSU Mitsuhiro proposed openstack/ironic: Fix minor grammar/comment description errors  https://review.openstack.org/21955114:24
openstackgerritSHIGEMATSU Mitsuhiro proposed openstack/ironic-python-agent: Use a tiny connection wrapper in the heartbeat thread  https://review.openstack.org/21990514:26
getvasanthlucasagomes: 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-ironic14:33
*** devlaps has quit IRC14:33
*** vvb1 has quit IRC14:34
*** mtanino has joined #openstack-ironic14:35
*** mgoddard has quit IRC14:35
lucasagomesgetvasanth, yeah. You can use local boot so you don't need to PXE boot again for the second boot14:36
lucasagomesor if you already deployed you probably can just set the boot device to disk14:37
lucasagomes(make sure the full disk image has a bootloader)14:37
*** thrash is now known as thrash|biab14:38
*** getvasanth has quit IRC14:39
*** getvasanth has joined #openstack-ironic14:40
getvasanthlucasagomes: 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
lucasagomesgetvasanth, heh network problems?14:40
*** garthb has joined #openstack-ironic14:40
lucasagomes<lucasagomes> getvasanth, yeah. You can use local boot so you don't need to PXE boot again for the second boot14:40
lucasagomes<lucasagomes> or if you already deployed you probably can just set the boot device to disk14:40
lucasagomes<lucasagomes> (make sure the full disk image has a bootloader)14:40
lucasagomesjroll, dtantsur python-ironicclient 0.8.0 is now release, I wonder if we could pin it in the g-r14:41
lucasagomesso that we can use things like native rety in the nova ironic driver etc14:41
getvasanthlucasagomes: yes, network trouble :(14:41
lucasagomesgetvasanth, happens :-)14:41
dtantsurlucasagomes ++14:42
jrolllucasagomes: I'm cool with that14:42
lucasagomesjroll, I want to use custom fields as well to minimize the logs in nova14:42
lucasagomesok, will put a patch to pin it14:42
lucasagomestoday is the last day right? To ping things in the g-r14:43
*** Marga_ has joined #openstack-ironic14:43
jrollyes14:43
jrollafaik14:43
lucasagomesok14:43
*** mgoddard1 has quit IRC14:43
jrolllucasagomes: if you bug people for reviews maybe add in https://review.openstack.org/#/c/219011/ :P14:44
lucasagomesjroll, will do! hah14:44
*** mgoddard has joined #openstack-ironic14:44
*** romainh has left #openstack-ironic14:44
getvasanthlucasagomes: thanks :)14:46
lucasagomesnp14:47
*** zz_natorious is now known as natorious14:47
lucasagomeshttps://review.openstack.org/22017414:50
lucasagomesfor those interested14:50
*** vvb1 has joined #openstack-ironic14:51
yuriyzmorning/evening Ironic, glad to see ironic-lib on pypi, but Programming Language :: Python :: 2.6 looks wrong14:52
*** mtanino has quit IRC14:53
lucasagomesyuriyz, morning14:58
lucasagomesyuriyz, heh indeed14:58
dtantsurgood catch! somehow I missed it when cleaning up setup.cfg...14:58
*** jistr is now known as jistr|call15:02
*** Marga_ has quit IRC15:05
openstackgerritMerged openstack/ironic-python-agent: Three attempts message is incorrect  https://review.openstack.org/21991315:07
*** thrash|biab is now known as thrash15:09
*** alexpilotti_ has joined #openstack-ironic15:16
*** alexpilotti has quit IRC15:17
*** alexpilotti_ is now known as alexpilotti15:17
*** harshs has quit IRC15:17
*** alexpilo_ has quit IRC15:20
NobodyCamgood late morning Ironicers15:22
NobodyCam:-p15:22
jrollmorning NobodyCam :)15:24
*** dims has quit IRC15:26
lucasagomesNobodyCam, hi there15:26
NobodyCamgood morning lucasagomes & jroll15:31
NobodyCamI'm moving slow this mornign15:31
NobodyCam:-p15:31
NobodyCamruff night with the kids :)15:31
dtantsurNobodyCam, morning15:31
NobodyCammorning dtantsur :)15:31
*** dims has joined #openstack-ironic15:36
*** mtanino has joined #openstack-ironic15:36
*** amotoki has quit IRC15:40
rloomorning NobodyCam. I hope the kids are ok now.15:41
rloohi lucasagomes, was wondering what the status is for deprecating the bash ramdisk.15:43
rloolucasagomes: https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit?pli=1#gid=133803727215:43
rloolucasagomes: still need code/patch in ironic?15:43
*** jistr|call is now known as jistr15:44
*** alexpilo_ has joined #openstack-ironic15:46
lucasagomesrloo, I think in Ironic is grand15:46
lucasagomesI just rebased two patches in the tripleo-incubator15:46
lucasagomesthere's a patch in DIB as well15: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/19416115:46
lucasagomes<openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Use ironic-python-agent as the default deploy ramdisk  https://review.openstack.org/210461/19416115:46
lucasagomes<openstackgerrit> Lucas Alvares Gomes proposed openstack/tripleo-incubator: Use ironic-python-agent as the default deploy ramdisk  https://review.openstack.org/21046115:46
lucasagomesdamn that's messy15:47
lucasagomeshttps://review.openstack.org/194161 and https://review.openstack.org/21046115:47
rloolucasagomes: 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
rloolucasagomes: or deprecating it starting in L? I'm losing track of time.15:48
lucasagomesrloo, well, we need to discuss it. Cause it would be important to have those patches in the other projects15:48
lucasagomesrloo, we can't just remove the endpoints if the tripleo-ci is not updated to use IPA for e.g15:48
lucasagomesor we are going to break it15:48
lucasagomesremove the endpoints in the Ironic API I mean15:48
lucasagomesrloo, yeah probably in L :-/15:48
rloolucasagomes: 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
rloolucasagomes: well, deprecating and removing are two different things/times.15:49
lucasagomesrloo, well I'm kinda confident we can get those patches merged in this cycle15:49
lucasagomesI will ping people15:49
rloolucasagomes: ok thx. can i update the google doc then, that it doesn't require any patches to ironic?15:49
lucasagomesrloo, yeah we are probably removing only in N15:49
lucasagomesrloo, not that I'm aware off15:50
lucasagomesof*15:50
* lucasagomes thinks15:50
rloolucasagomes: do we 'deprecate' log msgs in ironic code, where the ramdisk stuff is handled?15:50
NobodyCammornign rloo15:50
NobodyCamrloo: there doing okay... just really restless lastnight15:50
lucasagomesrloo, yeah https://review.openstack.org/#/c/209188/15:50
lucasagomesdocs seems fine too (I will double check)15:51
lucasagomesall the docs now are building IPA ramdisks instead of bash ones15:51
rloolucasagomes: 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-ironic15:51
lucasagomeshttps://review.openstack.org/#/c/209189/15:51
rlooNobodyCam: good to hear. I don't think it was a full moon last night either.15:51
lucasagomesrloo, ack15:51
lucasagomesrloo, yeah it seems just the non-ironic patches needs to land now15:52
rloothx lucasagomes. will update google doc.15:52
lucasagomesrloo, ok thank you15:52
dtantsurand I need to make one of J* review inspector IPA patch :)15:52
* dtantsur also tries to get rid of bash ramdisk15:52
* lucasagomes was just testing nova ironic driver with ironiclcient 0.8.0 to minimize the logs15:54
lucasagomesso much better, damn15:54
lucasagomesjust to not have that configdrive string there makes a lot of difference15:54
dtantsurI can imagine15:54
*** mgoddard has quit IRC16:01
*** mgoddard has joined #openstack-ironic16:02
JoshNangdtantsur: i'll take a look. https://review.openstack.org/#/c/205587/ right?16:02
dtantsurJoshNang, correct, thanks a lot!16:03
*** marzif has quit IRC16:04
*** getvasanth has quit IRC16:06
*** jistr has quit IRC16:06
*** jaypipes has joined #openstack-ironic16:07
*** ijw_ has joined #openstack-ironic16:08
jrolllucasagomes: rloo I think we have everything merged for 4.1 now yes?16:12
rloojroll: not yet. sec, let me check.16:12
* lucasagomes checking16:12
*** VikasC has joined #openstack-ironic16:12
jrollI had three on my list: enroll docs, release notes, and lucas' revert16:12
lucasagomeshttps://review.openstack.org/#/c/220003/ < not merged yet16:13
jrolloh, new ilo16:13
jrollyeah16:13
rloojroll: the enroll docs hasn't merged yet either16:13
jrolloh heh16:13
jrollslow gate16:13
jrollso, here's my plan:16:13
lucasagomeswhere's the enroll docs one?16:13
jrollintel/rackspace are having a bug bash today, includes some ironic work (y'all are welcome to join)16:14
lucasagomesoh found it16:14
jrolland friday is bad for releasing16:14
jrollso I'm thinking 4.1 tuesday16:14
lucasagomesjroll, I'm in the office today and will go home soon :-(16:14
jrollsoft freeze beginning thurs 9/1716:14
rloojroll: i'm fine with tuesday.16:14
lucasagomesbefore it closes16:14
jroll4.2 9/2416:14
lucasagomesbut yeah tuesday seems ok16:14
jrolllucasagomes: yeah no worries :)16:14
rloojroll: IF the gate is fast and those two land later today, you're welcome to cut a release if you want.16:15
jrollso 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
jrollrloo: nod, going to focus on bugs today though16:15
NobodyCamnice https://review.openstack.org/#/c/218879 (disk image builder ipa patch) landed this mornign  :_16:15
NobodyCams/_/)/16:15
rloojroll: 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
jrollrloo: yeah, no worries, I'm also going to try to review some feature patches today16:16
*** puranamr has joined #openstack-ironic16:16
lucasagomesrloo, I agree with ur comments on the enroll docs16:17
*** karimb has quit IRC16:17
lucasagomesperhaps we can update it ?16:17
lucasagomesor NobodyCam can take a look so we land it soonish?16:18
*** derekh has quit IRC16:19
rloolucasagomes: isn't the enroll doc approved?16:19
lucasagomesrloo, nop, https://review.openstack.org/#/c/209251/16:19
jrolllucasagomes: see her last comment16:19
lucasagomesohh16:20
lucasagomesdamn16:20
* lucasagomes was reviewing the wrong thing16:20
rloolucasagomes: crap. sorry about that. maybe i forgot to update the link in the etherpad.16:20
lucasagomesok I will abandon that one then, ack?16:20
*** ndipanov has quit IRC16:21
jrollyeah +116:21
rloolucasagomes: yeah, thx. i forgot that we could abandon them.16:22
lucasagomesdone :-) no bothers16:23
dtantsursee you tomorrow folks, happy bug bash for some of you :)16:29
rloobye dtantsur16:29
NobodyCamhave a good night dtantsur16:29
*** dtantsur is now known as dtantsur|afk16:31
lucasagomesdtantsur|afk, see ya16:31
jroll\o dtantsur|afk16:32
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - RPC  https://review.openstack.org/20624316:34
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - API  https://review.openstack.org/20624416:34
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - net  https://review.openstack.org/20624516:34
openstackgerritOm Kumar proposed openstack/ironic: Update the deploy drivers with network flipping logic  https://review.openstack.org/21326216:34
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - DB  https://review.openstack.org/20623216:34
openstackgerritOm Kumar proposed openstack/ironic: Add portgroups to support LAG interfaces - objs  https://review.openstack.org/20623816:34
openstackgerritOm Kumar proposed openstack/ironic: Add network provider interface and implementations  https://review.openstack.org/13968716:34
*** e0ne has quit IRC16:34
lazy_princeSukhdev: changes pushed ^^16:34
jrollwheeee16:36
openstackgerritSam Betts proposed openstack/ironic: Add Cisco IMC PXE Driver  https://review.openstack.org/21925316:36
lucasagomesfolks I'm heading home16:37
lucasagomeshave a good night everyone!16:37
lucasagomessee y'all later16:37
jrollnn \o16:37
*** lucasagomes has quit IRC16:39
*** ifarkas has quit IRC16:40
NobodyCamhave a good niht lucas16:41
*** alexpilo_ has quit IRC16:41
NobodyCamnight even16:41
* NobodyCam cann't type today16:41
*** meghal has joined #openstack-ironic16:42
*** meghal1 has joined #openstack-ironic16:45
*** meghal has quit IRC16:48
*** liliars has joined #openstack-ironic16:54
*** meghal1 has quit IRC16:58
*** priteau has quit IRC16:58
*** harshs has joined #openstack-ironic16:59
*** jrist-afk is now known as JRIST17:00
*** JRIST is now known as jrist17:00
*** puranamr has quit IRC17:02
*** lazy_prince has quit IRC17:04
sambettsI'm heading off too, cy'all tomorrow17:04
*** deray has joined #openstack-ironic17:05
*** alexpilo_ has joined #openstack-ironic17:07
*** priteau has joined #openstack-ironic17:10
*** puranamr has joined #openstack-ironic17:13
*** achanda has joined #openstack-ironic17:14
*** meghal has joined #openstack-ironic17:17
jrollnode.properties[local_gb] can be a float, right?17:18
jrollriiiiiight?17:18
jrollmaybe that's a nova question17:18
*** puranamr has quit IRC17:19
jrollint it is17:20
*** achanda has quit IRC17:21
*** Nisha has joined #openstack-ironic17:21
*** achanda has joined #openstack-ironic17:22
openstackgerritJohn L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments  https://review.openstack.org/21939017:27
openstackgerritJohn L. Villalovos proposed openstack/ironic: Enforce clarity by using keyword-only arguments  https://review.openstack.org/21939017:28
*** penick has joined #openstack-ironic17:32
*** Nisha_away has joined #openstack-ironic17:33
*** Nisha has quit IRC17:33
*** meghal has quit IRC17:33
*** meghal has joined #openstack-ironic17:34
jrolldoes anyone mind me closing https://bugs.launchpad.net/ironic/+bug/1390432 as a dupe of https://bugs.launchpad.net/ironic/+bug/146123917:35
openstackLaunchpad 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
jrollrloo: jlvillal ^17:35
openstackLaunchpad bug 1461239 in Ironic "replace objects with oslo.versionedobjects" [Low,In progress] - Assigned to Tan Lin (tan-lin-good)17:35
jrollin theory could be fixed earlier but meh17:35
jlvillaljroll: 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
jrolljlvillal: this bug is already fixed in o.vo17:37
*** meghal has quit IRC17:37
*** meghal has joined #openstack-ironic17:38
jrolljlvillal: 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 not17:38
jlvillaljroll: Ah. Then I think you are right that it is a dupe.17:38
jrollk, thanks17:38
jlvillaljroll: 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
jlvillalAnd then me cluelessly saying, why is there this temporary variable here. Doesn't seem needed :)17:39
jrolljlvillal: of course, I hate that bug so much :)17:39
jrollwhen you forget it while coding it's really hard to debug17:39
jlvillalYeah :(17:40
*** absubram has quit IRC17:43
*** vishwanathj has quit IRC17:43
*** e0ne has joined #openstack-ironic17:46
*** aarefiev22 has joined #openstack-ironic17:46
*** ijw_ has quit IRC17:48
*** bradjones has joined #openstack-ironic17:53
*** bradjones has quit IRC17:53
*** bradjones has joined #openstack-ironic17:53
*** Nisha has joined #openstack-ironic17:54
*** Nisha_away has quit IRC17:54
rloojroll: 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 IRC17:55
rloojroll: 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
openstackgerritRamakrishnan G proposed openstack/ironic: Add APIs for RAID configuration  https://review.openstack.org/19600717:56
*** VikasC has joined #openstack-ironic17:57
*** alexpilo_ has quit IRC18:00
*** deray has quit IRC18:04
JayFTo 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 IRC18:13
*** pelix has quit IRC18:17
jrollJayF: they do18:17
jrollrloo: idk, maybe you're right hm18:18
rloojroll, JayF: if you mean the GLOBAL requirements, they do. If you mean in our 'requirements.txt', they don't.18:18
JayFrloo: I did mean in global requirements :( damn18:19
JayFlekha: ^18:19
rlooJayF: better get it in quick then!18:19
*** garthb has quit IRC18:19
JayFjlvillal: should https://review.openstack.org/#/c/219390/ have a bug associated?18:22
jlvillalJayF: I'm not sure. I can create one if desired.18:23
JayFjlvillal: I was just asking b/c it was on the bug bash etherpad as a review18:23
jlvillalJayF: It is more of an enhancement than bug fix.18:24
*** alexpilo_ has joined #openstack-ironic18:26
jlvillalJayF: And for sure not a high priority18:27
*** marzif has joined #openstack-ironic18:28
JayFjlvillal: no problem, still going to review it :)18:29
JayFjlvillal: just wanted to make sure18:30
jlvillalJayF: Thanks18:30
*** harlowja has quit IRC18:31
lekhajlvillal: 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 IRC18:32
*** meghal has joined #openstack-ironic18:32
*** harlowja has joined #openstack-ironic18:32
jlvillallekha: That would be great. How does that compare to how the python-novaclient does it?18:32
jlvillalSimilar? Very different?18:32
lekhajlvillal: 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
jlvillallekha: Okay. I like the idea of the mimic stuff. Sound great to me.18:34
lekhajlvillal: awesome :)18:34
*** Nisha has joined #openstack-ironic18:34
jlvillallekha: Is mimic already supported in the Jenkins test jobs?18:34
JayFjlvillal: I'm about to push a PR to add it to global requirements18:35
*** Nisha has quit IRC18:35
JayFjlvillal: as far as running it is concerned, it's all hidden behind "tox" just like you'd expect18:35
jlvillalJayF: Ah great. I was looking and didn't see it.18:35
*** Nisha has joined #openstack-ironic18:35
JayFjlvillal: so no devstack or gate changes needed18:35
lekhajvillal: what JayF said18:35
jlvillallekha: JayF: Thanks18:36
JayFhttps://review.openstack.org/#/c/220268/ <-- adding mimic to g-r18:36
*** marzif has quit IRC18:38
*** Nisha has quit IRC18:40
*** ijw_ has joined #openstack-ironic18:50
*** ijw_ has quit IRC18:55
openstackgerritJuliana Motira proposed stackforge/pyghmi: Add custom OEM PSU Based Power Capping config for Lenovo ThinkServers  https://review.openstack.org/22009718:59
*** VikasC has quit IRC19:00
jrollJayF: left some commentary -- I'm set up to do constraints generation if you want me to grab that at some point19:02
JayFjroll: please do fix it19:03
jrollJayF: booo :)19:04
JayFjroll: lekha said 1.6.0 is the right version19:04
jrollJayF: lekha thanks19:04
*** meghal has quit IRC19:08
*** alexpilo_ has quit IRC19:08
*** meghal has joined #openstack-ironic19:10
*** MattMan1 has joined #openstack-ironic19:17
aarefiev22jroll: hi, are u around?19:17
*** achanda has quit IRC19:18
*** priteau has quit IRC19:18
jrollaarefiev22: hi19:19
aarefiev22jroll: could you take a look at https://review.openstack.org/#/c/217135/  please, it blocks further work on functional tests19:19
jrollaarefiev22: sure, I'll add it to my list :)19:20
aarefiev22jroll: thanks19:20
JayFI'll take a look too, although I'm not core19:20
lekhajroll: I have add/delete nodes for ironic in https://github.com/rackerlabs/mimic/pull/38719:20
lekhajroll: will release the next version of mimic with it, possibly today19:20
jrolllekha: /me adds to list19:21
*** meghal has quit IRC19:23
jrollaarefiev22: two questions19:25
jroll1) we don't need __init__.py, right?19:25
jrollsince there's no python code in hooks/19:25
jroll2) why not something like tox -e functional?19:26
*** alexpilotti has quit IRC19:26
*** alexpilotti has joined #openstack-ironic19:26
jrollor does this just fix tox -e functional?19:26
* jroll not sure he gets it19:26
*** alexpilotti has quit IRC19:28
aarefiev22jroll: yes about __init__ you are right , fail(19:28
*** alexpilotti has joined #openstack-ironic19:28
jrollit happens :)19:28
aarefiev22jroll: Many Eyes Make All Bugs Shallow)19:28
*** Marga_ has joined #openstack-ironic19:29
*** dguerri is now known as dguerri`19:29
jrollaarefiev22: so if you remove this file, I'm happy with the change19:30
jrollaarefiev22: though I'm not sure I fully understand why that fixes it19:30
*** puranamr has joined #openstack-ironic19:31
*** alexpilotti has quit IRC19:33
rloojroll: is this a 'valid' patch, or has it been superceded by other patches?19:34
*** ijw_ has joined #openstack-ironic19:34
rloojroll: I got that link via links in our priority googledoc19:34
jrollrloo: which one?19:35
rloojroll: https://review.openstack.org/#/c/155848/19:35
rloojroll: sorry, guess that would help19:35
jrollrloo: heh, I believe that's been superceded, need to abandon19:35
*** puranamr has quit IRC19:35
jrolland done, sorry about that19:35
rloojroll: thx. i'll see if i can find the patches for ironic-neutron integration.19:36
jrollrloo: should be https://review.openstack.org/#/q/status:open+project:openstack/ironic+branch:master+topic:bp/network_provider,n,z19:36
jrolljenkinsssss why19:36
jrolloh, that's still the wsme update problem19:36
rloojroll: ok thx. i'll update our spreadsheet19:36
jrollthanks!19:37
aarefiev22jroll: 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 results19:37
*** alexpilotti has joined #openstack-ironic19:39
jrollaarefiev22: ok, I can just update it for you today if that's ok :)19:39
aarefiev22jroll: sure, thanks a lot, I'm writting  from phone19:40
rlooNobodyCam, BadCub, or jroll do you have powers? Need to approve this BP: https://blueprints.launchpad.net/ironic/+spec/ironic-ml2-integration19:40
aarefiev22jroll: it would be my next level - update paches from smartphone)19:40
rlooNobodyCam, BadCub, jroll: and set url to http://specs.openstack.org/openstack/ironic-specs/specs/approved/ironic-ml2-integration.html19:41
jrollrloo: I haz no power19:41
jrollaarefiev22: lol, yes will update today :)19:41
rloojroll: so sad. too much power in hp hands me thinks :)19:41
jrollrloo: :/19:42
jrollrloo: only deva has power to update that group, too19:42
rloojroll: of course, we should be able to automate all this. oh well, it isn't a blocker.19:42
rloojroll: will let deva clean it up when he's back19:42
jrollrloo: indeed.19:43
jrollactually...19:43
jrolldhellmann: do you have the power to add me to ironic-drivers on launchpad?19:43
dhellmannjroll: maybe, let's see19:43
dhellmannjroll: is your lp id "jroll"19:43
dhellmann?19:44
dhellmannnm, I found you19:44
dhellmannjroll: done19:44
rlooooo, dhellmann has lots of power. good person to know :D thx dhellmann19:45
dhellmannrloo: :-)19:45
jrolldhellmann: yay, thanks!19:45
jrollrloo: approved and url set19:46
rloothx jroll19:46
jrollnp19:46
rlooNobodyCam, BadCub: ^^ never mind; we got it done w/o you.19:47
BadCubcool. Didn't mean to ignore, been heads down on internal things that required immediate attention19:48
*** aarefiev22 has quit IRC19:48
rlooBadCub: no worries19:48
rloojroll: 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
jrollrloo: they're kind of the same thing, they're in the same patch chain19:51
jrollrloo: the "ml2 integration" spec covers the data model so those are first19:52
rloojroll: oh. ok,19:52
jrollthe "add portgroups" patches19:52
rloojroll: got it. thx.19:53
jrollnp19:53
NobodyCamrloo: lol :)19:54
*** meghal has joined #openstack-ironic19:55
openstackgerritJim Rollenhagen proposed openstack/python-ironicclient: Fix functional tests job  https://review.openstack.org/21713519:55
jroll^ just removed unnecessary __init__.py, easy +2 otherwise19:56
*** Nisha has joined #openstack-ironic19:56
JoshNangjroll: i'm just gonna fix up 215505, looks like a minor change to make it landable19:57
jrollJoshNang: by all means19:57
*** Nisha has quit IRC19:57
*** Nisha has joined #openstack-ironic19:57
openstackgerritLindsay Kusmierek proposed openstack/ironic-python-agent: Fix plural agreement grammar error  https://review.openstack.org/22029319:58
*** Nisha has quit IRC19:58
*** Nisha has joined #openstack-ironic19:59
*** Marga_ has quit IRC20:01
*** derekh has joined #openstack-ironic20:02
*** Nisha has quit IRC20:08
*** e0ne has quit IRC20:12
jrolllekha: hmmmmm20:12
jrollCollecting 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
jrollNo matching distribution found for mimic>=1.6.0 (from -r global-requirements.txt (line 248))20:12
*** baoli_ has quit IRC20:13
jrolllekha: 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 thatr20:13
jrollmaybe a glyph question, idk, I don't immediately see external things in setup.py20:15
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Remove ordereddict as 2.6 is no longer supported  https://review.openstack.org/22029820:16
lekhajroll: hmm, not sure why that would be. Looking20:16
jrolllekha: no rush but it's going to block this thing20:17
jrollalso people are going to yell about twisted getting pulled in, heh20:17
jroll(global-requirements thing)20:17
*** dguerri` is now known as dguerri20:18
*** achanda has joined #openstack-ironic20:18
*** alexpilotti has quit IRC20:19
*** dguerri is now known as dguerri`20:20
JayFjoshnang: jroll: ^ 220298 is another easy ipa review20:20
jrollJayF: refresh20:20
*** derekh has quit IRC20:20
JoshNang+A'ed20:23
*** achanda has quit IRC20:24
*** e0ne has joined #openstack-ironic20:25
*** e0ne has quit IRC20:26
*** Marga_ has joined #openstack-ironic20:27
*** achanda has joined #openstack-ironic20:29
openstackgerritJosh Gachnang proposed openstack/ironic: Validate the input of properties  https://review.openstack.org/21550520:31
JoshNangjroll: ^20:33
jrollnice, thanks20:33
*** meghal has quit IRC20:34
*** achanda has quit IRC20:38
*** thiagop has joined #openstack-ironic20:42
*** priteau has joined #openstack-ironic20:42
thiagopmorning Ironicers20:42
*** priteau has quit IRC20:43
thiagopquick 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-ironic20:45
jrolllekha: found the bug :/20:46
*** achanda has joined #openstack-ironic20:47
rloojroll, 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.html20:47
* BadCub looks20:49
BadCubrloo: ^^ done20:50
lekhajroll: anything I ought to do in mimic20:50
rloothx BadCub. NobodyCam, jroll: ^^ don't bother, it's been done20:50
NobodyCamhehehe :)20:51
*** trown is now known as trown|outttypeww20:51
lekhaI 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 devstack20:52
lekhaany thoughts on whether I should be doing that as another functional test20:53
lekhajlvillal: ^20:53
jrolllekha: so for mimic... you're only publishing wheels, and only for python 2, so when it tries to install on py3 it just hard fails20:55
jrolllekha: I think if you publish an sdist it should just work20:55
lekhajroll: ok20:56
jroll(I'm not sure if it runs on py3 or not, but we can solve that problem... eventually)20:56
jrollstepping away for a while20:56
*** harshs has quit IRC20:59
*** harshs has joined #openstack-ironic21:01
lekhajroll: I just release 1.7.0 with the add/delete node capabilities. And that has the sdist.21:02
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Ensure all methods in util.py have docstrings.  https://review.openstack.org/22031321:03
*** harshs has quit IRC21:03
*** jjohnson2 has quit IRC21:03
jrolllekha: thanks I'll try it21:03
lekhajroll: thanks :)21:03
*** meghal has joined #openstack-ironic21:03
*** greghaynes has quit IRC21:10
jrolllekha: aaaand updated https://review.openstack.org/#/c/220268/21:12
jrollthanks for the assist21:13
jrollactually stepping away for a while now21:13
jroll... maybe after infra convo, heh21:17
jlvillallekha: 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
JoshNangjlvillal: it looks like what's in the functional testing for the client runs against devstack, right?21:23
jlvillalJoshNang: I think so. adam_g would know for sure.21:23
jlvillalJoshNang: 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
JoshNangjlvillal: unittests and ipa's functional jobs21:24
jlvillalJoshNang: Ah yeah. Duh!21:25
adam_gthe python-ironicclient functiional test runs against a minimal devstack that has only ironic/rabbit/db running21:25
adam_g(at least it did last time i looked)21:25
JoshNangadam_g: thanks! i think it still does, or it's doing some magic i can't figure out21:25
jlvillaladam_g: Thanks! I doubt it has changed. lekha is looking into using mimic.21:25
adam_gwhats mimic?21:25
adam_git 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 gate21:26
jlvillaladam_g: I am not the expert. mimic pretends to be a service that responds to REST API calls.21:26
JoshNangbasically a mock for api services (as i understand it)21:26
jlvillalI think you tell it, if you get this HTTP request, respond like this.  lekha is the expert on it.21:27
adam_gok ya, so in theory that should work too assuming  mimic is doing  what its supposed to21:27
JoshNangyeah, seems like they're both valuable in different ways21:27
JoshNangi still like the 'actually talk to a running ironic instance' jobs21:28
adam_gya 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 else21:28
jlvillal+1, good value in that.21:28
openstackgerritMerged openstack/ironic: Remove 'is_valid_event' method  https://review.openstack.org/21991821:29
* jlvillal wonders how we will know if an event is valid now...21:29
*** penick has quit IRC21:29
openstackgerritMerged openstack/ironic: Updated from global requirements  https://review.openstack.org/21890621:30
openstackgerritMerged openstack/ironic: agent_ilo fails to bring up instance  https://review.openstack.org/22000321:30
*** meghal has quit IRC21:30
*** penick has joined #openstack-ironic21:32
*** jcoufal has quit IRC21:32
*** meghal has joined #openstack-ironic21:36
lekhajroll: thanks21:36
lekhaadam_g: mimic will allow for running the functional tests locally21:36
adam_glekha, 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 good21:37
*** harshs has joined #openstack-ironic21:38
lekhaadam_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_glekha, :)21:38
*** marzif has joined #openstack-ironic21:39
*** harshs has quit IRC21:46
*** vvb1 has quit IRC21:59
*** harshs has joined #openstack-ironic22:01
openstackgerritThiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic  https://review.openstack.org/19182222:08
*** meghal has quit IRC22:10
*** penick has quit IRC22:14
openstackgerritThiago Paiva Brito proposed openstack/python-oneviewclient: Sending OneView Client implementation  https://review.openstack.org/21279622:26
thiagopGood night folks22:26
thiagopsee you tomorrow22:26
jrollnn thiagop :)22:26
thiagopjroll: ?22:26
jroll"night night"22:27
jroll:P22:27
thiagopjroll: awn, I understood as "no no", like "WAAIT!"22:27
jrollWAIT WE LIKE YOU22:27
jrollhow's that?22:27
* thiagop is flattered22:28
thiagophave a good night, sir22:28
thiagop;)22:28
*** thiagop has quit IRC22:28
jrollyou too22:28
*** penick has joined #openstack-ironic22:31
*** greghaynes has joined #openstack-ironic22:32
*** Marga_ has quit IRC22:35
*** natorious is now known as zz_natorious22:37
*** Marga_ has joined #openstack-ironic22:37
*** MattMan1 has left #openstack-ironic22:42
*** Marga_ has quit IRC22:43
openstackgerritMerged openstack/ironic-python-agent: Fix plural agreement grammar error  https://review.openstack.org/22029322:43
*** Marga_ has joined #openstack-ironic22:44
*** penick has quit IRC22:45
*** harshs has quit IRC22:45
*** penick has joined #openstack-ironic22:47
openstackgerritLindsay Kusmierek proposed openstack/ironic-python-agent: Add docstrings to the IPA api.controllers tree  https://review.openstack.org/22033922:52
jrollJoshNang: pep8 on 215505 :(22:52
jrollchange looks fine though22:53
JoshNangjroll: yeah, just saw that :/ my editor stopped doing the line at 80 chars for some reason22:54
openstackgerritJosh Gachnang proposed openstack/ironic: Validate the input of properties  https://review.openstack.org/21550522:54
jrolllol22:54
* JoshNang shakes fist at pycharm22:54
jrollset colorcolumn=8022:55
jrollftw22:55
jroll+2 btw22:55
JoshNang\o/22:56
JoshNangthanks!22:56
jrollthank you sir22:56
*** meghal has joined #openstack-ironic23:09
jlvillalJoshNang: I'll admit that patch confuses me :)23:09
*** Marga_ has quit IRC23:12
*** meghal has quit IRC23:13
*** zz_natorious is now known as natorious23:15
JoshNangjlvillal: how so?23:16
jlvillalJoshNang: Mainly because I hadn't read the code about IronicObject. It makes sense now...23:16
openstackgerritJay Faulkner proposed openstack/ironic-python-agent: Add docstrings for cmd/agent and extensions/clean  https://review.openstack.org/22034623:16
JoshNangjlvillal: no worries. i started at the end of the chain earlier and was quite confused23:18
jlvillalJoshNang: 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-ironic23:21
*** marzif has quit IRC23:21
JoshNangjlvillal: i'd tell you to fight with tan lin about it ;) i was just fixing an error message so it would be landable23:21
jlvillal:)23:21
*** Sukhdev_ has joined #openstack-ironic23:23
jlvillalJoshNang: +123:23
*** meghal has joined #openstack-ironic23:24
jrollJoshNang: are you actually working on https://bugs.launchpad.net/ironic/+bug/148382823:27
openstackLaunchpad bug 1483828 in Ironic "IPA exceptions are not printed properly" [High,In progress] - Assigned to Josh Gachnang (joshnang)23:27
JoshNangjroll: that's done, i believe23:28
jrollorly23:28
jrollgot a patch?23:28
JoshNanghttps://review.openstack.org/#/c/211692/23:28
JoshNangboo, forgot to do closes-bug23:28
*** puranamr has quit IRC23:29
*** penick has quit IRC23:29
jrollthanks23:29
*** bradjones has quit IRC23:31
*** puranamr has joined #openstack-ironic23:31
*** achanda has quit IRC23:32
*** dims has quit IRC23:44
*** ijw_ has quit IRC23:45
*** puranamr has quit IRC23:48
*** shoutm has joined #openstack-ironic23:50
openstackgerritJay Faulkner proposed openstack/ironic: Add python3-dev[el] to list of prereqs for devs  https://review.openstack.org/22035223:50
*** dims has joined #openstack-ironic23:53
*** derekh has joined #openstack-ironic23:53

Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!