Friday, 2021-06-04

opendevreviewEric Barrera proposed x/sushy-oem-idrac master: Use TOX_CONSTRAINTS_FILE  https://review.opendev.org/c/x/sushy-oem-idrac/+/79468901:19
opendevreviewlikui proposed openstack/python-ironicclient master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/python-ironicclient/+/79469001:20
opendevreviewlikui proposed openstack/python-ironicclient master: These should be indented two spaces to match the other things in this block  https://review.opendev.org/c/openstack/python-ironicclient/+/79469101:23
*** benj_ has quit IRC01:37
opendevreviewlikui proposed openstack/ironic-prometheus-exporter master: Replace deprecated UPPER_CONSTRAINTS_FILE variable  https://review.opendev.org/c/openstack/ironic-prometheus-exporter/+/79469601:49
*** benj_ has joined #openstack-ironic02:00
TheJuliastevebaker: looks like actual raw retrieval is something like 20% faster, but enumeration through all of the objects is only about 10% faster over master branch at present. I think it is all of the extra self imposed object conversions we have, but I'll have to time some of it tomorrow.02:05
*** jelabarre-rh has quit IRC02:06
TheJuliaThe extra conversion to a raw "TraitsList" because our object model enforces it makes it difficult unless we can remove that hurdle :\... that conversions feels stupidly expensive02:06
*** zaneb has quit IRC02:08
stevebakerTraitsList seems so heavy-weight for what it does02:08
opendevreviewEric Barrera proposed x/sushy-oem-idrac master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/x/sushy-oem-idrac/+/79469802:24
TheJuliaYeah, I *really* don't understand why we even have it wired in like it is02:24
TheJuliaAnd even if we were to do so something like independent updates of placement, it would still be more efficient to poll/post it directly out of the DB02:25
* TheJulia goes and folds cloths02:26
*** tzumainn has quit IRC02:35
*** stendulker has joined #openstack-ironic04:39
opendevreviewVerification of a change to openstack/ironic failed: Secure RBAC - Efficent node santiziation  https://review.opendev.org/c/openstack/ironic/+/79392905:24
arne_wiebalckGood morning, ironic!05:50
opendevreviewMerged openstack/bifrost stable/wallaby: Install DIB dependencies in bifrost-ironic-install role  https://review.opendev.org/c/openstack/bifrost/+/79417606:11
*** stendulker has quit IRC06:21
*** stendulker has joined #openstack-ironic06:33
iurygregorygood morning Ironic o/06:56
arne_wiebalckhey iurygregory o/07:01
iurygregoryarne_wiebalck, o/07:01
*** rpittau|afk is now known as rpittau07:05
rpittaugood morning ironic! o/07:05
rpittauhappy Friday!07:05
arne_wiebalckrpittau: good morning o/07:05
iurygregorymorning rpittau o/07:06
rpittauhey arne_wiebalck :)07:06
rpittauhey iurygregory :)07:06
arne_wiebalckRelease numbering question: for the inspector, will 10.7 be still Wallaby or Xena ... or is that unclear at this stage?07:07
rpittauinteresting, it seems the latest ubnuntu 20.04 has dependencies issues when installing python3-dev07:07
* arne_wiebalck is confused by numbers07:07
rpittauarne_wiebalck: you're not alone!07:07
arne_wiebalckrpittau: that helps! :)07:07
rpittauarne_wiebalck: 10.7 is xena :)07:07
arne_wiebalckrpittau: ty!07:08
iurygregorywe won't have a bugfix for inspector this time because we don't have a lot of commits =)07:10
rpittauiurygregory: that's fine, as far as we have one mid july ;)07:10
iurygregoryrpittau, ++07:11
rpittauif anyone has a minute please check these 2 fixes for stable branches in bifrost https://review.opendev.org/c/openstack/bifrost/+/794590 https://review.opendev.org/c/openstack/bifrost/+/79459407:11
iurygregoryI will check in a few07:12
opendevreviewArne Wiebalck proposed openstack/ironic-inspector master: Add versions to Wallaby release notes  https://review.opendev.org/c/openstack/ironic-inspector/+/79471507:12
iurygregoryneed more coffee :D07:12
rpittauarne_wiebalck: you were faster this time :)07:12
arne_wiebalckrpittau: heh07:12
iurygregoryrpittau, I liked the walalbyu in the commit message :D07:18
rpittaummm ? One of my countless typos for sure!07:18
rpittauoh yeah07:19
iurygregorydon't worry :D07:19
*** tosky has joined #openstack-ironic07:23
opendevreviewMerged openstack/ironic-inspector master: Add versions to Wallaby release notes  https://review.opendev.org/c/openstack/ironic-inspector/+/79471507:32
*** lucasagomes has joined #openstack-ironic08:04
yoctozeptomorning ironic08:04
iurygregorymorning yoctozepto08:04
yoctozeptomorning iurygregory08:04
yoctozeptobom dia08:04
iurygregory:D yeah08:04
yoctozeptogot a quick question on how to debug something08:05
yoctozeptoI have lldp in my network08:05
yoctozeptobut ironic has only "lldp: null" on inspected nodes08:05
yoctozeptoI assume something has failed08:05
yoctozeptowhat's the best way to find out>08:06
yoctozepto(^ that lldp on ports of nodes, not nodes directly obviously)08:06
iurygregoryyoctozepto, can you check the plugins that are enabled?08:07
iurygregoryfrom our docs I see "To enable LLDP in the inventory from IPA ipa-collect-lldp=1 should be passed as a kernel parameter to the IPA ramdisk. In order to avoid processing the raw LLDP data twice, the lldp_basic plugin should also be installed and run prior to this plugin."08:08
yoctozeptoah, I assumed it would be on by default08:08
yoctozeptojust using bifrost so it's obviously not08:09
yoctozeptono lldp hits08:09
yoctozeptook, I started on this from the wrong angle08:09
yoctozeptothanks iurygregory for resetting me08:09
iurygregoryyoctozepto, np!08:09
*** lucasagomes has quit IRC08:19
*** mjturek has quit IRC08:25
*** lucasagomes has joined #openstack-ironic08:26
opendevreviewMerged openstack/bifrost stable/wallaby: Fix upper-constraints reference  https://review.opendev.org/c/openstack/bifrost/+/79459008:28
opendevreviewMerged openstack/bifrost stable/victoria: Recognize TOX_CONSTRAINTS_FILE as a valid constraints variable  https://review.opendev.org/c/openstack/bifrost/+/79459408:28
*** lucasagomes has quit IRC08:48
*** lucasagomes has joined #openstack-ironic08:49
*** stendulker has quit IRC09:00
*** lucasagomes has quit IRC09:05
*** stendulker has joined #openstack-ironic09:21
*** lucasagomes has joined #openstack-ironic09:25
jandersgood morning iurygregory arne_wiebalck rpittau yoctozepto and Ironic o/09:29
rpittauhey janders :)09:29
arne_wiebalckhey janders o/09:29
yoctozeptomorning janders09:33
*** lucasagomes has quit IRC09:37
*** lucasagomes has joined #openstack-ironic09:43
iurygregoryhey janders o/09:51
*** lucasagomes has quit IRC09:54
*** lucasagomes has joined #openstack-ironic09:59
*** stendulker_ has joined #openstack-ironic10:25
*** stendulker has quit IRC10:30
opendevreviewMerged openstack/ironic stable/wallaby: Secure RBAC - Efficent node santiziation  https://review.opendev.org/c/openstack/ironic/+/79392910:45
*** lucasagomes has quit IRC10:53
*** lucasagomes has joined #openstack-ironic10:58
*** derekh has joined #openstack-ironic11:05
janderssee you tomorrow Ironic11:30
jandershave a great weekend everyone! o/11:30
yoctozeptobye janders11:30
yoctozeptohave a great one too11:31
arne_wiebalckbye janders o/11:32
*** osmanlicilegi has quit IRC11:38
*** stendulker_ has quit IRC11:45
opendevreviewRadosław Piliszek proposed openstack/ironic-inspector master: [doc] Improvements in processing_hooks docs  https://review.opendev.org/c/openstack/ironic-inspector/+/79478111:48
opendevreviewRadosław Piliszek proposed openstack/python-ironic-inspector-client master: [doc] Fix lldp_basic name  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/79478211:54
*** osmanlicilegi has joined #openstack-ironic12:00
*** osmanlicilegi has quit IRC12:11
*** jelabarre-rh has joined #openstack-ironic12:15
*** osmanlicilegi has joined #openstack-ironic12:35
*** lucasagomes has quit IRC12:38
*** lmcgann has joined #openstack-ironic12:39
*** osmanlicilegi has quit IRC12:46
*** zaneb has joined #openstack-ironic12:48
*** osmanlicilegi has joined #openstack-ironic12:55
*** lucasagomes has joined #openstack-ironic12:56
*** rloo has joined #openstack-ironic13:01
*** paras333 has joined #openstack-ironic13:01
opendevreviewMerged openstack/ironic-inspector master: [doc] Improvements in processing_hooks docs  https://review.opendev.org/c/openstack/ironic-inspector/+/79478113:21
yoctozeptodo you usually backport docs improvements?13:28
rpittauyoctozepto: depends, if it's a correction that applies to stable branches, yes13:36
rpittaubye everyone, have a great weekend! o/13:37
yoctozeptorpittau: ok, I believe mine is13:37
*** rpittau is now known as rpittau|afk13:37
yoctozeptorpittau|afk: bye, enjoy13:37
opendevreviewRadosław Piliszek proposed openstack/ironic-inspector stable/wallaby: [doc] Improvements in processing_hooks docs  https://review.opendev.org/c/openstack/ironic-inspector/+/79481313:38
TheJuliagood morning13:41
arne_wiebalckGood morning, TheJulia o/13:41
*** tzumainn has joined #openstack-ironic13:41
iurygregorygood morning TheJulia =)13:41
* TheJulia is very slowly waking up today13:59
*** kkillsfirst has joined #openstack-ironic14:04
*** lucasagomes has quit IRC14:18
*** lucasagomes has joined #openstack-ironic14:21
*** lucasagomes has quit IRC14:34
*** lucasagomes has joined #openstack-ironic14:35
*** sdanni has quit IRC14:37
*** sdanni has joined #openstack-ironic14:38
*** paras333 has quit IRC14:38
*** paras333 has joined #openstack-ironic14:38
*** gmann is now known as gmann_afk14:40
*** lucasagomes has quit IRC14:46
*** lucasagomes has joined #openstack-ironic14:50
opendevreviewRadosław Piliszek proposed openstack/python-ironic-inspector-client master: Expose more interface fields  https://review.opendev.org/c/openstack/python-ironic-inspector-client/+/79484314:54
*** stendulker has joined #openstack-ironic15:03
iurygregoryTheJulia, fyi today is international corgi day :D15:06
TheJuliaSWEET!15:06
yoctozeptohttps://twitter.com/intcorgiday15:06
iurygregorywait15:06
iurygregorythere is a twitter? =O15:07
iurygregoryomg15:07
* yoctozepto experienced cuteness overload15:07
iurygregoryit's expected :D15:08
yoctozepto:D15:09
*** gmann_afk is now known as gmann15:13
opendevreviewJulia Kreger proposed openstack/ironic master: Only return the requested fields  https://review.opendev.org/c/openstack/ironic/+/79227415:43
opendevreviewJulia Kreger proposed openstack/ironic master: Set stage for objects to handle selected field lists.  https://review.opendev.org/c/openstack/ironic/+/79227515:43
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: API to pass fields to node object list  https://review.opendev.org/c/openstack/ironic/+/79229615:43
*** lucasagomes has quit IRC15:45
*** lucasagomes has joined #openstack-ironic15:48
*** paras333 has quit IRC15:59
*** paras333 has joined #openstack-ironic16:00
*** lucasagomes has quit IRC16:04
*** stendulker has quit IRC16:11
arne_wiebalckbye everyone, have a good weekend o/16:15
yoctozeptobye arne_wiebalck, same to you16:16
arne_wiebalck:)16:17
NobodyCamGood Morning Ironic folks.. and ofc...16:18
NobodyCamTGIF16:19
TheJuliaspuc in a half hour?16:26
JayFI intend on being there16:27
*** jelabarre-rh has quit IRC16:28
TheJulia\o/16:31
*** derekh has quit IRC16:34
opendevreviewMerged openstack/ironic-python-agent master: Refactor: use convert_image from ironic_lib  https://review.opendev.org/c/openstack/ironic-python-agent/+/79395516:35
*** jlvillal has quit IRC16:48
trandlesSorry to leave spuc without actually saying goodbye. :(17:43
JayFo/ have a good one17:44
*** kkillsfirst has quit IRC18:28
TheJuliac'est la vie :)18:30
opendevreviewJulia Kreger proposed openstack/ironic master: Only return the requested fields  https://review.opendev.org/c/openstack/ironic/+/79227418:59
opendevreviewJulia Kreger proposed openstack/ironic master: Set stage for objects to handle selected field lists.  https://review.opendev.org/c/openstack/ironic/+/79227518:59
* TheJulia pokes gerrit...19:00
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: API to pass fields to node object list  https://review.opendev.org/c/openstack/ironic/+/79229619:00
opendevreviewJulia Kreger proposed openstack/ironic master: WIP: Allow node_sanitize function to be provided overrides  https://review.opendev.org/c/openstack/ironic/+/79488019:00
TheJuliaJayF: 513 nodes/second ^19:07
*** kkillsfirst has joined #openstack-ironic19:08
JayFvery nice19:09
* TheJulia wonders what that would make the nova sync look like19:14
TheJuliaNobodyCam: so.. about publishing directly to placement...19:14
* TheJulia whistles innocently19:14
NobodyCamI have some icky code That is working in our staging env here...19:15
NobodyCamI can try and get a patch up next week19:17
TheJuliaNobodyCam: okay, I think if we can do that, plus some of the perf fixes, we can really improve the experience for operators such as yourself19:27
opendevreviewJulia Kreger proposed openstack/ironic master: Allow node_sanitize function to be provided overrides  https://review.opendev.org/c/openstack/ironic/+/79488020:26
TheJuliaI feel like we have nowhere near enough unit testing wrapping around the intermediate sanitization/record pagination, but... we check end results so maybe I shouldn't worry20:28
opendevreviewJulia Kreger proposed openstack/ironic master: API to pass fields to node object list  https://review.opendev.org/c/openstack/ironic/+/79229620:39
opendevreviewJulia Kreger proposed openstack/ironic master: Allow node_sanitize function to be provided overrides  https://review.opendev.org/c/openstack/ironic/+/79488020:52
*** paras333 has quit IRC21:07
*** sdanni has quit IRC21:11
*** paras333 has joined #openstack-ironic21:12
*** paras333 has quit IRC21:20
*** lmcgann has quit IRC21:25
*** gmann is now known as gmann_afk21:51
*** tosky has quit IRC22:01
*** rloo has quit IRC22:11
*** zaneb has quit IRC23:25
*** kkillsfirst has quit IRC23:55

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!