Tuesday, 2022-03-29

*** seebaer is now known as seba00:14
rpittaugood morning ironic !o/06:02
arne_wiebalckGood morning rpittau and Ironic!06:13
rpittauhey arne_wiebalck :)06:13
iurygregorygood morning Ironic o/10:03
opendevreviewMark Goddard proposed openstack/tenks master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/tenks/+/83560710:30
ameya49Hi iurygregory!10:46
iurygregoryHi o/10:46
ameya49I was running test_introspection_basic test and had few observations which I would like to discuss10:47
ameya49In test_introspection_basic test : https://github.com/openstack/ironic-tempest-plugin/blob/d3360cf3b6ad8b89b9c80fc806dc5d4ba373dd01/ironic_tempest_plugin/tests/scenario/test_introspection_basic.py#L106 onwards where its creating an instance10:48
ameya49As I understand here, after successful introspection, it uses the introspection data to create a server 10:49
ameya49one of the observation here is, the nova quota-class default values are less than introspected data (in my baremetal scenario). As a workaround I set it equal or greater to the introspected data manually.10:50
ameya49after which it goes for creating an instance, but for me its failing with an Error of exceeding maximum retries10:51
ameya49I did check few logs where its failing while deploying instance and timeout while going in active state.10:52
ameya49Any pointers which I can look for to boot the instance successfully? 10:54
ameya49is the creation of server required for introspection test case? or can we have a provision of skipping particular block?10:55
ameya49iurygregory : Let me know if any more detail information/logs are needed for the same. Thanks! :)11:05
iurygregorylet me see11:08
ameya49iurygregory : sure. Thanks 11:09
opendevreviewMark Goddard proposed openstack/tenks master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/tenks/+/83560711:37
iurygregoryameya49, so the test_baremetal_introspection was designed with this workflow (https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/tests/scenario/test_introspection_basic.py#L54-L66 ) I checked the default config values we use  in the wait_for_nova_aware_of_bvms function and I think they are enough https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_te12:00
iurygregorympest_plugin/config.py#L220-L227 ofc you can try playing with this values to see if it would help12:00
iurygregorythe right url for the default values https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/config.py#L220-L227 * since the irc broke the long text :D12:01
ameya49iurygregory : Thanks, will go through above links to understand further :)12:03
iurygregorygood luck12:04
ameya49iurygregory : https://opendev.org/openstack/ironic-tempest-plugin/src/branch/master/ironic_tempest_plugin/config.py#L220-L227. As I understand these are timeouts which I can try by increasing for deployment of node.13:11
ameya49But then is the nova qouta-class default values are expected to set manually (as per the introspected data from baremetal)?13:12
ameya49we get nova quota-class exceeded limit with default nova quota-class values against introspected values (for ram and cores)13:20
iurygregoryhummmm13:22
iurygregoryameya49, do you have the traceback of the nova quota class exceeded?13:24
ameya49let me check else will generate one and provide.13:26
TheJuliagood morning13:29
iurygregorygood morning TheJulia, feeling better today?13:30
TheJuliaa lot better than yesterday, not 100%13:31
iurygregorytake it slow today =)13:32
TheJuliayeah, I need to schedule some meeting and I've got the anaconda stuff on my plate13:36
TheJuliaoh where oh where was I at writing unit tests13:46
TheJuliahave we been tweeting the survey link arne_wiebalck sent out?14:10
iurygregoryI did https://twitter.com/ArneWiebalck/status/1508108292383555585?cxt=HHwWgoC-pY2g8O0pAAAA 14:15
opendevreviewMerged openstack/tenks master: Use jinja2.pass_context instead of contextfilter  https://review.opendev.org/c/openstack/tenks/+/83560714:18
ameya49iurygregory : Do not have traceback handy as of now. Will reproduce it and provide by tomorrow. Thanks14:20
iurygregoryameya49, ack no worries, I'm checking if I can find things related to nova quota in tempest (to see if we can update or something)14:21
ameya49iurygregory : https://opendev.org/openstack/tempest/src/branch/master/tempest/api/compute/admin/test_quotas.py (test_update_all_quota_resources_for_tenant). I got this while going through few tests14:30
iurygregoryty14:50
rpittaugood night! o/16:07
opendevreviewNisha Agarwal proposed openstack/ironic master: Adds create_csr and add_https_certificate clean step  https://review.opendev.org/c/openstack/ironic/+/80448616:26
Nisha_stevebaker[m], Hi16:29
Nisha_iurygregory, when is ironic planned for release for Yoga release?16:40
iurygregoryNisha_, we already did the release 5 days ago16:43
iurygregorymaster now is for Zed 16:43
iurygregoryTheJulia, do you think we will need 3hrs to talk about the survey results? 16:47
TheJuliaeh, maybe 2-ish16:48
iurygregoryyeah, I was thinking this16:49
* arne_wiebalck checks answers ...16:52
* arne_wiebalck will not spoil things :-D16:53
iurygregoryPossible PTG schedule https://paste.opendev.org/show/bosAputU90U8RNKUbvKd/16:53
iurygregoryI'm going to send to the openstack-discuss and adjust according to feedback16:53
TheJuliaThat seems reasonable to me16:54
iurygregoryif we don't have enough time to review the specs I think we can re-activate review jams during april and focus on that16:54
iurygregoryin the week after the PTG16:55
arne_wiebalckiurygregory: TheJulia: I can share the results of the survey before, so that everyone can have a look at the data and we do not lose time at the PTG to scan it ... sounds ok?16:59
iurygregoryarne_wiebalck, ++16:59
TheJulia++16:59
arne_wiebalckI suggest to keep it open until the weekend or so, so please make noise about it so that we get some significant data set to work with :)17:00
TheJulia++17:01
iurygregoryyeah I agree17:03
iurygregorywe didn't switch metalsmith to CS9? '-'17:04
TheJuliaI don't remember17:09
TheJuliabut sounds like we missed it17:09
iurygregoryyeah, I'm double checking because I saw https://zuul.opendev.org/t/openstack/build/891809377d3b499a9d077b46bc26ba2c17:11
* TheJulia feels like a zombie17:18
iurygregoryoh ok, for metalsmith we switched from CentOS8 to Centos 8 Stream18:39
TheJuliawe should take that to stream9 then18:57
iurygregoryyeah, I'm woking on it now19:08
TheJuliaiurygregory: thanks!19:16
opendevreviewIury Gregory Melo Ferreira proposed openstack/metalsmith master: Move from CS8 to CS9  https://review.opendev.org/c/openstack/metalsmith/+/83574919:23
opendevreviewIury Gregory Melo Ferreira proposed openstack/ironic master: Use metalsmith cs9 job  https://review.opendev.org/c/openstack/ironic/+/83575019:26
iurygregoryDrop all CS8 jobs! \o/19:28
opendevreviewVerification of a change to openstack/ironic bugfix/19.0 failed: Fix rebuilds using anaconda deploy interface  https://review.opendev.org/c/openstack/ironic/+/83373520:21
opendevreviewHarald JensÃ¥s proposed openstack/networking-baremetal master: Add OpenConfig classes for switch vlans  https://review.opendev.org/c/openstack/networking-baremetal/+/83515821:41
opendevreviewMerged openstack/ironic bugfix/19.0: Fix rebuilds using anaconda deploy interface  https://review.opendev.org/c/openstack/ironic/+/83373523:29

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