Tuesday, 2014-07-29

*** eguz has quit IRC00:04
rloomrda: thx for explaining.00:08
*** rainya has quit IRC00:09
rloojroll, JoshNang, JayF: you guys happy with 20 minutes now? (this is where the spec ought to be updated but anyway)00:12
rloojroll, JoshNang: should I approve 81391 or wait for more comments?00:15
jrollrloo: go ahead :D00:20
rloojroll: done. Now to pray to the jenkins god, cuz it isn't clear to me that jenkins is that happy today.00:22
*** eghobo has joined #openstack-ironic00:25
*** eghobo has quit IRC00:25
*** eghobo has joined #openstack-ironic00:36
mrdarloo: np, thanks for the +2 :-)00:51
*** achanda has quit IRC00:52
*** wanyen has quit IRC00:53
*** scubacuda has quit IRC00:58
openstackgerritA change was merged to openstack/ironic: Adding swift temp url support  https://review.openstack.org/8139101:09
*** scubacuda has joined #openstack-ironic01:09
*** ellenh has quit IRC01:14
*** rloo has quit IRC01:21
*** eguz has joined #openstack-ironic01:29
*** eghobo has quit IRC01:33
*** eguz has quit IRC01:35
*** eghobo has joined #openstack-ironic01:36
*** _sfb_ has joined #openstack-ironic01:41
*** rwsu has quit IRC01:42
*** eghobo has quit IRC01:44
*** nosnos has joined #openstack-ironic01:48
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements send-data-to-ceilometer  https://review.openstack.org/7253802:26
*** rainya has joined #openstack-ironic02:44
*** mdorman has quit IRC02:51
*** rainya has quit IRC02:54
*** rainya has joined #openstack-ironic02:54
*** harlowja is now known as harlowja_away03:07
*** harlowja_away is now known as harlowja03:08
*** nosnos has quit IRC03:20
*** eghobo has joined #openstack-ironic03:29
*** mitz has quit IRC04:06
*** mitz has joined #openstack-ironic04:10
*** nosnos has joined #openstack-ironic04:16
*** eghobo has quit IRC04:42
*** eghobo has joined #openstack-ironic04:42
*** eguz has joined #openstack-ironic04:44
*** eghobo has quit IRC04:48
*** rainya has quit IRC05:10
*** harlowja is now known as harlowja_away05:26
*** sirushti_ has joined #openstack-ironic05:49
*** scubacuda has quit IRC05:55
*** Mikhail_D_ltp has joined #openstack-ironic05:56
*** LiveOne has quit IRC06:00
*** victor_lowther has quit IRC06:02
*** LiveOne has joined #openstack-ironic06:05
*** victor_lowther has joined #openstack-ironic06:10
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/11021506:10
*** victor_lowther has quit IRC06:22
*** pcrews has quit IRC06:22
*** victor_lowther has joined #openstack-ironic06:29
*** viktors|afk has quit IRC06:36
*** ndipanov has joined #openstack-ironic06:37
*** sirushti_ has quit IRC06:40
openstackgerritGopi Krishna S proposed a change to openstack/ironic-specs: Cisco UCS Driver  https://review.openstack.org/11021706:41
*** romcheg1 has joined #openstack-ironic06:48
*** eguz has quit IRC06:51
*** _sfb_ has quit IRC06:54
*** _sfb_ has joined #openstack-ironic06:58
Isotoppjroll: what rloo said, i was following http://docs.openstack.org/developer/ironic/deploy/install-guide.html#set-up-the-database-for-bare-metal on a clean machine07:15
Isotoppit assumes mysql, in the sqlalchemy instructions, but assumes the module is already present and it is not a dependency07:15
Isotoppso I needed to do07:17
Isotopp 1309  aptitude install libmysqlclient-dev07:17
Isotopp 1310  pip install MySQL-python07:17
openstackgerritGopi Krishna S proposed a change to openstack/ironic-specs: Cisco UCS Driver  https://review.openstack.org/11021707:22
*** jistr has joined #openstack-ironic07:30
*** ndipanov has quit IRC07:34
*** ifarkas has joined #openstack-ironic07:36
*** ndipanov has joined #openstack-ironic07:49
*** lsmola has joined #openstack-ironic07:49
*** jistr_ has joined #openstack-ironic07:58
*** jistr_ has quit IRC07:59
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Update_port should expect MACAlreadyExists  https://review.openstack.org/10910708:02
*** dtantsur|afk is now known as dtantsur08:09
dtantsurMorning, Ironic. Quite an easy patch above ^^^, I just changed the commit message08:09
*** romcheg1 has quit IRC08:10
yuriyzmorning all08:16
dtantsuryuriyz, morning08:19
*** killer_prince is now known as lazy_prince08:19
*** lazy_prince is now known as killer_prince08:22
*** ndipanov has quit IRC08:25
*** MattMan has quit IRC08:29
openstackgerritHaomeng,Wang proposed a change to openstack/ironic: Implements send-data-to-ceilometer  https://review.openstack.org/7253808:30
*** MattMan has joined #openstack-ironic08:30
*** Haomeng has quit IRC08:35
*** Haomeng has joined #openstack-ironic08:37
*** ndipanov has joined #openstack-ironic08:38
*** leopoldj has joined #openstack-ironic08:42
Isotopphm, i was thinking, maybe I can have multiple compute backends in nova like I can have multiple storage backends for cinder. but apparently there is https://blueprints.launchpad.net/nova/+spec/multi-back-ends-for-nova-compute08:50
Isotoppand i can't, yet08:50
openstackgerritA change was merged to openstack/python-ironicclient: Export 'client' and 'exc' modules  https://review.openstack.org/10897408:58
openstackgerritA change was merged to openstack/ironic: Update_port should expect MACAlreadyExists  https://review.openstack.org/10910709:20
*** derekh_ has joined #openstack-ironic09:33
dtantsurFolks, one more relatively easy thing to approve, if someone has time, please https://review.openstack.org/#/c/109610/09:56
openstackgerritGhe Rivero proposed a change to openstack/ironic: Raise MissingParameterValue instead of invalid  https://review.openstack.org/10845509:56
openstackgerritDmitry Tantsur proposed a change to openstack/ironic: Do not rely on hash ordering in tests  https://review.openstack.org/10993910:06
*** rakesh_hs has joined #openstack-ironic10:12
*** ryanpetrello has quit IRC10:16
*** ryanpetrello has joined #openstack-ironic10:21
*** marzif has joined #openstack-ironic10:22
*** Alexei_987 has joined #openstack-ironic10:27
Zerpexyoo people - can any send me their init scripts for ironic? seems like they wasn't a part of the git repo :(10:38
*** Haomeng|2 has joined #openstack-ironic10:44
dtantsurI don't have any - I use devstack which starts services manually10:45
*** Haomeng has quit IRC10:46
Zerpexwhatever - I'll do the same :310:48
openstackgerritA change was merged to openstack/ironic: SSH virsh to use the new ManagementInterface  https://review.openstack.org/8988411:01
dtantsurZerpex, well, if you just want to start experimenting with Ironic, devstack may be the best choice.11:13
dtantsurZerpex, that's how I do it on Fedora: https://etherpad.openstack.org/p/IronicTestingOnFedora11:14
*** krtaylor has quit IRC11:23
*** GheRiver1 is now known as GheRivero11:32
*** Haomeng has joined #openstack-ironic11:37
*** rakesh_hs has quit IRC11:37
*** Haomeng|2 has quit IRC11:37
openstackgerritGhe Rivero proposed a change to openstack/ironic: Raise MissingParameterValue instead of invalid  https://review.openstack.org/10845511:59
*** dguerri`afk is now known as dguerri12:10
openstackgerritImre Farkas proposed a change to openstack/ironic-specs: DRAC management driver  https://review.openstack.org/10703312:13
*** sirushti_ has joined #openstack-ironic12:23
Zerpexdtantsur: thanks - but since I already did configure openstack using RDO (bad choice :P) - I needed to do stuff manually - but any that knows how to fix12:27
Zerpexhttp://snaps.lucasrolff.com/bude4532c0m3as3.png12:27
dtantsurlemme find12:27
dtantsurZerpex, oh, I guess we don't have it documented... You need to set some parameters into driver_info field of a node12:29
*** jistr has quit IRC12:29
dtantsurwhat IPMI driver do you use?12:29
ZerpexI have access to an drac interface as the only thing I guess - let me ask if there's another ipmi thing12:30
dtantsurZerpex, for `deploy` part you need to set `pxe_deploy_kernel`, `pxe_deploy_ramdisk`12:30
dtantsurZerpex, I mean, what did you set as a `driver` for your node?12:30
Zerpexpxe_ipmitool12:30
dtantsurbrb12:30
dtantsurfor ipmitool: ipmi_address, maybe ipmi_username and ipmi_password12:32
Zerpexthose I have set at least :D12:32
openstackgerritImre Farkas proposed a change to openstack/ironic-specs: DRAC vendor passthru for RAID management  https://review.openstack.org/10798112:32
*** jistr has joined #openstack-ironic12:33
dtantsurZerpex, than it should be ok, other PXE drivers will be set by Nova12:33
dtantsur* drivers = settings12:33
*** dtantsur is now known as dtantsur|lunch12:33
Zerpexthanks dtantsur|lunch guess I only need to find out now, why I keep having 'scheduling' when trying to deploy cirros onto the machine :D12:48
*** nosnos has quit IRC12:51
*** Mikhail_D_ltp has quit IRC12:51
*** dtantsur|lunch is now known as dtantsur12:53
*** jasondotstar has joined #openstack-ironic12:57
*** jistr has quit IRC13:01
*** jgrimm has joined #openstack-ironic13:02
*** jistr has joined #openstack-ironic13:03
*** rloo has joined #openstack-ironic13:12
*** krtaylor has joined #openstack-ironic13:14
*** jbjohnso has joined #openstack-ironic13:24
*** dguerri is now known as dguerri`afk13:25
*** dguerri`afk is now known as dguerri13:29
*** lnxnut has joined #openstack-ironic13:48
*** pcrews has joined #openstack-ironic13:52
*** Mikhail_D_ltp has joined #openstack-ironic14:03
*** jistr has quit IRC14:12
*** jistr has joined #openstack-ironic14:13
*** dhellmann has quit IRC14:17
*** chuckC has joined #openstack-ironic14:18
*** krtaylor has quit IRC14:18
*** krtaylor has joined #openstack-ironic14:19
*** dhellmann has joined #openstack-ironic14:20
*** rainya has joined #openstack-ironic14:31
*** rainya has quit IRC14:32
*** rainya has joined #openstack-ironic14:32
*** mdorman has joined #openstack-ironic14:35
*** romcheg1 has joined #openstack-ironic14:41
*** dguerri is now known as dguerri`afk14:43
*** dguerri`afk is now known as dguerri14:45
*** romcheg1 has quit IRC14:48
*** dguerri is now known as dguerri`afk15:01
*** dguerri`afk is now known as dguerri15:01
*** jistr has quit IRC15:11
derekh_Hi, looks like our tripleo ironic job has started failing15:16
derekh_2014-07-29 14:38:05.781 4234 TRACE nova.compute.manager [instance: 7afb31da-61da-47b9-8e67-c84c86a091f4] InstanceDeployFailure: Failed to provision instance 7afb31da-61da-47b9-8e67-c84c86a091f4: Failed to deploy. Error: Failed to execute command via SSH: EDITOR="sed -i '//d;/<\/os>/i\'" /usr/bin/virsh --connect qemu:///system edit baremetalbrbm6_0.15:16
derekh_which looks like its commig from here https://review.openstack.org/#/c/89884/18/ironic/drivers/modules/ssh.py,cm15:17
derekh_the command in the logs doesn't look right e.g. "<boot \(dev\|order\)=*\>" is missing (not escaped right??)15:21
derekh_derekh_: also this wont work on our testenvs because their locked down to only allow certain command to be run15:21
derekh_see RE_COMMANDS http://git.openstack.org/cgit/openstack-infra/tripleo-ci/tree/elements/testenv-worker/bin/ci_commands#n6415:22
rlooderekh_: lucas isn't online now, but he should be within the hour I think. He's at the meetup.15:29
derekh_rloo: ok thanks15:30
* jroll rubs his eyes15:30
rlooderekh_: in the meantime, maybe open a bug?15:30
jrollgooooood morning15:30
derekh_I've opend a bug , bug 134991315:30
rloothx derekh_.15:30
rloojroll: good morning!15:31
jrollderekh_: I think that patch just landed, and I've tested it in devstack, so I don't think that's the real issue15:31
jrollmorning rloo :)15:31
derekh_jroll: the test environments tripleo maintains don't allow you to run arbitrary command over ssh15:32
jrollderekh_: yeah, just checked, you'll need to update that list15:32
jrollI think15:32
jrollalthough I would think the pxe driver doesn't actually use that function15:33
jrollrloo, dtantsur: goal today is to land (or get close to landing) https://review.openstack.org/10102015:34
dtantsurit's doubtful, that this new code will be executed in tests...15:34
jrollplease review :)15:34
jrolldtantsur: right...15:34
derekh_jroll: Yup, unfortunately this involves taking down and rebuilding all testenvs,15:34
jrollderekh_: have logs for this failure?15:34
dtantsurjroll, did someone finally approve the spec?15:34
jrolldtantsur: no ><15:34
rloojroll: I have a rule about not reviewing patches until their spec is approved.15:34
jrollwell15:34
rloojroll: are you asking me to break one of my rules?15:34
jrollreview the spec then?15:34
derekh_jroll: http://logs.openstack.org/89/108989/4/check-tripleo/check-tripleo-ironic-undercloud-precise-nonha/813fe50/logs/seed_logs/nova-compute.txt.gz15:34
dtantsurjroll, oh... I guess I already +2ed the spec, right?15:34
jrollor know the spec will likely land today15:34
jrolldtantsur: yeah :P15:35
rloojroll: I've already +1'd the spec.15:35
jrollah ha15:35
jrollthen ignore me :)15:35
jrollbut know that a large part of today will be working on landing 10102015:36
rloojroll: who's going to approve the spec? light a fire under them...15:36
dtantsurjroll, catch Lucas or devananda15:36
jrollderekh_: this is a continuous failure?15:36
rloojroll: a large part of your day, not mine? :-)15:36
dtantsurheh, my day is ending :)15:36
jrolldtantsur, rloo: JayF and NobodyCam made specs core yesterday, /me pokes them15:36
jrollrloo: :P15:36
dtantsurjroll, oh, good news! so we have no more excuses for not approving specs for weeks :D15:37
rloogreat news!15:37
derekh_jroll: its the same error in the 4 CI runs I've looked at15:38
jrollderekh_: so, here's where this comes from https://github.com/openstack/ironic/blob/master/ironic/drivers/modules/pxe.py#L52715:38
jrollderekh_: set_boot_device for the ssh driver is no longer a no-op15:38
jrollderekh_: so, while I apologize for us landing a thing that didn't pass your tests... it's too late now :|15:39
dtantsurwow. that's unexpected :-/ we need to make the job voting asap15:40
derekh_jroll: it didn't pass your test either15:40
derekh_check-tripleo-ironic-undercloud-precise-nonha FAILURE in 44m 40s15:40
dtantsurderekh_, non-voting jobs, especially those which fail too often... you know :)15:40
dtantsurderekh_, what can be done to fix it?15:41
rloowe can undo our change15:41
derekh_dtantsur: yup, agreed they fail to often, although these days because of legitimate bugs in various projects15:41
dtantsurrloo, I would see it as the last thing to do...15:41
derekh_dtantsur: we can do one of 2 things15:41
rloodtantsur: sure, if we can't figure out a solution soon. I don't like it that we broke their gate.15:42
dtantsurrloo, of course!15:42
*** rwsu has joined #openstack-ironic15:42
derekh_dtantsur: rebuild all of the test envs to allow that command15:43
jrollderekh_: right, I'm sorry we missed that15:43
derekh_dtantsur: I can start testing an update to see if it will work but probably wont have the time to complete until tomorrow (unless another tripleo-cd-admin) has time to do it15:43
jrollderekh_: I think the right thing to do is to rebuild the environment, unfortunately15:44
dtantsurWell, we can revert a change, but you still have to rebuild the env...15:44
jrollderekh_: we need that patch in to be able to test/land some more important work being done at our sprint today15:44
jrolldtantsur: exaxtly15:44
dtantsurso I would go that way, if it's possible in a reasonable amount of time15:44
jrollexactly, too, cannot type today15:45
derekh_dtantsur: yes I will have to rebuild regardless of what we do, also there is the question of the command itself15:45
derekh_dtantsur: see the log entry, the command being run doesn't match whats in the code15:45
derekh_<derekh_> the command in the logs doesn't look right e.g. "<boot \(dev\|order\)=*\>" is missing (not escaped right??)15:45
dtantsurderekh_, strange... it worked locally for me...15:46
*** dguerri is now known as dguerri`afk15:47
derekh_dtantsur: weird it also passed on patchset 1715:47
dtantsureven more strange... is it possible that it's actually correct? maybe wrong logging?15:48
* dtantsur is just throwing the ideas15:48
*** dguerri`afk is now known as dguerri15:49
*** ifarkas has quit IRC15:49
derekh_dtantsur: ahh, I think I can explain it, before the 25th was the error caught or the interface not used https://review.openstack.org/#/c/86588/2715:54
dtantsurderekh_, I don't get it... patchset 17 should not pass anyway15:56
dtantsurit's baded on review you mention15:56
dtantsur* based even15:56
*** eghobo has joined #openstack-ironic15:56
jrollderekh_: ssh driver set_boot_device was a no-op before that patch landed today15:56
jrollderekh_: also, that command is escaped fine. "<boot \(dev\|order\)=*\>" is part of a sed command15:57
*** eguz has joined #openstack-ironic15:58
*** hemna has joined #openstack-ironic16:01
*** eghobo has quit IRC16:02
*** lucasagomes has joined #openstack-ironic16:02
derekh_dtantsur: jroll ok, testing revert in tripleo CI in the meantime investigating an testenv update16:02
mrda\o16:02
NobodyCamgood morning Ironic16:03
dtantsurderekh_, ack16:03
derekh_lucasagomes: hi ya, FYI tripleo ironic test is failing see https://bugs.launchpad.net/ironic/+bug/134991316:03
mrdaNobodyCam: and good morning to you...16:03
dtantsurmrda, NobodyCam, morning :)16:03
lucasagomesderekh_, :( I heard I broke ooo16:03
lucasagomeswill take a look16:04
NobodyCammrda: \o16:04
NobodyCammrning dtantsur16:04
*** hemna has quit IRC16:04
mrdahey dtantsur16:04
*** hemna has joined #openstack-ironic16:04
derekh_lucasagomes: I'm looking into updating the testenvs, not sure if I'll get that done before I leave16:05
lucasagomesderekh_, ack I will take a look16:05
lucasagomesderekh_, you wanna revert that patch16:05
lucasagomes?16:05
derekh_lucasagomes: I'm going to have to update the testenvs anyways but it would take the pressure off16:06
lucasagomesderekh_, ack16:07
JayFjroll: https://review.openstack.org/#/c/98506/7 imo you should fix the blank line that can fix the render issue dtantsur pointed out16:08
JayFjroll: there are enough people that you should be able to get the +2 back16:08
jrollJayF: mehhhhhhhhhhhhhhhhhhhhhhhhhhhhh16:09
jrollit's a new line issue16:09
jrollin a spec16:09
JayFjroll: http://25.media.tumblr.com/tumblr_ltdpwkHbqj1qcro1do1_500.gif16:09
jrollJayF: nope.gif16:10
*** scubacuda has joined #openstack-ironic16:23
*** Nisha has joined #openstack-ironic16:25
*** ellenh has joined #openstack-ironic16:26
*** rakesh_hs has joined #openstack-ironic16:31
openstackgerritJim Rollenhagen proposed a change to openstack/ironic: Add ironic-python-agent deploy driver  https://review.openstack.org/10102016:31
*** rakesh_hs2 has joined #openstack-ironic16:32
*** rakesh_hs has quit IRC16:35
*** hemna has joined #openstack-ironic16:37
*** chuckC has quit IRC16:38
*** Alexei_987 has quit IRC16:41
*** Nisha has quit IRC16:46
openstackgerritA change was merged to openstack/ironic: Split test case in ironic.tests.conductor.test_manager  https://review.openstack.org/10961016:53
*** marzif has quit IRC16:54
*** leopoldj has quit IRC16:57
*** eguz has quit IRC16:59
*** rainya has quit IRC17:00
*** rainya has joined #openstack-ironic17:07
*** rainya has quit IRC17:08
*** eghobo has joined #openstack-ironic17:08
*** rainya has joined #openstack-ironic17:09
ShrewsSo, reading up on (and playing with) docker. Am I totally bonkers for thinking we maybe could use this as a tool to do some embedded scenario testing that tempest can't currently cover?17:09
derekh_lucasagomes: I gotta run, still working on the testenv change but not complete, I've kicked off a temp revert for tripleo CI https://review.openstack.org/#/c/110352/ if you can can you check in on that later17:09
Shrewsdevananda: thinking crazy thawts above ^^^^  :)17:10
*** derekh_ has quit IRC17:10
*** coolsvap has quit IRC17:22
rloowhat do people do, when reviewing those 'Imported Translations from Transifex' patches?17:24
*** coolsvap has joined #openstack-ironic17:26
Shrewsrloo: mainly just make sure it looks sane17:29
rlooShrews: it is allllllll sane to me! :-)17:29
rlooShrews: I don't think Ironic gets much translated, and the English part I can handle. So these should be fairly straightforward to approve then.17:30
Shrewsi seem to recall translation tests existing somewhere... or maybe i'm mis-remembering17:31
dtantsurshopping time, folks :) maybe will be back later17:34
*** dtantsur is now known as dtantsur|afk17:34
*** harlowja_away is now known as harlowja17:36
JayFhttps://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/reboot.sh is the best script ever17:39
JayFjroll: ^ remember that? hahaha17:39
jrollJayF: ogod17:39
devanandarloo: sanity check, then +2/+A17:40
rloothx devananda!17:41
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Add two undeclared dependencies to the Dockerfile  https://review.openstack.org/11039817:54
JayFQuick and easy review: https://review.openstack.org/#/c/110398 -- just adding two dependencies to Dockerfile, no actual impact (the deps are in the docker base image) but better to be explicit17:55
*** romcheg1 has joined #openstack-ironic18:13
openstackgerritA change was merged to openstack/ironic: Imported Translations from Transifex  https://review.openstack.org/11021518:35
*** wanyen has joined #openstack-ironic18:38
*** jcoufal has joined #openstack-ironic18:39
*** romcheg1 has quit IRC18:40
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/10656918:41
*** scubacuda has quit IRC18:48
*** christopheraedo has quit IRC18:49
*** christopheraedo has joined #openstack-ironic18:52
*** jistr has joined #openstack-ironic18:56
*** igordcard has joined #openstack-ironic19:09
*** scubacuda has joined #openstack-ironic19:19
*** ellenh has quit IRC19:26
*** rainya has quit IRC19:28
*** rainya has joined #openstack-ironic19:29
*** derekh_ has joined #openstack-ironic19:30
*** ellenh has joined #openstack-ironic19:43
*** rakesh_hs2 has quit IRC19:44
*** openstackgerrit has quit IRC19:48
*** jasondotstar has quit IRC19:51
*** jasondotstar has joined #openstack-ironic19:52
*** romcheg1 has joined #openstack-ironic19:57
*** mikal_ is now known as mikal20:01
*** eguz has joined #openstack-ironic20:04
*** jistr has quit IRC20:06
*** eghobo has quit IRC20:07
*** derekh_ has quit IRC20:12
*** hemna is now known as hemnafk20:14
*** Penick has joined #openstack-ironic20:17
*** Mikhail_D_ltp has quit IRC20:21
JayFJoshNang: https://review.openstack.org/#/c/110451/20:23
*** lnxnut has quit IRC20:30
*** rainya has quit IRC20:30
*** rainya has joined #openstack-ironic20:31
*** rainya has quit IRC20:33
*** rainya has joined #openstack-ironic20:34
*** dguerri is now known as _dguerri20:45
*** _dguerri is now known as dguerri`afk20:45
*** hemnafk is now known as hemna20:57
*** openstackgerrit has joined #openstack-ironic20:58
*** eguz has quit IRC21:03
*** romcheg1 has quit IRC21:08
*** eghobo has joined #openstack-ironic21:10
*** tatyana has joined #openstack-ironic21:12
JayFJoshNang: jroll: others --> https://review.openstack.org/#/c/110451/21:13
*** eghobo has quit IRC21:15
*** dguerri`afk is now known as _dguerri21:15
JayFjroll: JoshNang others --> https://review.openstack.org/#/c/110463/21:18
JayFboth of those are deps for landing global requirements21:18
*** eghobo has joined #openstack-ironic21:19
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Upgrade pip inside the Docker build  https://review.openstack.org/11047321:25
*** sirushti_ has quit IRC21:25
*** jasondotstar has quit IRC21:25
*** sirushti_ has joined #openstack-ironic21:26
JayFI abandoned 110463; this is the better version: https://review.openstack.org/#/c/110473/121:26
*** sirushti_ has quit IRC21:30
*** _dguerri is now known as dguerri`afk21:31
jrollJayF: https://review.openstack.org/#/c/110451/2/ironic_python_agent/hardware.py21:41
*** stevebaker is now known as steveburt21:41
*** steveburt is now known as stevebaker21:41
lucasagomesdevananda, cubswin:)21:45
lucasagomesI think21:45
JayFjroll: decidedly meh21:45
jrollsomeone plssss https://review.openstack.org/#/c/98506/21:53
openstackgerritA change was merged to openstack/ironic-python-agent: Add support for psutil v1 as well as v2  https://review.openstack.org/11045121:57
openstackgerritGhe Rivero proposed a change to openstack/ironic: Raise MissingParameterValue instead of Invalid  https://review.openstack.org/10845522:00
*** chuckC has joined #openstack-ironic22:00
lucasagomesdevananda, there's a couple of comments on the api patch, I will fix them22:01
*** BadCub02 has quit IRC22:14
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Allow pip to use wheels inside the Docker build  https://review.openstack.org/11047322:21
*** rainya has quit IRC22:22
openstackgerritA change was merged to openstack/ironic: Do not rely on hash ordering in tests  https://review.openstack.org/10993922:24
*** tatyana has quit IRC22:25
*** Penick has quit IRC22:28
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Allow pip to use wheels inside the Docker build  https://review.openstack.org/11047322:28
*** rainya has joined #openstack-ironic22:28
openstackgerritJay Faulkner proposed a change to openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/8872222:29
*** rainya has quit IRC22:30
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-specs: Add deploy driver for ironic-python-agent  https://review.openstack.org/9850622:31
wanyenplease review https://review.openstack.org/#/c/107996/  got one +2 need one more reviewer.22:33
jrollwanyen: are you going to address ruby's comments?22:35
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE Installation Guide documentation  https://review.openstack.org/10680922:36
openstackgerritLucas Alvares Gomes proposed a change to openstack/ironic: Add iPXE support for Ironic  https://review.openstack.org/9931822:36
wanyenjroll: yes.  we will.  Thanks for point out ruby's comments.22:37
wanyenplease review https://review.openstack.org/#/c/97744/ and https://review.openstack.org/#/c/108445/.   The new reversions  has been uploaded for a while without any review.22:39
wanyens/reversion/version22:41
*** igordcard has quit IRC22:44
*** eghobo has quit IRC22:50
*** eghobo has joined #openstack-ironic22:51
lucasagomesJayF, https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo22:52
openstackgerritA change was merged to openstack/ironic-specs: Add deploy driver for ironic-python-agent  https://review.openstack.org/9850622:54
*** derekh_ has joined #openstack-ironic22:54
*** jgrimm has quit IRC22:56
*** jcoufal has quit IRC22:58
wanyenDeva, we removed Nova dependencies from UEFI boot spec https://review.openstack.org/#/c/99850.  Can you unblock this spec?23:01
openstackgerritJim Rollenhagen proposed a change to openstack/ironic-python-agent: Bump CoreOS to latest stable  https://review.openstack.org/11049423:05
jrollNobodyCam: how do I dib?23:07
jrollNobodyCam: python2.7-dev is not found in PATH. Please ensure your elements install it23:17
jrolloh, I see the problem23:18
*** rainya has joined #openstack-ironic23:20
*** rainya has quit IRC23:22
*** rainya has joined #openstack-ironic23:22
*** rainya has quit IRC23:27
jrollNobodyCam: it built23:27
jrollI doubt it's going to work...23:27
*** lucasagomes has quit IRC23:27
*** rainya has joined #openstack-ironic23:27
jrollhad to remove a bunch of things from binary deps, as they weren't binaries that existed23:27
*** dkehn_ has joined #openstack-ironic23:27
*** dkehnx has quit IRC23:31
openstackgerritGregory Haynes proposed a change to openstack/ironic: WIP: Add conductor_hostname property to nodes  https://review.openstack.org/10968823:32
*** _sfb_ has quit IRC23:34
*** mdorman has quit IRC23:36
*** Haomeng|2 has joined #openstack-ironic23:43
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic: Updated from global requirements  https://review.openstack.org/10656923:44
openstackgerritOpenStack Proposal Bot proposed a change to openstack/ironic-python-agent: Updated from global requirements  https://review.openstack.org/8872223:44
*** Haomeng has quit IRC23:45
*** derekh_ has quit IRC23:46
jrollJayF: when you have a moment https://review.openstack.org/11049423:47
jrollit works in devstack23:47
jrolland much faster too23:47
JayF+2'd23:48
jrollthanks23:51
JoshNangand +A23:52
JoshNangwooo stability23:52
*** dkehn_ is now known as dkehnx23:53

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