| opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | 03:37 |
|---|---|---|
| opendevreview | melanie witt proposed openstack/nova master: Use split kernel/initramfs Cirros UEC image by default in jobs https://review.opendev.org/c/openstack/nova/+/902217 | 07:25 |
| sean-k-mooney[m] | gibi: melwitt can either of ye take a look at https://review.opendev.org/c/openstack/nova/+/883682 | 08:47 |
| sean-k-mooney[m] | its too late to pull that into our next downstream release but it would still be nice to get that fixed before the end of the year if we can | 08:48 |
| sean-k-mooney[m] | melwitt: oh i see you have some comments since i last reviewd ill read over them now | 08:50 |
| sean-k-mooney[m] | ok dropped back to +1 based on your comments | 09:03 |
| sean-k-mooney[m] | bauzas: can you review this functional repoducer test quickly. gibi and i are +2 on the actual fix https://review.opendev.org/c/openstack/nova/+/891290/1 gibi helped write the repoducer so we need another core to review | 09:45 |
| opendevreview | Merged openstack/nova stable/2023.1: Do not manage CPU0's state https://review.opendev.org/c/openstack/nova/+/901567 | 10:11 |
| opendevreview | Merged openstack/nova stable/2023.1: Allow enabling cpu_power_management with 0 dedicated CPUs https://review.opendev.org/c/openstack/nova/+/901660 | 10:31 |
| bauzas | sean-k-mooney: sorry I was off due to some issue with my car, I'm back now | 10:36 |
| stephenfin | JayF: If it's okay, I'd rather someone else take that. I think it's reasonably complete | 10:40 |
| sean-k-mooney | bauzas: no worries | 10:45 |
| bauzas | I needed to go to the policy just because the folk that hitted my car with his skis wasn't replied to me :( | 10:46 |
| bauzas | replying* | 10:46 |
| bauzas | police* | 10:47 |
| bauzas | shit, I need to remember how to run a specific stestr worker | 10:58 |
| * bauzas goes back at https://stestr.readthedocs.io/en/latest/MANUAL.html | 10:59 | |
| bauzas | yeah found $ stestr last --subunit | subunit-filter -s --xfail --with-tag=worker-3 | subunit-ls > slave-3.list | 11:01 |
| bauzas | hah | 11:09 |
| bauzas | once you have a list of tests from the same worker, stestr run --analyze-isolation does the job for you | 11:10 |
| bauzas | naïce | 11:10 |
| bauzas | https://paste.opendev.org/show/bDpwRC0njzsbM1PmxTKr/ | 11:10 |
| bauzas | \o/ | 11:10 |
| bauzas | not sure auniyal6 and Uggla know about this ^ | 11:11 |
| dvo-plv | Hello, sean-k-mooney gibi your comments for glance were resolved and zull verification pass successful | 11:15 |
| dvo-plv | https://review.opendev.org/c/openstack/glance/+/882133 | 11:15 |
| *** tobias-urdin-pto is now known as tobias-urdin | 11:15 | |
| elodilles | bauzas: fyi, i need to take off the rest of the day, so i'm almost certain that i won't be around at the nova meeting :S i'll update the stable release patches when i'll be back so that we can release them finally | 11:21 |
| *** elodilles is now known as elodilles_pto | 11:21 | |
| bauzas | ++ | 11:23 |
| sean-k-mooney | dvo-plv: cool just bare in mind gibi and i dont have core rights on glance | 11:24 |
| sean-k-mooney | but ill re review and add my +1 so they know nova are fine with the patch | 11:24 |
| dvo-plv | okay, thanks, I saw Abhishek in the reviewers, but I need one more person, whom I should to ask for review ? | 11:25 |
| sean-k-mooney | i added dansmith to the review list but im not sure if they are on PTO for the rest of they year yet. i think tehy were here this week but not sure | 11:26 |
| dvo-plv | I see, thank you | 11:27 |
| opendevreview | Merged openstack/nova master: Remove workaround for old python-ironicclient https://review.opendev.org/c/openstack/nova/+/823247 | 11:38 |
| opendevreview | Merged openstack/nova master: Add retry on attachment_update https://review.opendev.org/c/openstack/nova/+/901815 | 11:38 |
| opendevreview | Merged openstack/nova master: [libvirt]Add migration_inbound_addr https://review.opendev.org/c/openstack/nova/+/900203 | 11:43 |
| opendevreview | Merged openstack/nova master: Reproduce bug #2025480 in a functional test https://review.opendev.org/c/openstack/nova/+/891290 | 11:43 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for add/remove instance info from node https://review.opendev.org/c/openstack/nova/+/659691 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for getting network metadata from node https://review.opendev.org/c/openstack/nova/+/670213 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for volume_target.* https://review.opendev.org/c/openstack/nova/+/867790 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_provision_state https://review.opendev.org/c/openstack/nova/+/867791 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.set_power_state https://review.opendev.org/c/openstack/nova/+/867792 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for node.vif* https://review.opendev.org/c/openstack/nova/+/867793 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining network operations https://review.opendev.org/c/openstack/nova/+/867794 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for remaining ironic driver calls https://review.opendev.org/c/openstack/nova/+/867795 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use SDK for deprecated baremetal proxy API https://review.opendev.org/c/openstack/nova/+/867796 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Remove Ironic client wrapper https://review.opendev.org/c/openstack/nova/+/867797 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Remove unnecessary aliasing https://review.opendev.org/c/openstack/nova/+/867798 | 12:10 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Use real SDK objects in tests https://review.opendev.org/c/openstack/nova/+/867799 | 12:10 |
| stephenfin | JayF: Actually, it was a trivial enough rebase | 12:10 |
| stephenfin | I thought it would have rotted more | 12:10 |
| stephenfin | sean-k-mooney: Could you look at https://review.opendev.org/c/openstack/nova/+/896227/ and https://review.opendev.org/c/openstack/nova/+/896228/ again, when you get a chance? | 12:29 |
| sean-k-mooney | i just got the email pings so yes | 12:29 |
| stephenfin | ta | 12:31 |
| sean-k-mooney | done, im happy with both of those and all the commits below | 12:38 |
| opendevreview | Merged openstack/nova master: Do not untrack resources of a server being unshelved https://review.opendev.org/c/openstack/nova/+/890406 | 12:56 |
| opendevreview | Merged openstack/nova master: doc: Remove crud from conf.py file https://review.opendev.org/c/openstack/nova/+/896224 | 12:56 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Remove nova.wsgi module https://review.opendev.org/c/openstack/nova/+/902686 | 14:48 |
| opendevreview | Stephen Finucane proposed openstack/nova master: Add new nova.wsgi module https://review.opendev.org/c/openstack/nova/+/902687 | 14:48 |
| opendevreview | Stephen Finucane proposed openstack/nova master: setup: Remove pbr's wsgi_scripts https://review.opendev.org/c/openstack/nova/+/902688 | 14:48 |
| stephenfin | sean-k-mooney: I think that's where we are now ^ | 14:48 |
| sean-k-mooney | i started looking at makeing pbr work but i didnt have time to actully fix the wsgi_sripts functionatly | 14:51 |
| sean-k-mooney | but im ok with also takeign the approch of avoiding the extion if others are | 14:52 |
| sean-k-mooney | ill try an review them again shortly | 14:52 |
| sean-k-mooney | i know rougly where to make the change just havnt figureoud out exactly how too. | 14:52 |
| opendevreview | Merged openstack/nova master: pre-commit: Update plugin versions https://review.opendev.org/c/openstack/nova/+/896226 | 14:53 |
| sahid | o/ quick question guys | 14:54 |
| sahid | regarding https://review.opendev.org/c/openstack/nova/+/896512 | 14:54 |
| sahid | I'm glad that may consider it | 14:54 |
| sahid | just, naything else is needed? | 14:54 |
| opendevreview | Merged openstack/nova master: pre-commit: Use native hacking pre-commit hook https://review.opendev.org/c/openstack/nova/+/897005 | 14:55 |
| sean-k-mooney | sahid: procedurelly jsut need to be approved in the team meeting later | 14:56 |
| sean-k-mooney | i have withdrawn my objections so im fine with tracking this as a specless blueprint | 14:56 |
| jkulik | on the topic of WSGI: there's this bug open https://bugs.launchpad.net/nova/+bug/1930406 which talks about 2 things: 1) dropping the rpc-call to nova-compute on volume-attach to reserve_block_device_name() and 2) running Nova with threads instead of eventlet - where could I look for the state/updates on these topics? | 15:06 |
| sean-k-mooney | jkulik: im not sure that anyone is working on them currecnly i will remove lees name form it as they nolonger work on openstack | 15:11 |
| sean-k-mooney | threads>1 is not somethign we are likely to supprot going forward | 15:12 |
| sean-k-mooney | at least not in the short term | 15:13 |
| sean-k-mooney | we may howeveer do that indirecly fi we end up removing our use of eventlets entirly for the nova-api when run under wsgi | 15:13 |
| sean-k-mooney | i know melwitt had partly looked at changign to using treadpools at one point | 15:13 |
| sean-k-mooney | i cant recall if we merged that | 15:13 |
| jkulik | ok. maybe I can find a review-request for that. I'll take a look | 15:14 |
| jkulik | thank you | 15:14 |
| sahid | sean-k-mooney: ack thank you | 15:16 |
| sean-k-mooney | jkulik: https://review.opendev.org/c/openstack/nova/+/650172 | 15:16 |
| sahid | dansmith: regarding your reviews on it, thanks a lot. There are some that I'm not sure about, I have kept them as unrosolved, if you can have a look | 15:17 |
| sean-k-mooney | jkulik: i dont reacall if there was anything preventing that form movign forword other then the fact i and dansmith have been busy and melwitt hasnt pinged use to review | 15:17 |
| sean-k-mooney | but the last patch passed ci | 15:17 |
| sean-k-mooney | so perhaps we can move forward with that. | 15:17 |
| jkulik | sounds good. I'll read through it, too. | 15:18 |
| bauzas | gibi: given my next internal meeting, I can't really do the upstream meeting | 15:19 |
| sean-k-mooney | i just rebased https://review.opendev.org/c/openstack/devstack/+/893302 to get new results too | 15:19 |
| bauzas | gibi: could you please chair it ? | 15:20 |
| jkulik | sean-k-mooney: FYI, I found an abandoned spec https://review.opendev.org/c/openstack/nova-specs/+/765097 for "Remove device from attach_volume request response" | 15:28 |
| sean-k-mooney | jkulik: ya so this feel off our radar since we dint have anyoen working on it | 15:30 |
| jkulik | yeah, I'll put it into the bug report to maybe re-propose it later on | 15:30 |
| bauzas | man the global CONF object is killing me | 15:55 |
| bauzas | if some test runs before some other, then a group is persistent between them | 15:56 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Dec 5 16:00:53 2023 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| bauzas | as I said earlier, I have a lot of things to do now | 16:01 |
| bauzas | so, please, let's do a very short meeting (like 10 mins) | 16:01 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| bauzas | I can start | 16:02 |
| bauzas | #topic Bugs (stuck/critical) | 16:02 |
| bauzas | #info No Critical bug | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 38 new untriaged bugs (+0 since the last meeting) | 16:02 |
| bauzas | #info Add yourself in the team bug roster if you want to help https://etherpad.opendev.org/p/nova-bug-triage-roster | 16:02 |
| bauzas | #info bug baton is gibi | 16:02 |
| bauzas | let's move on | 16:02 |
| bauzas | #topic Gate status | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:02 |
| bauzas | #link https://etherpad.opendev.org/p/nova-ci-failures-minimal | 16:02 |
| bauzas | thanks to melwitt ^ | 16:03 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fnova&project=openstack%2Fplacement&pipeline=periodic-weekly Nova&Placement periodic jobs status | 16:03 |
| bauzas | periodics are green for the master branch | 16:03 |
| bauzas | #info Please look at the gate failures and file a bug report with the gate-failure tag. | 16:03 |
| bauzas | moving on again | 16:03 |
| bauzas | #topic Release Planning | 16:03 |
| bauzas | #link https://releases.openstack.org/caracal/schedule.html#nova | 16:03 |
| bauzas | #info Caracal-2 (and spec freeze) milestone in 5 weeks | 16:03 |
| bauzas | given both of some personal issues this morning + some hard work on bugfixes, I forgot about our today's spec freeze day | 16:04 |
| bauzas | so, | 16:04 |
| bauzas | #info Spec review day tomorrow | 16:04 |
| bauzas | #action bauzas to send an email accordingly | 16:04 |
| * gibi joining late | 16:04 | |
| bauzas | for the records, next cycle, I'll propose our review days to be on Wednesdays | 16:04 |
| bauzas | easier to not forget about them | 16:04 |
| bauzas | and my personal action is to add the ical in my gcal | 16:05 |
| bauzas | moving on | 16:05 |
| bauzas | #topic Review priorities | 16:05 |
| bauzas | #link https://etherpad.opendev.org/p/nova-caracal-status | 16:05 |
| bauzas | I've moved two bugfixes series ^ | 16:05 |
| bauzas | you can also review them | 16:05 |
| bauzas | I'll lookup the feature series once I have time | 16:05 |
| bauzas | (hopefully before 2024) | 16:06 |
| bauzas | #topic Stable Branches | 16:06 |
| bauzas | elodilles_pto is on pto | 16:06 |
| * gibi taking the bug baton without promises | 16:06 | |
| bauzas | copying the notes | 16:06 |
| bauzas | #info stable gates don't seem blocked | 16:06 |
| bauzas | #info stable release patches still open for review: https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova | 16:06 |
| bauzas | +1: yoga is going to be unmaintained, so final stable/yoga release should happen ASAP - https://etherpad.opendev.org/p/nova-stable-yoga-eom | 16:07 |
| bauzas | #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci | 16:07 |
| bauzas | I'll ask elodilles_pto about yoga tomrrow | 16:07 |
| bauzas | fwiw, I +1d the antelope release patch with a note | 16:07 |
| bauzas | asking whether we need a .y version | 16:07 |
| bauzas | moving on | 16:07 |
| bauzas | #topic vmwareapi 3rd-party CI efforts Highlights | 16:07 |
| bauzas | copying the notes too | 16:07 |
| Uggla | o/ | 16:08 |
| bauzas | #Info A productive 768GB host reserved for CI and "outer" hypervisor has been being installed. Test VMs are going to be nested. Main challenge: Network setup for nested hypervisor, testing in lab environment. | 16:08 |
| bauzas | bravo fwiesel | 16:08 |
| bauzas | and thanks for the update | 16:08 |
| bauzas | last topic | 16:08 |
| bauzas | #topic Open discussion | 16:08 |
| bauzas | (artom) Specless blueprint for persistent mdevs | 16:08 |
| bauzas | #link https://blueprints.launchpad.net/nova/+spec/persistent-mdevs | 16:08 |
| bauzas | no upgrade concerns I can see | 16:09 |
| bauzas | the libvirt version needed for this is older than Antelope, but I need to doublecheck | 16:09 |
| bauzas | given the lack of quorum, I propose to punt the approval until next week | 16:10 |
| bauzas | and hopefully, I'd have more time to doublecheck the upgrade concerns | 16:10 |
| JayF | I have a couple items for open discussion, if that's OK? | 16:11 |
| bauzas | like I said at the beginning I have a very constrained agenda for today | 16:12 |
| bauzas | hence the very short meeting | 16:12 |
| bauzas | so either other folks are joining or we punt that to next week | 16:12 |
| JayF | Well, one is a question can be fielded outside, the other is a specless blueprint you'd likely punt for quorum anyway, so we can hit it next week. | 16:12 |
| bauzas | JayF: please add it in https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:13 |
| bauzas | so we won't forget :) | 16:13 |
| bauzas | (last open discussion topic) | 16:13 |
| JayF | I will once you clear it for this one :D | 16:14 |
| bauzas | okay, thanks folks | 16:15 |
| bauzas | that was a short one | 16:15 |
| bauzas | sorry for the rush | 16:15 |
| bauzas | #endmeeting | 16:15 |
| opendevmeet | Meeting ended Tue Dec 5 16:15:31 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:15 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-12-05-16.00.html | 16:15 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-12-05-16.00.txt | 16:15 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-12-05-16.00.log.html | 16:15 |
| JayF | bauzas: the question was, do I need to resubmit https://specs.openstack.org/openstack/nova-specs/specs/2023.2/approved/ironic-shards.html for caracal; I think I answered my question as "yes" | 16:15 |
| JayF | and am working on that right now | 16:15 |
| bauzas | JayF: please do indeed | 16:15 |
| fwiesel | thanks, bauzas | 16:15 |
| opendevreview | Jay Faulkner proposed openstack/nova-specs master: Re-submit Ironic-shards for Caracal https://review.opendev.org/c/openstack/nova-specs/+/902698 | 16:17 |
| bauzas | JayF: right in time for the spec review day tomorrow | 16:19 |
| JayF | that's what I saw that gave me the moment of "I need to do this" :D | 16:21 |
| JayF | I'm going to go review all of stephen's changes to put SDK support in nova<>ironic driver for now though | 16:21 |
| JayF | trying to get that in first... remove the footgun that caused the failure last cycle rather than just charging forward | 16:21 |
| bauzas | the global CONF object is getting me nuts | 16:22 |
| bauzas | how the fuck can I ensure that it's cleaned up in my test ? | 16:22 |
| bauzas | I fucking need to reset the whole blob | 16:23 |
| sean-k-mooney | bauzas: we have a fixture for that | 16:30 |
| bauzas | ohohoh | 16:31 |
| bauzas | my brain fscked | 16:31 |
| bauzas | I remember we have a solution but I forgot the pattern | 16:31 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/test.py#L214-L219 | 16:32 |
| sean-k-mooney | its done in the base test case | 16:32 |
| bauzas | context : https://review.opendev.org/c/openstack/nova/+/902084/3/nova/tests/unit/virt/libvirt/test_driver.py#26492 fails because the CONF object is persistent across tests | 16:32 |
| sean-k-mooney | the base nova one | 16:32 |
| sean-k-mooney | it shoudl not be unless you did somethign odd | 16:32 |
| bauzas | once the dynamic group is registered in some test, it's kept on the global CONF obj | 16:32 |
| bauzas | sean-k-mooney: I can prove it | 16:33 |
| sean-k-mooney | oh well i guess then the fixture is incomplete | 16:33 |
| sean-k-mooney | and needs to be updated to supprot that | 16:33 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/test.py#L219 | 16:33 |
| sean-k-mooney | we do use the fixture in any test case that inherits form nova.test.TestCase | 16:34 |
| sean-k-mooney | its defiend here https://github.com/openstack/nova/blob/master/nova/tests/fixtures/conf.py#L26 | 16:34 |
| bauzas | I see it | 16:35 |
| sean-k-mooney | https://github.com/openstack/nova/blob/master/nova/tests/fixtures/conf.py#L69 | 16:35 |
| sean-k-mooney | that is likely relevent | 16:35 |
| bauzas | this is normal if you did set the flags before | 16:35 |
| sean-k-mooney | the flags are not ment to exscape the current test case | 16:36 |
| bauzas | what's not normal is that I still have the groups registered in some test before I set them | 16:36 |
| bauzas | anyway, I need to prepare some internal meetup | 16:36 |
| bauzas | so I'll have to hardstop | 16:36 |
| sean-k-mooney | ok | 16:36 |
| opendevreview | Merged openstack/nova stable/2023.1: Translate VF network capabilities to port binding https://review.opendev.org/c/openstack/nova/+/898945 | 16:49 |
| melwitt | sean-k-mooney: https://review.opendev.org/c/openstack/nova/+/650172 is ready for review iirc, I don't remember if there are unaddressed comments so if there are, just lmk | 17:05 |
| melwitt | sean-k-mooney: and the nova patch with uec images passed finally! https://review.opendev.org/c/openstack/nova/+/902217 | 17:06 |
| sean-k-mooney | melwitt: woot | 17:57 |
| sean-k-mooney | i noticed you working on it quite late last night considering it was early in my morning :) | 17:58 |
| melwitt | sean-k-mooney: the patch was so close to working 😁 | 18:16 |
| clarkb | melwitt: fwiw we should have 30 day retention of job logs in our swift backends | 18:16 |
| clarkb | that range originated with our old 10TB fileserver for logs. That was basically as much as we could fit at the time into that space with some headroom for maintenance and we carried it over when we moved to swift | 18:17 |
| melwitt | clarkb: ah ok. good to know. I always use "last 30 days" but I feel like I haven't seen results that go back further than around 10 days. I'll check again | 18:18 |
| clarkb | https://zuul.opendev.org/t/zuul/build/19dae94a70524ae28db9cd8ed4257136/logs these are almost 30 days old | 18:19 |
| melwitt | hrrmm ok, thanks | 18:21 |
| melwitt | dpawlik: do you know if there is a time limit of some kind on the opensearch log indexing? I generally see around 10 days worth of search results but the logs are actually kept for 30 days? ^ | 18:31 |
| clarkb | oh you are talking about retention in opensearch | 18:35 |
| clarkb | in the old elasticsearch system we had between 7-10 days of retention due to indexing the data causing it to explode in size and that is after filtering out the debug logs | 18:35 |
| melwitt | yes, sorry, I thought your comment was in reference to my ML post about the gate issues in nova and my mention of 10 days worth of opensearch results :) | 18:39 |
| clarkb | it was and I misread it to say you are giving opensearch links because the static log storage doesn't have more than 10 days of retention | 18:43 |
| melwitt | ahhh gotcha | 18:43 |
| sean-k-mooney | the static log storage used to be 30 in the past right | 18:44 |
| sean-k-mooney | then it was 30 days subject to storage not being full | 18:44 |
| sean-k-mooney | when did it drop to 10 days for zuul logs out of interest | 18:44 |
| clarkb | sean-k-mooney: when we moved to swift it has always been 30 days since we weren't limited by filesystems aynmore. The 10 day thing is opensearch specific I think | 18:45 |
| melwitt | the opensearch share links were sometimes not working at all, for lack of a better way to describe it. I would copy paste or click the share icon and get the link, then to test it I pasted it into a new browser tab and after the first visit, the link no longer worked and would not bring up the search results and would just load the page with no search query at all | 18:45 |
| sean-k-mooney | ah ok | 18:46 |
| sean-k-mooney | melwitt: i alwasy put the opensarch link thorugh tiny url by the way | 18:46 |
| sean-k-mooney | https://tinyurl.com/app | 18:46 |
| melwitt | I'll try that. I tried the built in url shortener in opensearch but it doesn't work or isn't enabled or something and I didn't go to another one | 18:47 |
| melwitt | gmann: you might be interested in these, some patches intended to improve our CI stability in nova https://review.opendev.org/q/topic:%22nova-ci-improvements%22 one patch is for tempest to support UEC images | 18:58 |
| gmann | melwitt: ack, thanks | 19:13 |
| gmann | melwitt: commented in https://review.opendev.org/c/openstack/nova/+/902217 | 20:04 |
| gmann | melwitt: also I did not get how tempest change is dependency of it | 20:04 |
| gmann | melwitt: btw I like the idea of using UEC image, maybe we can try in a few of the tempest jobs also | 20:05 |
| melwitt | gmann: thanks for looking! the tempest change is needed bc there are a couple of places where tempest creates an image itself, and those couple of places currently don't support UEC images. without the tempest change, when tempest creates an image based on the default image, it doesn't set the kernel and ramdisk image IDs in the create request that are needed by the AMI image, so when it tries to create a server from that image, it | 22:16 |
| melwitt | won't work. the server won't boot. similarly when tempest creates a fresh image from the tarball, currently it can't support that, it won't extract the tarball and create the kernel and ramdisk images in glance, and associate the AMI image it creates with them | 22:16 |
| gmann | melwitt: ah i see, tempest uses img_file in create, got it thanks https://review.opendev.org/c/openstack/tempest/+/902609/3/tempest/scenario/manager.py#771 | 22:24 |
| gmann | melwitt: +2 on tempest change, on nova change did you see my comment about CIRROS_VERSION | 22:25 |
| gmann | setting https://review.opendev.org/c/openstack/nova/+/902217/9/.zuul.yaml#512 | 22:25 |
| gmann | melwitt: as it has one +2 and work in current devstack default. I am ok to convert my vote and ok to do it | 22:25 |
| gmann | that in follow up | 22:25 |
| melwitt | gmann: yes, just replied. you are right about not needing to set those default image vars for nova-next. setting those is residue from my original PS that was setting defaults in project.vars and trying to inherit from that. that ultimately couldn't work bc devstack-tempest sets IMAGE_URLS and overrides project.vars, so I had to change it around | 22:27 |
| melwitt | gmann: ok sure, sounds good. I'll upload a follow up | 22:28 |
| gmann | melwitt: thanks, +A | 22:31 |
| opendevreview | melanie witt proposed openstack/nova master: Remove redundant setting of DEFAULT_IMAGE_NAME(_FILE) https://review.opendev.org/c/openstack/nova/+/902734 | 22:35 |
| melwitt | ^ thanks gmann | 22:36 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!