Wednesday, 2021-05-05

« Tuesday, 2021-05-04 Index Thursday, 2021-05-06 »
NobodyCamnow I wonder what ever happened to the "Poke the bunny" game.. it also was flash00:09
NobodyCamLOL Openstack needs to standardize common conf file settings00:10
*** tkajinam has joined #openstack-ironic01:13
*** pmannidi has quit IRC01:23
openstackgerritYuehuiLei proposed openstack/sushy-cli master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/sushy-cli/+/78969302:20
*** sdanni has quit IRC02:35
*** rcernin has quit IRC03:05
*** paras333_ has joined #openstack-ironic03:05
*** paras333_ has quit IRC03:10
*** mkrai has joined #openstack-ironic03:13
*** rcernin has joined #openstack-ironic03:16
*** tzumainn has quit IRC03:17
*** paras333_ has joined #openstack-ironic03:36
*** paras333_ has quit IRC03:40
*** ociuhandu has joined #openstack-ironic03:49
*** ociuhandu has quit IRC03:54
*** mkrai has quit IRC05:37
*** pmannidi has joined #openstack-ironic05:43
*** pmannidi has quit IRC05:47
*** mkrai has joined #openstack-ironic05:57
*** pmannidi has joined #openstack-ironic06:05
*** dhellmann has quit IRC06:10
*** uzumaki has quit IRC06:15
iurygregorygood morning Ironic o/06:21
*** vmud213 has joined #openstack-ironic06:28
*** gyee has quit IRC06:32
arne_wiebalckGood morning iurygregory and Ironic!06:33
iurygregoryarne_wiebalck, o/06:44
*** mkrai has quit IRC06:50
*** ociuhandu has joined #openstack-ironic07:10
*** rpittau|afk is now known as rpittau07:12
rpittaugood morning ironic! o/07:12
*** rcernin has quit IRC07:21
*** rcernin has joined #openstack-ironic07:25
*** tosky has joined #openstack-ironic07:30
*** paras333_ has joined #openstack-ironic07:38
*** paras333_ has quit IRC07:42
openstackgerritMerged openstack/ironic-tempest-plugin master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/78952507:46
iurygregorymorning rpittau o/07:49
openstackgerritMerged openstack/networking-baremetal master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/networking-baremetal/+/78951607:49
rpittauhey iurygregory :)07:50
openstackgerritMerged openstack/networking-generic-switch master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/networking-generic-switch/+/78951407:50
*** ociuhandu has quit IRC07:54
*** ociuhandu has joined #openstack-ironic07:56
jandersquick question on behalf of a customer: is it possible to use PXE (not iPXE) with EFI? If not, what is the reason?07:58
janders(context: power8 can support iPXE or EFI but not both at the same time for some power-y reason)07:58
*** rcernin has quit IRC07:59
*** lucasagomes has joined #openstack-ironic07:59
*** ociuhandu has quit IRC08:00
*** derekh has joined #openstack-ironic08:00
openstackgerritMerged openstack/ironic-inspector master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/ironic-inspector/+/78953308:03
*** ociuhandu has joined #openstack-ironic08:12
*** ianychoi_ has quit IRC08:13
*** rcernin has joined #openstack-ironic08:23
*** rcernin has quit IRC08:29
*** rcernin has joined #openstack-ironic08:38
*** rcernin has quit IRC08:42
*** rcernin has joined #openstack-ironic08:46
*** derekh has quit IRC09:05
*** mkrai has joined #openstack-ironic09:08
*** dtantsur|afk is now known as dtantsur09:21
dtantsurmorning ironic09:21
dtantsurjanders: yes, PXE can be used with UEFI, we have docs for that09:21
iurygregorymorning dtantsur o/09:23
*** rcernin has quit IRC09:30
*** paras333_ has joined #openstack-ironic09:39
*** paras333_ has quit IRC09:44
*** derekh has joined #openstack-ironic09:53
*** rcernin has joined #openstack-ironic09:54
*** rcernin has quit IRC10:05
*** k_mouza has joined #openstack-ironic10:09
*** uzumaki has joined #openstack-ironic10:14
*** sshnaidm|afk is now known as sshnaidm10:34
openstackgerritBob Fournier proposed openstack/ironic master: Retrieve BIOS registry and include fields in bios API  https://review.opendev.org/c/openstack/ironic/+/78670710:36
*** tosky has quit IRC10:38
*** tosky has joined #openstack-ironic10:38
*** tosky_ has joined #openstack-ironic10:45
*** tosky has quit IRC10:47
*** tosky_ is now known as tosky10:49
*** k_mouza has quit IRC10:51
*** rcernin has joined #openstack-ironic10:52
*** mkrai has quit IRC10:56
*** rcernin has quit IRC10:58
*** rcernin has joined #openstack-ironic11:02
*** k_mouza has joined #openstack-ironic11:03
*** ociuhandu has quit IRC11:06
*** rcernin has quit IRC11:06
*** uzumaki has quit IRC11:08
*** mkrai has joined #openstack-ironic11:14
openstackgerritMerged openstack/ironic-specs master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/ironic-specs/+/78871711:20
*** ociuhandu has joined #openstack-ironic11:22
*** rcernin has joined #openstack-ironic11:24
*** ociuhandu has quit IRC11:27
*** mkrai has quit IRC11:27
*** rcernin has quit IRC11:28
dtantsurTheJulia: we need an RSS feed for https://ironicbaremetal.org/blog/11:37
* dtantsur just tried adding it to feedly11:37
*** ociuhandu has joined #openstack-ironic11:38
dtantsurTheJulia: and add https://owlet.today/posts/ephemeral-workloads-with-ironic/ (ramdisk deploy) maybe?11:39
dtantsurand https://owlet.today/posts/deploy-steps-tutorial/ (I can propose both if you're okay)11:40
jandersthank you dtantsur. w/r/t PXE & EFI, I will have further question in the internal channel if that's OK11:42
*** ociuhandu has quit IRC11:42
*** derekh has quit IRC11:45
*** rcernin has joined #openstack-ironic11:45
*** ociuhandu has joined #openstack-ironic11:54
*** rcernin has quit IRC11:55
*** ociuhandu has quit IRC11:59
*** lmcgann has joined #openstack-ironic12:00
*** ociuhandu has joined #openstack-ironic12:11
openstackgerritRiccardo Pittau proposed openstack/ironic master: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78873412:14
*** rcernin has joined #openstack-ironic12:18
*** ociuhandu has quit IRC12:18
openstackgerritMerged openstack/ironic master: Inherit InvalidImageRef from InvalidParameterValue  https://review.opendev.org/c/openstack/ironic/+/78959312:21
*** rcernin has quit IRC12:23
*** rh-jelabarre has joined #openstack-ironic12:29
*** ociuhandu has joined #openstack-ironic12:33
openstackgerritDmitry Tantsur proposed openstack/ironic stable/wallaby: Inherit InvalidImageRef from InvalidParameterValue  https://review.opendev.org/c/openstack/ironic/+/78980512:37
openstackgerritDmitry Tantsur proposed openstack/ironic stable/victoria: Inherit InvalidImageRef from InvalidParameterValue  https://review.opendev.org/c/openstack/ironic/+/78980612:37
*** ociuhandu has quit IRC12:44
*** tzumainn has joined #openstack-ironic12:55
*** paras333_ has joined #openstack-ironic13:01
TheJuliagood morning13:02
TheJuliadtantsur: I've got a call with Wes later... I think. I can ask.13:03
*** rloo has joined #openstack-ironic13:04
TheJuliadtantsur: do you want to propose that as a blog update, or do you want me to? Also I was thinking writing a proper entry about performance, in terms of the eye opening experience.13:04
iurygregorygood morning TheJulia =)13:04
dtantsurTheJulia: I was thinking of adding links to these two items, similar to links to CERN blog13:04
dtantsurand good morning13:05
TheJuliadtantsur: oh I see, I've not had coffee hit my blood stream yet13:05
TheJuliaYeah, add both13:05
TheJuliaand I'll try to put up some local content on it very soon13:06
dtantsurokay, will get to it in the next KubeCon break13:06
TheJuliaerr, no call with wes later today13:07
TheJuliagrr13:07
dtantsurcan we an email ;)13:07
dtantsurs/we/be/13:07
TheJuliaEh, yeah. Jimmy may also know but looks like maybe I'll talk to him tomorrow13:08
*** derekh has joined #openstack-ironic13:10
openstackgerritMerged openstack/sushy-cli master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/sushy-cli/+/78969313:11
openstackgerritMerged openstack/ironic master: Do not mask configdrive when executing in-band deploy steps  https://review.opendev.org/c/openstack/ironic/+/78871813:12
openstackgerritMerged openstack/ironic master: Update refarch with information about image_download_source  https://review.opendev.org/c/openstack/ironic/+/78937713:12
openstackgerritMerged openstack/ironic master: Bye-bye iSCSI deploy, you served us well  https://review.opendev.org/c/openstack/ironic/+/78938213:12
dtantsur\o/ \o/ \o/13:12
TheJuliadtantsur: congrats, that merged quickly13:12
*** ociuhandu has joined #openstack-ironic13:12
openstackgerritMerged openstack/ironic-python-agent master: setup.cfg: Replace dashes with underscores  https://review.opendev.org/c/openstack/ironic-python-agent/+/78953213:13
* dtantsur gets ready for the potential fallout13:13
TheJuliadtantsur: did you get the doc references?13:13
dtantsurTheJulia: I hope the patch fixes EVERYTHING in ironic13:13
dtantsurI need to check for references in other projects13:13
TheJuliaheh, I wish I had a chance to check it13:13
dtantsur(I only checked the CI so far)13:13
TheJuliadoh13:13
TheJuliaIt is a good start, we'll see13:13
dtantsurTheJulia: if you leave any comments, I'll try to address them asap13:14
TheJuliaI'll try to look at it in the next hour, coffee is slowly kicking in13:14
TheJuliaI didn't sleep well13:14
dtantsur:(13:14
openstackgerritDmitry Tantsur proposed openstack/ironic stable/wallaby: Do not mask configdrive when executing in-band deploy steps  https://review.opendev.org/c/openstack/ironic/+/78980713:18
openstackgerritDmitry Tantsur proposed openstack/ironic stable/wallaby: Update refarch with information about image_download_source  https://review.opendev.org/c/openstack/ironic/+/78980813:18
TheJuliahmm yeah, I was looking at the upgrade guides and thinking "we need to either revisit those or have a general one" and not duplicate the release notes.13:20
TheJuliaOn a plus side, we're fairly good at non-huge-horrible changes that haven't been telegraphed.... but if people don't read the release notes, there is not much we can do13:21
*** akahat|ruck has quit IRC13:21
dtantsurTheJulia: do we have labels for posts other than "News & Announcements"?13:23
TheJuliaI think we can just define new ones13:23
dtantsur"Deep Dives"?13:23
TheJuliaand I don't think, based on what I've seen, we need to do anything specific13:24
TheJulia++13:24
TheJuliawell, anything specific beyond re-use the same id and label in the markdown header13:24
dtantsurare labels declared anywhere or do I just use a new one?13:24
TheJuliaI think just use a new one13:24
dtantsurok, thanks13:24
TheJuliaI haven't seen anywhere where they are defined13:24
TheJuliaIt reads everything in when building and builds a graph, and then splits it all out13:24
dtantsurhmm, I need a category ID13:25
*** paras333_ has quit IRC13:25
TheJuliaI think I've changed it on one when trying to troubleshoot something and it didn't care13:25
dtantsurcontent/blog-config.json13:25
TheJuliaoh fun13:25
TheJuliaokay, missed that one13:25
TheJuliaoh, btw, on numerous wiki pages over the last week, I've put messages pointing to our actual documentation and stating that the contents are just there on the wiki for historical reasons.13:26
dtantsurstill unclear where the ID is coming from13:26
TheJuliaI think it is just random13:26
TheJuliaThere is no backing db13:26
dtantsuris it used in URLs? should we use something meaningful?13:26
TheJulianot used anywhere meaningful that I've seen13:26
dtantsurinteresting13:27
*** ociuhandu has quit IRC13:27
dtantsurTheJulia: is it only me or are links not rendered in https://ironicbaremetal.org/blog/ ?13:30
TheJuliaon the right hand side?13:30
dtantsurTheJulia: inside posts13:31
iurygregorysome information is missing13:31
dtantsurexactly, the links are completely missing, even their titles13:31
TheJuliadtantsur: I've not had any issues with getting links to render13:31
dtantsur"the awesome folks at StackHPC have written about ."13:31
TheJuliaseriously13:31
TheJuliaoh... ugh13:31
TheJuliathey changed the links13:32
dtantsur"an article on SuperUser titled . "13:32
TheJulia*sigh*13:32
iurygregoryyeah13:32
* TheJulia puts aside making the object handling faster on field selected lists and goes and sees how to fix the links13:32
*** akahat has joined #openstack-ironic13:33
TheJuliaumm13:34
TheJuliaokay13:34
TheJuliaI have links in my browser13:34
* TheJulia fires up a different browser13:35
TheJuliaFirefox ESR has "on SuperUser titled Scaling Bare Metal Provisioning with Nova and Ironic at CERN: Challenges & Solutions."13:36
TheJuliawith a valid link. Can you check the page source? maybe this ios some weird browser stylesheet thing13:36
iurygregoryI'm using firefox in fedora3413:37
TheJuliaand no link I take it?13:37
dtantsurare you sure you're looking at the listing, not at each page?13:37
dtantsurspecific pages have links13:37
dtantsuri.e. https://ironicbaremetal.org/blog/stackhpc-software-raid-in-ironic/ is okay, https://ironicbaremetal.org/blog/ is not13:37
TheJuliaoh13:37
iurygregoryI can only check the link after clicking to see the post13:37
TheJuliano link on the truncated summary13:37
iurygregoryyup13:38
*** ayoung has quit IRC13:38
TheJuliaahh13:38
TheJuliahmm13:38
TheJuliaemail send to jimmy/wes13:42
TheJuliaI suspect we'll need to reword them but maybe they have some magic13:42
iurygregoryyeah13:43
TheJuliaHonestly, it is likely for the best that we do that, and I had actually started down that path before they fixed the links on the non-video submissions I loaded up.13:43
iurygregoryso maybe we should limit the summary a little bit?13:45
iurygregoryso people need to click in the post to see the link13:45
TheJulialikely13:45
TheJuliaThey will likey get back to me in an hour or two13:45
openstackgerritKaifeng Wang proposed openstack/ironic master: Support port name: API  https://review.opendev.org/c/openstack/ironic/+/76556913:47
iurygregoryjust wondering if it would be a good "user experience" you need to click on the post to see the link for the blog post with the content13:48
TheJuliaI think since there can be varied content... well there already is, it is a good idea to drive that13:51
*** ociuhandu has joined #openstack-ironic13:51
dtantsurhaving to click a link to see one paragraph is not user-friendly13:52
dtantsurmeanwhile, https://github.com/OpenStackweb/ironic-website/pull/3013:53
TheJuliathis is also true13:53
iurygregorywe could use a bit more of space for the summary if the Categories and Connect where in the left side13:56
iurygregoryI see too much white in the page hehe13:56
TheJuliaI was kind of thinking the same thing, but hadn't mucked with the stylesheet yet13:57
openstackgerritMerged openstack/ironic stable/wallaby: Update refarch with information about image_download_source  https://review.opendev.org/c/openstack/ironic/+/78980813:57
TheJuliahmm, always amusing when you think the slower option is actually the faster option.13:57
TheJuliabut also turns out you need part of the first option13:57
*** ociuhandu has quit IRC13:58
*** ociuhandu has joined #openstack-ironic14:00
*** dhellmann has joined #openstack-ironic14:02
dtantsurbtw have you had a chance to talk to the foundation folks re disabling indexing of old release docs?14:04
openstackgerritDmitry Tantsur proposed openstack/ironic master: Document the custom-agent deploy interface  https://review.opendev.org/c/openstack/ironic/+/78984014:06
TheJuliano, that wouldn't actually be foundation, it would be the docs team or infra most likely14:08
TheJulialooks like we could likely just post a change, but we need to figure out what actually places the file14:13
*** vmud213 has quit IRC14:20
TheJulialooks like ussuri rocky and pike are the favorites14:26
iurygregoryrocky pike WOW14:27
dtantsurTheJulia: I think it has to go to an upper level14:28
TheJuliaon a positive note, seems like the search engines are actually now leaning towards latest14:29
TheJuliadtantsur: what do you mean?14:29
dtantsurTheJulia: robots.txt has to go to docs.openstack.org/ironic/ I think. We don't control that yet.14:29
TheJuliaI don't think so14:32
TheJuliahttps://en.wikipedia.org/wiki/Robots_exclusion_standard14:33
* TheJulia waits for git review to return14:37
TheJulia https://review.opendev.org/c/openstack/openstack-manuals/+/78984514:37
TheJuliathe oldest on the site seems to be mitaka, and I ran the exclusion through queens14:37
TheJuliawhich puts users rocky/stein/train/etc results as possibilities14:38
TheJuliaWe can be more aggressive though14:38
dtantsurI'd exclude everything in EM14:38
* TheJulia adjustss14:39
iurygregory++ to exclude everything in EM14:39
TheJuliauploading14:40
TheJulialeft train out, but it can be added a little later14:40
TheJuliasince it is not officially in EM yet14:40
TheJuliaso staff's response is to be verbose upfront on the summary and try and drive people in to help with SEO index magic14:41
dkingdtantsur: I wanted to thank you for your help yesterday. That was the solution. I was able to set user_data from the command line like this: openstack baremetal node deploy "$uuid" --config-drive '{"meta_data": {"hostname": ...}, "user_data": "#cloud-config\n{\"users\": [{\"name\": ...}]}"}'14:42
dtantsurI'm glad to hear :)14:43
TheJulia\o/14:47
dtantsurI wonder if we should document it somewhere, since cloud-init docs are.. questionable14:48
TheJuliaI was just thinking the same thing.14:48
* TheJulia goes and gets more coffee and hopes what she just found will return as a performance gain14:55
TheJuliaoh, wrt rss, they are going to look at it, they think it should be easy14:58
*** sdanni has joined #openstack-ironic15:22
*** k_mouza has quit IRC15:24
NobodyCamGood Morning Ironic'ers, Happy hump day!15:24
TheJuliagood morning NobodyCam15:25
*** k_mouza has joined #openstack-ironic15:25
NobodyCamo/ TheJulia15:25
iurygregorymorning NobodyCam o/15:25
NobodyCammorning iurygregory !15:25
dtantsuro/15:32
NobodyCamMorning dtantsur15:32
NobodyCamhumm IrcCloud looks to be having database issues ;p15:36
rpittauhey NobodyCam :)15:40
NobodyCamgood Morning rpittau :)15:40
openstackgerritMerged openstack/sushy master: Add support for BIOS Attribute Registry  https://review.opendev.org/c/openstack/sushy/+/78451615:41
TheJuliaNobodyCam: they were having some issues yesterday as well :(15:49
NobodyCam:(15:49
TheJuliaoh hey, got a thousand nodes down to consistenly 8.5 seconds from 1015:49
NobodyCamwith indexing alone?15:49
TheJuliano, its the object conversions that kill us as well15:50
TheJuliaindexing is still a big impact15:50
TheJulialike 0.4 seconds without indexes and 0.01 or 0.02 second cursor return when just running queries15:50
TheJuliasqlalchemy unfortunately also downloads the content before returning it15:50
TheJuliaheh15:51
TheJuliai broke it somewhere15:51
NobodyCambut that's an awesome improvement! \o/15:52
rpittausee ya folks! o/15:52
*** rpittau is now known as rpittau|afk15:52
NobodyCamhave a good night rpittau|afk15:53
TheJuliapart of the problem is we don't completely honor the list of fields all the way down to selectively get $things15:53
NobodyCam++ I was playing with the -c Column option and noticed there is no limit to what I can request. I thought it odd that I could get fields that are not on the original output15:56
TheJuliaNobodyCam: default is 1000 nodes15:56
NobodyCamwithout having to use --long15:56
TheJuliaseriously you were getting extra fields?15:56
NobodyCamI asked for them15:56
TheJuliaoh, okay15:56
TheJuliayeah15:56
NobodyCam```-c UUID -c Name -c 'Resource Class' -c 'Power State' -c 'Target Power State' -c 'Provisioning State' -c Maintenance -c 'Last Error' -c Properties -c 'Instance Info' -c 'Instance UUID' -c 'Driver Info'```15:57
NobodyCamtype stuff15:57
NobodyCamI expected the -c to to me more a filter then a fetch list15:58
NobodyCams/me/be/15:58
TheJuliaoh, yeah, they are disjointed15:59
dtantsur-c and -f come from cliff15:59
dtantsurthey have no knowledge of ironic16:00
NobodyCam++ how about index's, what happens to performance if I use -c to to select only unindexed fields?16:01
*** lucasagomes has quit IRC16:01
TheJuliano difference16:02
NobodyCam:)16:02
TheJuliabecause the result set has already been collected by db column off the primary key16:02
TheJuliaso it just goes and fetches the row and splits the data back16:03
TheJulianow, if there is lots of data, and we're not honoring columns, you can guess how quickly that can go sideways in transfer back16:03
NobodyCam++16:04
TheJuliaI think steve was headed mentally in the right direction16:04
TheJuliadirect db query from the API16:04
TheJuliastrip as many object conversions out16:04
TheJuliaas possible, and only handle columns16:04
TheJuliaor well, in the case when specific columns are sought after16:04
NobodyCamInitial thoughts are that could be a can of worms. having to maintain / secure / transactionalize (real word???) things, but would need to think about it more16:06
TheJuliawell, the bonus points on the secure we mostly do that db side wise if the user is not a system-admin (newer concept, part of the Secure RBAC effort), transactionalize wise... not horrible using the same db call substrate with ORM at least buys us consistency. Maintenance may be the headache since it is a bit of a pruned down code path... but trying to figure that out now16:08
*** ociuhandu has quit IRC16:10
openstackgerritMerged openstack/ironic stable/victoria: Inherit InvalidImageRef from InvalidParameterValue  https://review.opendev.org/c/openstack/ironic/+/78980616:11
TheJuliaAmusing, someone referenced shibboleth in a mailing list post16:18
*** k_mouza has quit IRC16:19
TheJuliaso 90 seconds flat for 10k nodes to be looped through16:19
iurygregorywow, this takes me back to 2015 when I was adding K2K Federation support to puppet O.o16:19
iurygregoryre shibboleth =)16:19
*** k_mouza has joined #openstack-ironic16:21
openstackgerritVerification of a change to openstack/ironic failed: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78873416:22
*** gyee has joined #openstack-ironic16:33
*** ociuhandu has joined #openstack-ironic16:33
*** ociuhandu has quit IRC16:33
TheJulia~110-111 records/second16:34
*** ociuhandu has joined #openstack-ironic16:34
TheJuliagoing back to the object conversion code and re-running to see what we get16:35
*** dtantsur is now known as dtantsur|afk16:38
dtantsur|afko/16:38
*** ociuhandu has quit IRC16:40
TheJuliaso selected object conversion seems to be faster16:41
TheJulia114/second16:42
*** ociuhandu has joined #openstack-ironic16:43
*** zzzeek has joined #openstack-ironic16:45
*** ociuhandu has quit IRC16:48
*** zzzeek has quit IRC16:50
TheJulia114-115/second with the middle object conversion16:50
TheJuliawut16:50
*** zzzeek has joined #openstack-ironic16:52
*** derekh has quit IRC17:00
arne_wiebalckbye everyone o/17:01
TheJuliastarting convert with links17:01
TheJulia1620234047.739879617:01
TheJuliafinished convert17:01
TheJulia1620234056.0874805 <-- To quote the mythbusters, there's our problem.17:01
*** zzzeek has quit IRC17:04
*** zzzeek has joined #openstack-ironic17:08
*** k_mouza_ has joined #openstack-ironic17:14
*** k_mouza has quit IRC17:17
sdanniHi TheJulia! I have a question about keylime ironic implementation. Keylime server's ip address needs to be passed to the node during deploy ramdisk booting so that the agent on the node will know who to talk with. I've tried to put them into pxe_parameters in ironic.conf and it worked. But since an ironic controller may have bunch of nodes and they may communicate with different keylime servers, is there another17:17
sdannioption to pass the ip address to the node other than passing in ironic.conf?17:17
TheJuliasdanni: have a general pattern of allowing a default in ironic.conf and a node level driver_info field as an override17:18
sdanniTheJulia: how does parameter in driver_info passed to the node?17:21
TheJuliasame way ipmi_address or redfish_address is set17:21
sdanniGot it. Thanks!17:25
*** k_mouza_ has quit IRC17:29
TheJuliaso the down side of sanitization, is the policy checking is applied17:29
*** k_mouza has joined #openstack-ironic17:29
openstackgerritMerged openstack/ironic stable/wallaby: Inherit InvalidImageRef from InvalidParameterValue  https://review.opendev.org/c/openstack/ironic/+/78980517:34
openstackgerritMerged openstack/ironic stable/wallaby: Do not mask configdrive when executing in-band deploy steps  https://review.opendev.org/c/openstack/ironic/+/78980717:35
TheJulia438 nodes/second17:47
TheJuliaJayF: NobodyCam: please tell me this is along the lines of what you see with CLI commands17:48
JayFIDK what you're asking?17:48
TheJuliafor a node list17:48
JayFnode list, limited at 1000, I should be able to give you a time value for that without unleashing any sekrits17:48
TheJuliawould be much appreciated17:48
TheJuliajust time to execute works :)17:49
JayFreal 0m9.031s17:49
JayFuser 0m4.005s17:50
JayFsys 0m0.227s17:50
JayFran `time openstack baremetal node list --limit 1000`17:50
TheJuliaany difference if you limit to a couple fields?17:50
JayFbe specific?17:50
JayFwhich fields17:50
TheJuliasay ask for last_error, uuid, provision_state17:51
NobodyCam```real0m25.551s17:51
NobodyCamuser0m0.752s17:51
NobodyCamsys0m0.189s```17:51
* TheJulia blinks at nobodycam's numbers17:51
JayFNo recognized column names in ['last_error', 'uuid', 'provision_state']. Recognized columns are ('UUID', 'Name', 'Instance UUID', 'Power State', 'Provisioning State', 'Maintenance'). My least favorite openstack cliism17:51
TheJuliadtantsur|afk: we need to get something like ironic-stats into the field17:51
*** k_mouza_ has joined #openstack-ironic17:52
NobodyCam```ckrelle-mlt:ussuri ckrelle$ time openstack baremetal node list | wc -l17:52
NobodyCam     93317:52
NobodyCamreal1m1.869s17:52
NobodyCamuser0m0.755s17:52
NobodyCamsys0m0.187s```17:52
TheJuliawow, okay17:52
JayFtime openstack baremetal node list --limit 1000 -c Name -c UUID -c 'Provisioning State'17:52
TheJuliatons of variables here17:52
NobodyCamone region ;p17:52
JayFreal 0m8.799s17:52
JayFuser 0m3.664s17:52
JayFsys 0m0.190s17:52
JayFabout the same17:52
TheJuliajanders: ack, thanks17:52
TheJuliawell, performance issues are always death by a thousand cuts17:53
*** k_mouza_ has quit IRC17:53
JayFI really, really hope that kind of performance dip isn't waiting for us when we upgrade17:53
JayFbecause NobodyCam's numbers are BRUTAL17:53
TheJuliasomething like 2.275 seconds per 1000 with no intermediate network17:54
TheJuliadtantsur|afk: ^^^ is what we should be able to get it up to, fyi17:54
TheJuliawe also don't know what else is going on, but we should be able to backport some of this fairly far17:54
*** k_mouza_ has joined #openstack-ironic17:55
TheJuliaI guess I'll package it all up in one change and start cutting out the major pieces that make sense17:55
*** k_mouza has quit IRC17:55
*** k_mouza_ has quit IRC17:59
* NobodyCam dances17:59
*** knikolla has joined #openstack-ironic18:10
openstackgerritVerification of a change to openstack/ironic failed: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78873418:11
*** k_mouza has joined #openstack-ironic18:29
*** k_mouza has quit IRC18:34
*** ociuhandu has joined #openstack-ironic18:36
*** ociuhandu has quit IRC18:41
*** k_mouza has joined #openstack-ironic19:13
*** k_mouza has quit IRC19:18
openstackgerritMerged openstack/ironic master: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78873420:56
openstackgerritRiccardo Pittau proposed openstack/ironic stable/wallaby: Provide an option to not cache bootable iso ramdisks  https://review.opendev.org/c/openstack/ironic/+/78986321:04
*** lmcgann has quit IRC21:33
*** sdanni has quit IRC21:51
*** k_mouza has joined #openstack-ironic22:30
*** k_mouza has quit IRC22:35
*** gyee has quit IRC22:41
TheJuliahttps://deploy-preview-31--ironicbaremetal.netlify.app/blog/the-search-for-performance/ https://deploy-preview-31--ironicbaremetal.netlify.app//img/blog-performance.png <-- i don't know why the url in the post is so weird, but anyway you guys will hopefully find it amusing and semi-reassuring22:48
TheJuliaNobodyCam: JayF ^^^22:48
TheJuliadtantsur|afk: ^^ also, I'll upload the state into gerrit tomorrow of my test vm, and likely work on splitting it into proper patches next week23:01
*** tosky has quit IRC23:03
*** rcernin has joined #openstack-ironic23:04
NobodyCamoh sweet23:07
* NobodyCam clicks23:07
TheJuliait definitely won't pass any testing, it is about as hacked up as I can make it to just make the stats command return the data as fast as possible23:10
TheJuliabut individual changes should be good23:10
jandersgood morning Ironic o/23:20
*** rloo has quit IRC23:22
NobodyCamGood Morning janders23:30
NobodyCamoh If I haven't sed it today or lately Ironic has an Awesome PTL23:31
janders+1 :)23:36
« Tuesday, 2021-05-04 Index Thursday, 2021-05-06 »

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