Friday, 2019-03-29

*** ianychoi has quit IRC00:17
*** ianychoi has joined #openstack-ironic00:18
*** sthussey has quit IRC00:20
*** hamzy has joined #openstack-ironic00:44
*** jistr has quit IRC01:00
*** jistr has joined #openstack-ironic01:00
*** yolanda has quit IRC01:07
*** yolanda has joined #openstack-ironic01:15
*** whoami-rajat has joined #openstack-ironic01:19
*** rloo has quit IRC01:26
*** andrein has quit IRC03:03
*** jaypipes_ has quit IRC04:02
*** jaypipes_ has joined #openstack-ironic04:02
*** openstackgerrit has joined #openstack-ironic04:27
openstackgerritUros Orozel proposed openstack/ironic-lib master: Include partiton name and flags from parted output  https://review.openstack.org/64860604:27
openstackgerritUros Orozel proposed openstack/ironic master: UEFI iPXE provisioning fails when ipxe_use_swift is enabled  https://review.openstack.org/64807405:01
*** early` has quit IRC05:06
*** early has joined #openstack-ironic05:09
*** e0ne has joined #openstack-ironic06:06
openstackgerritUros Orozel proposed openstack/ironic-lib master: Include partiton name and flags from parted output  https://review.openstack.org/64860606:07
*** e0ne has quit IRC06:10
*** rcernin has quit IRC06:38
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/rocky: Imported Translations from Zanata  https://review.openstack.org/64862307:14
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/64862407:15
openstackgerritOpenStack Proposal Bot proposed openstack/ironic-ui stable/queens: Imported Translations from Zanata  https://review.openstack.org/64862507:20
arne_wiebalckgood morning, ironic!07:29
*** e0ne has joined #openstack-ironic07:32
openstackgerritfrancotseng proposed openstack/ironic-ui master: Fix misused html attribute 'translation' in base-node.html  https://review.openstack.org/64862907:33
rpittau|afkgood morning ironic! o/07:39
*** rpittau|afk is now known as rpittau07:39
rpittauhappy friday!07:39
*** andrein has joined #openstack-ironic08:00
arne_wiebalckgood morning rpittau o/08:01
rpittauhey arne_wiebalck :)08:02
etingofTGIF, no irony!08:09
*** S4ren has joined #openstack-ironic08:20
*** early has quit IRC08:20
rpittauetingof :)08:21
*** rcernin has joined #openstack-ironic08:22
*** dsneddon has joined #openstack-ironic08:27
*** early has joined #openstack-ironic08:29
iurygregorymorning all08:30
*** e0ne has quit IRC08:31
*** dsneddon has quit IRC08:32
*** tssurya has joined #openstack-ironic08:57
*** rcernin has quit IRC08:57
*** e0ne has joined #openstack-ironic09:00
*** lucasagomes has joined #openstack-ironic09:02
*** dsneddon has joined #openstack-ironic09:02
*** lucasagomes has quit IRC09:03
*** lucasagomes has joined #openstack-ironic09:03
*** dsneddon has quit IRC09:08
openstackgerritUros Orozel proposed openstack/ironic master: UEFI iPXE provisioning fails when ipxe_use_swift is enabled  https://review.openstack.org/64807409:11
*** pcaruana has joined #openstack-ironic09:15
*** andrein has quit IRC09:21
*** andrein has joined #openstack-ironic09:23
*** dsneddon has joined #openstack-ironic09:27
*** priteau has joined #openstack-ironic09:33
*** dsneddon has quit IRC09:33
*** stendulker has joined #openstack-ironic09:33
openstackgerritAkihiro Motoki proposed openstack/ironic-ui master: i18n: Extract strings from template files  https://review.openstack.org/64865409:41
*** rachit7 has joined #openstack-ironic09:41
*** dsneddon has joined #openstack-ironic09:42
*** dtantsur|afk is now known as dtantsur09:43
*** monica_pardhi_ has joined #openstack-ironic09:46
*** dsneddon has quit IRC09:47
*** andrein has quit IRC09:48
*** e0ne has quit IRC09:49
openstackgerritNikolay Fedotov proposed openstack/ironic-python-agent master: Discover IPv6 BMC address  https://review.openstack.org/64079509:53
*** e0ne has joined #openstack-ironic09:54
dtantsureandersson, TheJulia, jroll, a potential proposal: https://storyboard.openstack.org/#!/story/200532809:57
*** dsneddon has joined #openstack-ironic10:00
*** diga has joined #openstack-ironic10:01
openstackgerritDmitry Tantsur proposed openstack/ironic-tempest-plugin master: Allocation API: add tests on backfilling allocations  https://review.openstack.org/64840310:03
monica_pardhi_dtantsur: Hi , can you please approve my affiliation patch? here is patch id https://review.openstack.org/#/c/64476610:04
patchbotpatch 644766 - stackalytics - Add my affiliation to Dell Emc - 2 patch sets10:04
*** dsneddon has quit IRC10:05
dtantsurmonica_pardhi_: no, sorry, I cannot. I have neither core rights there, nor any relationship to Dell EMC.10:05
monica_pardhi_dtantsur: Ok. thanks for update.10:07
digaHello Ironic10:09
digaGood Morning10:09
rachit7Morning10:10
iurygregorymonica_pardhi_, you may want to ask in openstack-dev or openstack irc channels10:10
monica_pardhi_iurygregory: Thank you.10:12
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Create/delete configurations  https://review.openstack.org/63468110:14
openstackgerritArne Wiebalck proposed openstack/ironic-python-agent master: Software RAID: Add IPA deploy support  https://review.openstack.org/63939010:14
openstackgerritUros Orozel proposed openstack/ironic-python-agent master: Ironic python agent does not extract correct available memory  https://review.openstack.org/64809610:15
*** andrein has joined #openstack-ironic10:28
*** Lucas_Gray has joined #openstack-ironic10:31
iurygregorydtantsur, you have the best comments for code XD10:35
iurygregory (because life is pain)10:35
*** dsneddon has joined #openstack-ironic10:40
openstackgerritUros Orozel proposed openstack/ironic-lib master: Include partiton name and flags from parted output  https://review.openstack.org/64860610:47
*** Lucas_Gray has quit IRC10:48
*** Lucas_Gray has joined #openstack-ironic10:50
*** e0ne has quit IRC10:50
*** Lucas_Gray has quit IRC10:50
*** Lucas_Gray has joined #openstack-ironic10:51
rpittauif anyone has a minute, I think we should merge the workaround for the uefi job -> https://review.openstack.org/64768711:08
patchbotpatch 647687 - ironic - Workaround for uefi job with ubuntu bionic - 5 patch sets11:08
*** stendulker has quit IRC11:27
iurygregoryrpittau, magic11:39
*** S4ren has quit IRC11:40
rpittauiurygregory, broken packages :P11:40
iurygregoryyeah XD11:40
*** whoami-rajat has quit IRC11:42
*** e0ne has joined #openstack-ironic11:54
*** diga has quit IRC11:58
*** e0ne has quit IRC12:02
*** S4ren has joined #openstack-ironic12:11
*** monica_pardhi_ has quit IRC12:16
*** trown|outtypewww is now known as trown12:20
*** EmilienM is now known as EvilienM12:21
ijpascual3TheJulia I have seen your note regarding change attribute and CentOS in bifrost playbooks to create VM volumes. Have you experience:Q12:22
ijpascual3Have you experienced this failure?  "Operation not supported while setting flags"12:23
*** ijpascual3 is now known as ijpascual12:25
ijpascualhttps://github.com/openstack/bifrost/blob/master/playbooks/roles/bifrost-create-vm-nodes/tasks/create_vm.yml#L6612:27
ijpascualTheJulia: ^12:28
dtantsuriurygregory: :)12:28
dtantsurrpittau: have you seen https://bugs.launchpad.net/ubuntu/+source/edk2/+bug/1821729/comments/5 ?12:30
openstackLaunchpad bug 1821729 in edk2 (Ubuntu Cosmic) "UEFI in ovmf package causes kernel panic" [Undecided,Incomplete]12:30
*** jaypipes_ is now known as leakypipes12:30
*** S4ren has quit IRC12:31
rpittaudtantsur, yes, I saw it, I was testing it yesterday but I had a little PEBKAC issue and have to test it again :/12:35
jroll+w on that12:35
rpittauluckily realized before replying12:35
jrolland good morning everyone12:35
dtantsurmorning jroll12:35
arne_wiebalckjroll: o/12:35
rpittauhey jroll :)12:36
*** e0ne has joined #openstack-ironic12:36
jrolldtantsur: your proposal for network/storage seems fine to me. I don't think there's any compatibility issues for out of tree drivers, right?12:36
* rpittau goes for lunch before the keyboard starts to become appetizing12:37
dtantsurjroll: out of tree drivers currently have to define their own HardwareType and override supported_{iface}_interfaces, which makes us less likely to break them12:37
jrollright12:38
jrollwell, they can inherit it as well, but that's also fine in this case12:38
openstackgerritIlya Etingof proposed openstack/ironic master: Run IPMI, SNMP and Redfish BMC emulators as stack  https://review.openstack.org/64849612:38
jrollyep, still don't see anything. put a +1 there12:39
*** rloo has joined #openstack-ironic12:44
*** jtomasek has joined #openstack-ironic12:47
dtantsurjroll: I actually missed one thing there: what to do with the defaults calculation12:49
jrollmmm12:50
* jroll re-reads that code12:50
dtantsurwe rely on the ordering from supported_XXX_interfaces12:51
jrollright12:51
* dtantsur would say that the driver composition is a mess, but he knows who's to blame for that :D12:52
jrollme, right? :P12:52
jrollthat was fun code to write, at least12:52
dtantsurI think I'm more to blame :)12:53
dtantsurwe could probably create defaults for default_network_interface and default_storage_interface in the config12:53
jrollthat was one thought I had12:54
dtantsurwhich is kinda what is done with storage in https://github.com/openstack/ironic/blob/master/ironic/common/driver_factory.py#L135-L14212:54
jrollthat could work12:56
jrollactually, how that's done is what I was thinking12:56
jrollbut with a list as the value in additional_defaults12:56
jrollso that if e.g. neutron is disabled, we can fall back to the next default12:57
dtantsurI don't really like this additional_defaults thingy since it's buried in the code12:58
dtantsura completely different approach would be to have a special value like ANY12:58
jrollyeah, but not sure how you'd order that12:58
dtantsuradding it to supported_blah-blah would make compatibility check pass, but we'll still have the normal ordering12:59
dtantsurbonus: we won't have to special case FakeHardware in driver_factory12:59
* dtantsur should probably prototype that12:59
jrollactually, I think setting defaults for default_network/storage_interface would be fine12:59
jrollit'll blow up at startup if the interface is not enabled, and defaults would be sane for most deployments13:00
jrolland easy to switch if using something else13:00
jrollrpittau: dtantsur: do we sitll need this?13:01
jrollhttps://review.openstack.org/#/c/642015/13:01
patchbotpatch 642015 - ironic - Move all testing to Bionic - 2 patch sets13:01
dtantsurI think no13:01
jrollsame13:01
dtantsurMy concern with changing default_network_interface=flat is that some people may have enabled_network_interfaces=[neutron] and currently it just works13:01
jrollthanks13:01
jrollthis is true13:02
*** e0ne has quit IRC13:02
jrollbut since we would fail to start the conductor, it isn't a huge burden13:02
jrollespecially with release notes13:02
jrollor we could do it slower with warnings13:03
openstackgerritDmitry Tantsur proposed openstack/ironic master: Make 'noop' the explicit default of default_storage_interface  https://review.openstack.org/64867913:05
dtantsurrelated ^^13:05
dtantsurthis is not breaking since it's already the default13:05
* dtantsur prepares lunch while thinking about all this13:06
*** e0ne has joined #openstack-ironic13:11
*** e0ne has quit IRC13:13
*** S4ren has joined #openstack-ironic13:25
*** jtomasek has quit IRC13:34
*** jtomasek has joined #openstack-ironic13:34
*** jtomasek has quit IRC13:36
*** priteau has quit IRC13:37
*** whoami-rajat has joined #openstack-ironic13:38
TheJuliagood morning everyone13:41
*** e0ne has joined #openstack-ironic13:42
jrollmorning TheJulia13:42
TheJuliadtantsur: so with your proposal, someone would just be able to add it to load and add their magical special network driver to the list?13:43
TheJuliaijpascual: That was a REALLY long time ago....13:44
TheJuliaijpascual: We experienced it on the centos images we were operating on, I don't remember any details, but chattr +C cleared things up for bifrost. Why do you ask?13:45
*** S4ren has quit IRC13:45
TheJuliarpittau: desires to eat the keyboard are slightly alarming, perhaps a glass of wine #JuliaIsABadInfluence :)13:46
openstackgerritArne Wiebalck proposed openstack/ironic-specs master: Add details on target_raid_config for Software RAID support  https://review.openstack.org/64310013:47
*** bnemec is now known as beekneemech13:47
rpittauTheJulia, which wine goes well with keyboard? :)13:48
*** S4ren has joined #openstack-ironic13:49
TheJuliarpittau: A malbec I think13:49
* arne_wiebalck hears crunching noises13:49
*** rachit7 has quit IRC13:50
TheJuliajeeze, how many emails can I get in 12 hours13:50
*** baha has joined #openstack-ironic13:59
ijpascualTheJulia is because I am facing that error in Ubuntu for some reason. Although it does not stop the playbook and execution goes fine, I was wondering if it was an important step.14:01
TheJuliaijpascual: I think it was preventing virutal machines launching for testing, in what context are you seeing this error?14:02
dtantsurTheJulia: that's the goal. but see the last comment and the discussion above: we need a solution for the defaults calculation14:09
TheJuliadtantsur: Yeah, didn't quite get that far while I was scrolling through and typing14:09
TheJuliaIn that case, it does sort of make sense to just encourage the use of defaults overrides, then again they might be explicitly creating nodes with specific interfaces14:10
TheJuliaeandersson: Thoughts ^^^14:10
openstackgerritMerged openstack/ironic master: Workaround for uefi job with ubuntu bionic  https://review.openstack.org/64768714:11
dtantsurTheJulia: yeah, I'm worried about breaking people with enabled_network_interfaces=neutron or something like that14:12
TheJuliayeah, which is kind of why I'm thinking an operator point of view might be good. arne_wiebalck any thoughts as well? I realize you guys are likely running a more stock driver set though.14:20
* arne_wiebalck reads the backlog14:27
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Include all libvirt domains in Systems  https://review.openstack.org/64870014:27
openstackgerritIlya Etingof proposed openstack/ironic master: Run IPMI, SNMP and Redfish BMC emulators as stack  https://review.openstack.org/64849614:51
arne_wiebalckTheJulia: Sorry, got dragged into some other discussion.14:54
amotokihi, ironic folks. is there any chance to land https://review.openstack.org/#/c/648654/ in Stein? I had a report in #openstack-i18n channel that some strings in ironic-ui cannot be translated.14:55
patchbotpatch 648654 - ironic-ui - i18n: Extract strings from template files - 1 patch set14:55
*** jcoufal has joined #openstack-ironic14:57
*** e0ne has quit IRC14:58
arne_wiebalckTheJulia: dtantsur : I'm not sure what input to give ... is the question whether ironic moving to sensible defaults too much of a risk as it may break existing deployments?15:01
*** e0ne has joined #openstack-ironic15:07
*** e0ne has quit IRC15:12
NobodyCamGood Morning Ironic'ers15:21
NobodyCamand Ofc ..15:21
NobodyCamTGIF!15:21
dtantsurmorning NobodyCam :)15:22
NobodyCamhappy Friday dtantsur :)15:23
rpittauhey NobodyCam :)15:23
NobodyCamhey hey rpittau :)15:23
*** dhellmann_ has joined #openstack-ironic15:24
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Ensure non-empty libvirt <loader/> tag  https://review.openstack.org/64272815:24
*** andrein has quit IRC15:24
*** dmellado has quit IRC15:24
*** dhellmann has quit IRC15:24
*** dmellado has joined #openstack-ironic15:26
*** dhellmann_ is now known as dhellmann15:27
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Create libvirt boot loader element if needed  https://review.openstack.org/64258115:29
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on UEFI boot libvirt-based emulation  https://review.openstack.org/64283015:30
*** gmann is now known as gmann_afk15:32
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Rename `drivers` to `systems`  https://review.openstack.org/63849215:34
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers resource support  https://review.openstack.org/63868215:49
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add read-only Virtual Media support  https://review.openstack.org/63878715:50
openstackgerritMerged openstack/ironic-ui master: Fix misused html attribute 'translation' in base-node.html  https://review.openstack.org/64862915:51
openstackgerritMerged openstack/ironic-ui master: Imported Translations from Zanata  https://review.openstack.org/64862415:51
dtantsuramotoki: hi, it will need a backport, otherwise we're good with it15:51
openstackgerritMerged openstack/ironic master: Check for deploy.deploy deploy step in heartbeat  https://review.openstack.org/64748515:53
*** trown is now known as trown|lunch15:58
openstackgerritMerged openstack/ironic-ui master: i18n: Extract strings from template files  https://review.openstack.org/64865416:14
*** e0ne has joined #openstack-ironic16:14
*** e0ne has quit IRC16:14
iurygregoryyay bugs \o/16:19
*** baha has quit IRC16:25
TheJuliayeah, not terribly surprising given the level of sensor data use16:32
*** Lucas_Gray has quit IRC16:37
*** Lucas_Gray has joined #openstack-ironic16:38
iurygregoryhttps://storyboard.openstack.org/#!/story/200533216:40
iurygregorySensor data bug \o/16:40
openstackgerritJulia Kreger proposed openstack/ironic master: Make it possible to send sensor data for all nodes  https://review.openstack.org/64872816:40
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add writable Virtual Media resource mocks  https://review.openstack.org/63933816:40
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add Managers<->Systems linkage in sushy-emulator  https://review.openstack.org/64030416:40
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add libvirt-based virtual media emulation  https://review.openstack.org/64045216:43
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Render System->Managers linkage  https://review.openstack.org/64309416:43
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add docs on libvirt virtual media emulation  https://review.openstack.org/64302716:43
TheJuliaiurygregory: thanks16:43
iurygregoryTheJulia, yw o/16:43
TheJuliaI think the answer is to make the code more tollerant of unknown tag data16:43
iurygregorymakes sense to me16:43
iurygregoryprobably different versions of ipmi have different tags...16:44
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Add HTTP image download for virtual media emulation  https://review.openstack.org/64333016:44
openstackgerritIlya Etingof proposed openstack/sushy-tools master: Fix Flask config load under WSGI  https://review.openstack.org/64840716:45
TheJuliait almost looks like ipmitool is spitting out a bunch of stuff we're not expecting16:45
TheJuliaso I see what it is... I think16:48
TheJuliait is debug beign turned on16:48
*** lucasagomes has quit IRC16:49
*** Lucas_Gray has quit IRC16:50
iurygregoryoh16:51
TheJuliaThe exchange extra detail :)16:51
TheJulia<< and >>16:51
TheJuliaI think we append -v when we're in debug16:51
TheJuliaoh yes,  yes we do16:51
iurygregoryso if we dont we are fine?16:52
iurygregoryand the magic will happen? \o/16:52
TheJuliaiurygregory: can you add the link from the log to the story?16:55
TheJuliaI think so...16:55
TheJuliagot to wait 10 minutes :\16:55
iurygregoryTheJulia, now =)16:55
TheJuliain the mean time16:55
TheJuliathanks16:55
TheJuliaint he mean time, writing a unit test to help me fix this16:55
iurygregoryok o/16:56
rpittaugood night and have a great weekend! o/16:57
*** rpittau is now known as rpittau|afk16:57
*** tssurya has quit IRC16:57
TheJuliaso stdout has limited debug info16:59
iurygregorythats good =D17:01
* iurygregory be back later o/17:01
*** iurygregory has quit IRC17:02
*** dtantsur is now known as dtantsur|afk17:06
dtantsur|afkhave a great weekend17:06
openstackgerritIlya Etingof proposed openstack/ironic master: Run IPMI, SNMP and Redfish BMC emulators as stack  https://review.openstack.org/64849617:07
openstackgerritAkihiro Motoki proposed openstack/ironic-ui stable/stein: i18n: Extract strings from template files  https://review.openstack.org/64873617:08
*** sambetts_ has joined #openstack-ironic17:24
arne_wiebalckhave a good weekend o/17:35
TheJuliayou too!17:36
* TheJulia types commit message for Iury's bug17:36
openstackgerritJulia Kreger proposed openstack/ironic master: ipmi: Ignore sensor debug data  https://review.openstack.org/64874617:37
openstackgerritJulia Kreger proposed openstack/ironic master: ipmi: Ignore sensor debug data  https://review.openstack.org/64874617:37
*** baha has joined #openstack-ironic17:39
*** jistr is now known as jistr|off17:45
*** jistr|off is now known as jistr17:45
*** yolanda has quit IRC17:53
*** trown|lunch is now known as trown18:08
*** iurygregory has joined #openstack-ironic18:17
*** S4ren has quit IRC18:24
eanderssonTheJulia, sounds good to me18:30
eanderssonThanks!18:30
*** pcaruana has quit IRC18:38
*** MattMan_ has quit IRC19:08
*** MattMan_ has joined #openstack-ironic19:09
*** gmann_afk is now known as gmann19:19
*** S4ren has joined #openstack-ironic19:26
*** sthussey has joined #openstack-ironic19:33
*** S4ren has quit IRC19:35
*** yolanda has joined #openstack-ironic19:58
eanderssonAnyone seen this when trying to write unit-tests20:07
eandersson> DefaultValueError: Error processing default value on_failure for Opt type of String(choices=[('always', u'always collect the logs'), ('on_failure', u'only collect logs if there is a failure'), ('never', u'never collect logs')])20:07
eanderssonnvm think I figured it out20:09
eanderssonHad the wrong upper reqs set20:09
*** jtomasek has joined #openstack-ironic20:16
*** jtomasek has quit IRC20:26
*** baha has quit IRC20:42
*** iurygregory has quit IRC20:45
*** jcoufal has quit IRC20:58
*** trown is now known as trown|outtypewww21:00
openstackgerritMerged openstack/ironic master: ipmi: Ignore sensor debug data  https://review.openstack.org/64874621:22
TheJuliawell that was fast21:28
* TheJulia takes her cat to the vet21:28
zigoI'm getting some import error when listing unit tests when I am buliding the Debian package.21:45
zigo    from openstack.baremetal import configdrive as os_configdrive21:45
zigoImportError: cannot import name \'configdrive\' from \'openstack.baremetal\' (/usr/lib/python3/dist-packages/openstack/baremetal/__init__.py)21:45
zigoFrom what module is this comming from?21:45
zigoeandersson: ^21:46
eanderssonzigo, probably the wrong version of the openstacksdk21:58
zigoeandersson: Ah, thanks, I'll force a newer version in the package dependency then.21:58
eanderssonYou need at least 0.22.021:59
zigoOh, I got it wrong ... ;)21:59
zigoFYI, I'm nearly done with all of the Stein packaging in Debian.22:00
zigoHalf of the services are there...22:00
zigoWell, maybe half way, if we consider Horizon (big piece...).22:01
eanderssonAwesome!22:02
*** ijw has joined #openstack-ironic22:31
*** sthussey has quit IRC23:01
*** rloo has quit IRC23:33
*** rh-jelabarre has quit IRC23:34
*** whoami-rajat has quit IRC23:47

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