| *** hamzy has joined #openstack-ironic | 00:05 | |
| *** mjturek has quit IRC | 00:07 | |
| *** baha has joined #openstack-ironic | 00:18 | |
| *** trungnv has quit IRC | 00:25 | |
| *** trungnv has joined #openstack-ironic | 00:25 | |
| *** moshele has joined #openstack-ironic | 00:35 | |
| *** openstackgerrit has joined #openstack-ironic | 01:19 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Activating/deactiating console can cause node being exclusively locked by the conductor and never unlocked after. https://review.openstack.org/603812 | 01:19 |
|---|---|---|
| *** rloo has quit IRC | 01:26 | |
| *** tiendc has joined #openstack-ironic | 01:33 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up to I55a112d877d94f31d27487846ff59fe27f602f8b https://review.openstack.org/612867 | 01:37 |
| *** litao has joined #openstack-ironic | 01:40 | |
| *** moshele has quit IRC | 01:46 | |
| *** MattMan_ has quit IRC | 01:47 | |
| *** MattMan_ has joined #openstack-ironic | 01:47 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Add a locking interface https://review.openstack.org/599191 | 01:49 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix DHCPv6 support https://review.openstack.org/611600 | 01:56 |
| openstackgerrit | Digambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver https://review.openstack.org/609982 | 02:13 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove pywsman reference https://review.openstack.org/612869 | 02:18 |
| *** baha has quit IRC | 02:39 | |
| *** gabys has joined #openstack-ironic | 02:46 | |
| *** jcoufal has quit IRC | 02:50 | |
| openstackgerrit | Kaifeng Wang proposed openstack/ironic-python-agent master: Follow up to parallel disk erasure https://review.openstack.org/612874 | 02:54 |
| *** sambetts|afk has quit IRC | 02:56 | |
| *** sambetts_ has joined #openstack-ironic | 03:00 | |
| *** mmethot has quit IRC | 03:00 | |
| *** mmethot has joined #openstack-ironic | 03:01 | |
| openstackgerrit | Merged openstack/sushy-tools master: Rename `glance` for `image` service in openstacksdk https://review.openstack.org/611912 | 03:07 |
| *** gabys has quit IRC | 03:15 | |
| *** gabys has joined #openstack-ironic | 03:16 | |
| *** gabys has quit IRC | 03:20 | |
| *** zhangfei has joined #openstack-ironic | 03:38 | |
| *** skazi has joined #openstack-ironic | 04:01 | |
| openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Follow-up to fix not exist deploy image of patch 592247 https://review.openstack.org/612880 | 04:07 |
| *** zhangfei has quit IRC | 04:18 | |
| *** tiendc has quit IRC | 04:26 | |
| *** skazi has quit IRC | 04:35 | |
| *** skazi has joined #openstack-ironic | 06:00 | |
| *** ddurst has quit IRC | 06:15 | |
| *** ddurst has joined #openstack-ironic | 06:29 | |
| *** raginbajin has quit IRC | 06:32 | |
| *** raginbajin has joined #openstack-ironic | 06:34 | |
| *** ddurst has quit IRC | 06:46 | |
| *** ddurst has joined #openstack-ironic | 06:55 | |
| iurygregory | good morning | 06:56 |
| *** rcernin has quit IRC | 07:03 | |
| *** pcaruana has joined #openstack-ironic | 07:05 | |
| *** skazi has quit IRC | 07:06 | |
| *** serlex has joined #openstack-ironic | 07:07 | |
| *** threestrands has quit IRC | 07:13 | |
| *** adrianc has joined #openstack-ironic | 07:43 | |
| *** adrianc_ has joined #openstack-ironic | 07:43 | |
| *** gabys has joined #openstack-ironic | 07:53 | |
| *** e0ne has joined #openstack-ironic | 07:55 | |
| *** ianychoi_ is now known as ianychoi | 08:03 | |
| *** gabys has quit IRC | 08:15 | |
| *** gabys has joined #openstack-ironic | 08:16 | |
| *** dougsz has joined #openstack-ironic | 08:16 | |
| *** gabys has quit IRC | 08:20 | |
| *** moshele has joined #openstack-ironic | 08:38 | |
| *** derekh has joined #openstack-ironic | 08:39 | |
| *** priteau has joined #openstack-ironic | 08:48 | |
| *** vabada has quit IRC | 08:51 | |
| *** vabada has joined #openstack-ironic | 08:52 | |
| *** sambetts_ is now known as sambetts|afk | 09:07 | |
| *** Emine has joined #openstack-ironic | 09:13 | |
| *** miniTurtle has joined #openstack-ironic | 09:13 | |
| *** stendulker has joined #openstack-ironic | 09:15 | |
| *** dtantsur|afk is now known as dtantsur | 09:34 | |
| dtantsur | morning ironic, morning iurygregory | 09:35 |
| iurygregory | morning dtantsur o/ | 09:36 |
| stendulker | dtantsur, iurygregory: o/ | 09:42 |
| iurygregory | stendulker, \o | 09:43 |
| *** jesusaur has quit IRC | 09:45 | |
| etingof | good morning stendulker dtantsur and iurygregory o/ | 09:46 |
| stendulker | etingof: o/ | 09:46 |
| iurygregory | etingof, \o | 09:46 |
| jiapei | Good morning all | 09:48 |
| *** jesusaur has joined #openstack-ironic | 09:49 | |
| *** skazi has joined #openstack-ironic | 09:56 | |
| *** skazi has quit IRC | 09:57 | |
| *** stendulker has quit IRC | 09:59 | |
| dtantsur | o/ | 10:00 |
| *** iurygregory is now known as iury|lunch | 10:07 | |
| miniTurtle | hello | 10:11 |
| dtantsur | hi miniTurtle, cool nick! :) | 10:24 |
| openstackgerrit | Merged openstack/ironic master: Document how to implement a new deploy step https://review.openstack.org/609342 | 10:29 |
| dtantsur | etingof: hi! do you plan on finalizing https://review.openstack.org/#/c/611095/ today? | 10:36 |
| patchbot | patch 611095 - sushy - Untie `Connection` from authenticator on `.close()` - 4 patch sets | 10:36 |
| etingof | dtantsur, Let me see if I could overhaul this thing | 10:37 |
| etingof | today | 10:37 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: WIP: Add memoization to emulator connections https://review.openstack.org/612758 | 10:37 |
| etingof | dtantsur, btw, this patch ^ makes sushy approximately as slow as openstackbmc in the OVB. does this kind of memoization make sense to you? | 10:39 |
| dtantsur | etingof: I'll check after I finish reviewing another thing | 10:39 |
| etingof | thanks! I am interested in general go/no-go, no need for code review atm | 10:40 |
| dtantsur | etingof: so, what's the idea? to cache all responses? | 10:45 |
| *** iury|lunch is now known as iurygregory | 10:45 | |
| etingof | dtantsur, basically, yes but in a tricky way... | 10:47 |
| etingof | dtantsur, some info from nova can be cached for life | 10:47 |
| etingof | dtantsur, however some info can't really be cached so we cache it just for a single REST call | 10:47 |
| dtantsur | mmm, I need to take a deeper look, I guess | 10:48 |
| etingof | let me push a revealing docstring | 10:48 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: WIP: Add memoization to emulator connections https://review.openstack.org/612758 | 10:50 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: WIP: Add memoization to emulator connections https://review.openstack.org/612758 | 10:56 |
| * etingof tried to explain what's going on in the commit message and in the memoize.py for dtantsur to -2 it right away ;) | 10:56 | |
| etingof | power status call via sushy-tools now takes ~2 secs on average on rdocloud | 10:59 |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add foundation for supporting Redfish OEMs https://review.openstack.org/463829 | 11:06 |
| *** adrianc_ has quit IRC | 11:07 | |
| *** adrianc has quit IRC | 11:07 | |
| *** adrianc_ has joined #openstack-ironic | 11:10 | |
| *** adrianc has joined #openstack-ironic | 11:10 | |
| jroll | morning ironic | 11:15 |
| jroll | fyi, I don't think I mentioned it, but I'm out tomorrow to monday | 11:15 |
| iurygregory | o/ | 11:15 |
| *** moshele has quit IRC | 11:16 | |
| dtantsur | morning jroll | 11:17 |
| *** adrianc_ has quit IRC | 11:29 | |
| *** adrianc has quit IRC | 11:29 | |
| openstackgerrit | Merged openstack/sushy master: Requests session keyword arguments for sushy connector https://review.openstack.org/607809 | 11:32 |
| *** litao has quit IRC | 11:34 | |
| openstackgerrit | Merged openstack/sushy master: Introduce ``cache_it`` and ``cache_clear`` https://review.openstack.org/581645 | 11:46 |
| *** moshele has joined #openstack-ironic | 11:56 | |
| *** rh-jelabarre has joined #openstack-ironic | 11:57 | |
| *** dnuka has joined #openstack-ironic | 11:58 | |
| *** adrianc has joined #openstack-ironic | 11:58 | |
| *** trown|outtypewww is now known as trown | 11:59 | |
| *** hjensas has quit IRC | 12:10 | |
| *** serlex has quit IRC | 12:14 | |
| openstackgerrit | Merged openstack/ironic master: Stop console at tearing down without unsetting console_enabled https://review.openstack.org/608175 | 12:20 |
| openstackgerrit | Merged openstack/ironic master: Add api-ref for conductor group https://review.openstack.org/611415 | 12:21 |
| *** moshele has quit IRC | 12:26 | |
| *** moshele has joined #openstack-ironic | 12:27 | |
| TheJulia | braaains | 12:37 |
| openstackgerrit | Digambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver https://review.openstack.org/609982 | 12:38 |
| *** jrist has quit IRC | 12:40 | |
| iurygregory | morning TheJulia o/ | 12:42 |
| TheJulia | dtantsur: I feel like now-ish might be good time to cut a sushy release :) | 12:42 |
| TheJulia | well, etings patch would be good | 12:44 |
| * etingof is working on the session reuse patch to sushy | 12:45 | |
| openstackgerrit | Matt Riedemann proposed openstack/ironic master: Revert "Add openstack/placement as a required project for ironic-grenade*" https://review.openstack.org/613007 | 12:47 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix DHCPv6 support https://review.openstack.org/611600 | 12:48 |
| TheJulia | dtantsur: ^^^ just reno fix | 12:48 |
| dtantsur | cool, lemme finish my tea | 12:49 |
| dtantsur | and g'morning | 12:49 |
| * TheJulia looks at her list of meetings and wonders if if there will be any productivity before lunch | 12:50 | |
| TheJulia | iurygregory: since 612869 is just minor clean up, perhaps you could edit it to include what you found? | 12:52 |
| iurygregory | TheJulia, sure o/ | 12:52 |
| TheJulia | awesome | 12:53 |
| * TheJulia has a strong desire to go back to bed | 12:53 | |
| iurygregory | TheJulia, I can keep the same commit message right? | 12:57 |
| TheJulia | iurygregory: yeah, just edit the change | 12:57 |
| TheJulia | and ammend the commit | 12:57 |
| openstackgerrit | Iury Gregory Melo Ferreira proposed openstack/ironic master: Remove pywsman reference https://review.openstack.org/612869 | 12:57 |
| iurygregory | done =) | 12:58 |
| *** gabys has joined #openstack-ironic | 12:58 | |
| *** sthussey has joined #openstack-ironic | 12:59 | |
| *** bnemec has joined #openstack-ironic | 13:04 | |
| *** dtantsur has quit IRC | 13:04 | |
| *** dtantsur has joined #openstack-ironic | 13:09 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Support for protecting nodes from undeploying and rebuilding https://review.openstack.org/611662 | 13:10 |
| TheJulia | dtantsur: I commented on your questions on https://review.openstack.org/#/c/606199 hopefully that provides a little more clarity | 13:12 |
| patchbot | patch 606199 - ironic-specs - A fresh way of looking at step retrieval - 4 patch sets | 13:12 |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add API tests for protected nodes https://review.openstack.org/613014 | 13:14 |
| dtantsur | okie | 13:14 |
| *** gabys has quit IRC | 13:16 | |
| *** gabys has joined #openstack-ironic | 13:17 | |
| openstackgerrit | Dmitry Galkin proposed openstack/ironic master: Fix node exclusive lock not released on console start/restart. https://review.openstack.org/603812 | 13:20 |
| *** gabys has quit IRC | 13:22 | |
| *** jcoufal has joined #openstack-ironic | 13:23 | |
| *** rloo has joined #openstack-ironic | 13:32 | |
| *** adrianc has quit IRC | 13:34 | |
| *** adrianc has joined #openstack-ironic | 13:34 | |
| TheJulia | useful IPA reviews for anyone: https://review.openstack.org/#/c/605707/ https://review.openstack.org/592639 | 13:36 |
| patchbot | patch 605707 - ironic-python-agent - Allow streaming raw partition images - 3 patch sets | 13:36 |
| patchbot | patch 592639 - ironic-python-agent - Fix multi-device behavior - 5 patch sets | 13:37 |
| *** trown is now known as trown|brb | 13:38 | |
| *** moshele has quit IRC | 13:47 | |
| *** dnuka has quit IRC | 13:48 | |
| *** mjturek has joined #openstack-ironic | 13:49 | |
| *** baha has joined #openstack-ironic | 13:49 | |
| openstackgerrit | Merged openstack/ironic-python-agent master: Follow up to parallel disk erasure https://review.openstack.org/612874 | 13:51 |
| *** trown|brb is now known as trown | 13:59 | |
| *** moshele has joined #openstack-ironic | 14:05 | |
| *** jrist has joined #openstack-ironic | 14:06 | |
| *** jrist has quit IRC | 14:09 | |
| *** jrist has joined #openstack-ironic | 14:09 | |
| *** UForgotten has joined #openstack-ironic | 14:19 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Sort columns in node detailed list alphabetically https://review.openstack.org/613045 | 14:23 |
| *** jrist has quit IRC | 14:24 | |
| *** moshele has quit IRC | 14:24 | |
| dtantsur | TheJulia: you asked for this ^^ | 14:29 |
| dtantsur | (will conflict with the automated_clean patch currently in flight) | 14:29 |
| iurygregory | sanity reasons wow | 14:30 |
| TheJulia | dtantsur: i was just thinking it would be nice :) | 14:30 |
| dtantsur | heh | 14:31 |
| TheJulia | I can rebase after the merge :) | 14:31 |
| dtantsur | TheJulia: no worries, I'll have to rebase it anyway because of my next patch (with protected and protected_reason) | 14:31 |
| TheJulia | I really want to cut an ipa and sushy release this week.. I guess we could do an initial client release soon() | 14:32 |
| dtantsur | ++ | 14:33 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Support for protected and protected_reason fields https://review.openstack.org/613053 | 14:34 |
| openstackgerrit | Merged openstack/python-ironicclient master: Add management of automated_clean field https://review.openstack.org/586508 | 14:37 |
| dtantsur | yay, time for rebase | 14:38 |
| openstackgerrit | Digambar proposed openstack/ironic master: Fix CPU count returned by introspection in Ironic iDRAC driver https://review.openstack.org/609982 | 14:39 |
| TheJulia | \o/ | 14:39 |
| *** jrist has joined #openstack-ironic | 14:41 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Sort columns in node detailed list alphabetically https://review.openstack.org/613045 | 14:42 |
| *** skazi has joined #openstack-ironic | 14:47 | |
| openstackgerrit | Ilya Etingof proposed openstack/sushy master: WIP: Allow HTTP session info to be set on authenticator https://review.openstack.org/613062 | 14:54 |
| openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Support for protected and protected_reason fields https://review.openstack.org/613053 | 14:54 |
| *** gabys has joined #openstack-ironic | 15:11 | |
| dtantsur | rloo: hi! I cannot find any traces of any code that updates object versions, so it seems okay (?) that they stayed on Pike. Now I wonder what blows up in dbsync.. | 15:14 |
| *** gyee has joined #openstack-ironic | 15:15 | |
| rloo | dtantsur: the code that updates object versions is in the dbsync stuff (or normally). i'm assuming here, that we're talking about the dbsync path. was it reproducible (i suspect yes). | 15:16 |
| rpioso | Good morning, ironicers | 15:16 |
| dtantsur | rloo: yeah, I got an environment that went Newton->..->Queens. Node versions are 1.21 (Pike). | 15:17 |
| dtantsur | rloo: our code seems to tolerate any versions actually. I'm not sure if that's the original idea or not, but currently we do not upgrade versions. | 15:21 |
| rloo | dtantsur: so in ironic Queens, run ironic-dbsync online_data_migrations fine. After that, run ironic-dbsync upgrade and it fails? | 15:21 |
| dtantsur | rloo: yeah, it fails on upgrade to Rocky | 15:21 |
| rloo | dtantsur: the code tolerates versions w/o a release. | 15:21 |
| rloo | w/i i mean. | 15:21 |
| rloo | wondering if the logic is incorrect there... | 15:21 |
| *** Emine has quit IRC | 15:22 | |
| dtantsur | rloo: I don't think so. | 15:22 |
| dtantsur | rloo: this collects versions from all known releases: https://github.com/openstack/ironic/blob/master/ironic/common/release_mappings.py#L187-L188 | 15:22 |
| rloo | dtantsur: right, if no release is specified. | 15:23 |
| dtantsur | rloo: we never specify it: https://github.com/openstack/ironic/search?utf8=%E2%9C%93&q=get_object_versions&type= | 15:23 |
| rpioso | dtantsur: I'm trying to better understand Depends-On and requirements.txt. Does https://review.openstack.org/609982 need the changes for those that were suggested? | 15:25 |
| patchbot | patch 609982 - ironic - Fix CPU count returned by introspection in Ironic ... - 9 patch sets | 15:25 |
| rpioso | The python-dracclient change that it depends on merged almost two months ago, and a new release of the library containing that dependency was released. It seems to me that Depends-On is not needed. Bumping the library version in requirements.txt is less clear to me. | 15:26 |
| rloo | dtantsur: ugh. i need to think about this. i see that when we implemented this, node was in v 1.21. | 15:26 |
| dtantsur | rpioso: depends-on does not work with libraries at all. and I think stuff like dracclient is not in u-c, so just the latest version is used. | 15:27 |
| rloo | implemented in pike. odd, don't we test the upgrade in grenade? | 15:27 |
| dtantsur | rloo: such problems will never show up in r->r+1 upgrades. you need r->r+1->r+2 | 15:27 |
| dtantsur | like, p->q->r in our case | 15:27 |
| rloo | dtantsur: or 'early' queens (v9.2). | 15:28 |
| rloo | the reason why grenade works is cuz it is queens v10.1 -> rocky 11.0, 11.1. guess we can't change it to test queens 9.2 -> rocky? | 15:32 |
| dtantsur | yeah, I doubt it | 15:33 |
| rpioso | dtantsur: Oh! I had no idea that depends-on doesn't work with libraries. I thought it's a Zuul thing for expressing cross-repository dependencies at merge time. I didn't realize it doesn't apply it to library repos. | 15:34 |
| rloo | dtantsur: i don't have a test env right now and would need to recall what i did. Let me look into it for an hour or so. is there a story about it? | 15:34 |
| dtantsur | rpioso: it only applies to stuff that is consumed from git (services, etc), but not from pypi (libraries) | 15:35 |
| dtantsur | rloo: not yet, I'm trying to understand what is going on first | 15:36 |
| dtantsur | rloo: https://bugzilla.redhat.com/show_bug.cgi?id=1641653 has some information | 15:37 |
| openstack | bugzilla.redhat.com bug 1641653 in openstack-tripleo-heat-templates "[UPGRADES][14] Failed to upgrade uc post ffwd: The database is not compatible with this release of ironic" [High,New] - Assigned to dtantsur | 15:37 |
| rloo | dtantsur: my theory is that the dbsync online-data-migrations code (from queens) passes. then you try running dbsync upgrade (from rocky -- at which point we've deleted v9.2 from release mappings) fails. | 15:38 |
| rloo | dtantsur: so the online-d-m code needs to upgrade to the latest version that is supported, instead of being complacent and not doing anything if an object version is in the 'accepted' list of versions. | 15:38 |
| dtantsur | rloo: we never deleted any release | 15:39 |
| rloo | dtantsur: we should, didn't we? let me look at release mappings... | 15:39 |
| dtantsur | rloo: we were supposed to, I think. But I have this file open, and it lists 9.2.. | 15:40 |
| dtantsur | these are supposed versions on queens: {'Node': set(['1.23', '1.22', '1.21']), 'Conductor': set(['1.2']), 'Trait': set(['1.0']), 'VolumeTarget': set(['1.0']), 'Port': set(['1.6', '1.7']), 'VolumeConnector': set(['1.0']), 'Chassis': set(['1.3']), 'Portgroup': set(['1.3']), 'TraitList': set(['1.0'])} | 15:41 |
| rpioso | dtantsur: I found some documentation about pypi at https://docs.openstack.org/infra/manual/developers.html#limitations-and-caveats | 15:42 |
| rpioso | That discusses gate testing, but not merge limitations. | 15:42 |
| dtantsur | rpioso: the patch in question is okay. note that it's not backportable because of the version dump though. | 15:44 |
| *** gabys has quit IRC | 15:44 | |
| dtantsur | if you want to backport it, you may want to go a more complex way and make it compatible with all versions | 15:44 |
| *** gabys has joined #openstack-ironic | 15:44 | |
| rloo | dtantsur: my first question, is are we supposed to support/be able to upgrade all versions of ironic in a particular dev cycle, to any/all versions in the next dev cycle. | 15:45 |
| dtantsur | rloo: well, in theory. in practice it's completely untested and unlikely anybody ever did it. | 15:45 |
| dtantsur | but we should not fail hard in such case for sure, since downgrades are also not possible. | 15:46 |
| rloo | dtantsur: yours is the first case, so I guess that is good... ? | 15:46 |
| rloo | good that we haven't broken anyone yet before this. that we know of... | 15:46 |
| dtantsur | well, we're talking about final releases here | 15:46 |
| dtantsur | what is also confusing here is that 1.21 was probably the versin for both 9.1 (Pike) and 9.2 (Queens) | 15:47 |
| dtantsur | (that's the only explanation I have) | 15:47 |
| rpioso | dtantsur: I'm still trying to understand depends-on. Does Zuul merge processing honor it for libraries? I understand its gate testing doesn't if the library is obtained from pypi. | 15:48 |
| rloo | dtantsur: right, i think that's the problem. i think we can fix it. just want to remember what we're supporting... | 15:48 |
| dtantsur | rpioso: for merging - yes. it won't merge code until the dependency is merged. | 15:49 |
| *** gabys has quit IRC | 15:49 | |
| *** serlex has joined #openstack-ironic | 15:49 | |
| *** pcaruana has quit IRC | 15:49 | |
| rpioso | dtantsur: Cool! For that patch, me thinks it's not needed, because the dependency merged so looooong ago. | 15:50 |
| dtantsur | correct | 15:50 |
| rloo | dtantsur: i am reading this, and it doesn't say anything about semvers (except when upgrading to master). https://specs.openstack.org/openstack/ironic-specs/specs/9.1/support-rolling-upgrade.html#rolling-upgrades-between-releases | 15:50 |
| dtantsur | let's put this aside for a second. this is a curious topic, but we need to solve the upgrade between named releases first.. | 15:51 |
| dtantsur | doing too much makes my head hurt | 15:51 |
| rloo | dtantsur: i think they're connected. | 15:51 |
| dtantsur | maybe? currently the problem is that we do not upgrade versions beyond the initial 1.21 | 15:52 |
| dtantsur | and that means that P->Q->R cannot work (nor can Q->R->S) | 15:52 |
| rloo | dtantsur: if we only allow upgrades between named releases, then for rocky, we shouldn't have 9.2 and 10 in the release_mapping, and for queens, we shouldn't have 8 & 9. | 15:52 |
| dtantsur | I think we do allow these | 15:53 |
| rpioso | Regarding requirements.txt, are you saying it's okay to bump it to 1.5? You reminded me of the details of a prior chat we had about requirements.txt and u-c. AFAIU, bumping it in requirements.txt is not enforced if it's not in u-c. It sort of documents the dependency, but that would be better placed in, well, documentation. | 15:53 |
| dtantsur | rpioso: you have to dump it, driver-requirements is also kind of documentation. and you cannot dump it on stable branches. | 15:54 |
| rloo | dtantsur: so you think we should allow an upgrade from 9.2 (during queens cycle, but not queens 10.1 release), to 9.2, 10 or 10.1 (all releases during rocky) | 15:54 |
| rloo | oop, sorry to 11 & 11.1 (rocky) | 15:54 |
| dtantsur | rloo: I'd say any queens release to the final rocky release | 15:55 |
| dtantsur | and from any queens release to the final queens release too? | 15:56 |
| rloo | dtantsur: ok. i think that's what i was trying to do when i coded. (any queens to final rocky) | 15:56 |
| dtantsur | so yeah, 9.2 to 10.1 or 11.1, but not to 10.0 | 15:56 |
| rloo | and yes, any queens release to final queens release, that's easier :) | 15:56 |
| rloo | oh, i think 9.2 to 10 works, at least 9.2 to master should work, and master becomes 10. | 15:57 |
| dtantsur | I think it works, yes. I'm saying what we should officially support. | 15:58 |
| *** munimeha1 has joined #openstack-ironic | 15:58 | |
| rloo | with grenade, we'll test the last named release (11.1 rocky) against master, right? but if we release eg 12 during stein, grenade is still 11.1 rocky against master, not 12. | 15:59 |
| dtantsur | correct | 15:59 |
| rloo | dtantsur: i'm going to disappear and look at the code... | 15:59 |
| dtantsur | sure. I'll probably go soon, but ping me anyway if you spot anything | 16:00 |
| dtantsur | I also still have an upgraded environment if you need to check smth | 16:00 |
| dtantsur | (though I may not be able to keep it all the time) | 16:01 |
| *** jcoufal_ has joined #openstack-ironic | 16:02 | |
| *** jcoufal has quit IRC | 16:03 | |
| rpioso | dtantsur: Thank you :) | 16:04 |
| *** gabys has joined #openstack-ironic | 16:04 | |
| *** hjensas has joined #openstack-ironic | 16:31 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Add API tests for protected nodes https://review.openstack.org/613014 | 16:34 |
| *** dtantsur is now known as dtantsur|afk | 16:46 | |
| *** trown is now known as trown|lunch | 16:46 | |
| *** pcaruana has joined #openstack-ironic | 16:46 | |
| *** dougsz has quit IRC | 16:59 | |
| *** serlex has quit IRC | 16:59 | |
| *** e0ne has quit IRC | 17:00 | |
| *** adrianc has quit IRC | 17:00 | |
| *** moshele has joined #openstack-ironic | 17:00 | |
| openstackgerrit | Debayan Ray proposed openstack/sushy master: Add foundation for supporting Redfish OEMs https://review.openstack.org/463829 | 17:06 |
| *** derekh has quit IRC | 17:07 | |
| *** irclogbot_3 has quit IRC | 17:08 | |
| *** irclogbot_3 has joined #openstack-ironic | 17:08 | |
| openstackgerrit | Ilya Etingof proposed openstack/ironic master: Reuse Redfish session tokens https://review.openstack.org/605408 | 17:15 |
| *** yolanda has quit IRC | 17:16 | |
| *** irclogbot_3 has quit IRC | 17:21 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up to fix not exist deploy image of patch 592247 https://review.openstack.org/612880 | 17:22 |
| openstackgerrit | Ilya Etingof proposed openstack/sushy master: Allow HTTP session info to be set on authenticator https://review.openstack.org/613062 | 17:30 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix node exclusive lock not released on console start/restart. https://review.openstack.org/603812 | 17:37 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Follow-up to I55a112d877d94f31d27487846ff59fe27f602f8b https://review.openstack.org/612867 | 17:37 |
| openstackgerrit | Bill Dodd proposed openstack/ironic master: WIP: Add BIOS interface to Redfish hardware type https://review.openstack.org/582314 | 17:37 |
| TheJulia | ooohhh ahhhh | 17:43 |
| rloo | dtantsur|afk: ouch, you are right: 11:14 dtantsur: rloo: hi! I cannot find any traces of any code that updates object versions, so it seems okay (?) that they stayed on Pike. Now I wonder what blows up in dbsync.. | 17:46 |
| rloo | dtantsur|afk: facepalm. I thought there was code to do that in the online_data_migrations, but there isn't :-(. I think we need to add a migration step, as the last step in the list, to always migrate to the latest version of an object. that should do it. | 17:47 |
| TheJulia | wow :( | 18:02 |
| * TheJulia goes and finds lunch | 18:05 | |
| *** trown|lunch is now known as trown | 18:13 | |
| openstackgerrit | Gabriela proposed openstack/sushy master: Add `ChassisCollection` and `Chassis` classes https://review.openstack.org/608177 | 18:28 |
| *** dsneddon__ has joined #openstack-ironic | 18:39 | |
| *** mmethot_ has joined #openstack-ironic | 18:40 | |
| *** jistr_ has joined #openstack-ironic | 18:42 | |
| *** chihhsin has joined #openstack-ironic | 18:43 | |
| *** jhesketh_ has joined #openstack-ironic | 18:44 | |
| *** hjensas has quit IRC | 18:48 | |
| *** jesusaur has quit IRC | 18:48 | |
| *** priteau has quit IRC | 18:48 | |
| *** raginbajin has quit IRC | 18:48 | |
| *** mmethot has quit IRC | 18:48 | |
| *** dsneddon has quit IRC | 18:48 | |
| *** zzzeek has quit IRC | 18:48 | |
| *** jistr has quit IRC | 18:48 | |
| *** SpamapS has quit IRC | 18:48 | |
| *** mcarden has quit IRC | 18:48 | |
| *** spotz has quit IRC | 18:48 | |
| *** radez has quit IRC | 18:48 | |
| *** JayF has quit IRC | 18:48 | |
| *** chihhsin_ has quit IRC | 18:48 | |
| *** jhesketh has quit IRC | 18:48 | |
| *** hjensas has joined #openstack-ironic | 18:49 | |
| *** jesusaur has joined #openstack-ironic | 18:49 | |
| *** SpamapS has joined #openstack-ironic | 18:49 | |
| *** e0ne has joined #openstack-ironic | 18:51 | |
| *** moshele has quit IRC | 18:51 | |
| *** spotz has joined #openstack-ironic | 18:52 | |
| *** openstackgerrit has quit IRC | 19:06 | |
| *** e0ne has quit IRC | 19:13 | |
| *** e0ne has joined #openstack-ironic | 19:18 | |
| *** irclogbot_3 has joined #openstack-ironic | 19:24 | |
| *** moshele has joined #openstack-ironic | 19:26 | |
| *** bfournie has quit IRC | 19:26 | |
| *** moshele has quit IRC | 19:29 | |
| *** gabys has quit IRC | 19:31 | |
| *** gabys has joined #openstack-ironic | 19:32 | |
| *** gabys has quit IRC | 19:36 | |
| *** e0ne has quit IRC | 19:39 | |
| *** gabys has joined #openstack-ironic | 19:39 | |
| *** gabys has quit IRC | 19:43 | |
| *** gabys has joined #openstack-ironic | 19:43 | |
| *** gabys has quit IRC | 19:48 | |
| *** openstackgerrit has joined #openstack-ironic | 19:51 | |
| openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add public resource loading and message parsing https://review.openstack.org/590747 | 19:51 |
| *** gabys has joined #openstack-ironic | 20:03 | |
| *** gabys has quit IRC | 20:04 | |
| *** gabys has joined #openstack-ironic | 20:04 | |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: Remove the xclarity deprecation https://review.openstack.org/613150 | 20:05 |
| *** gabys has quit IRC | 20:09 | |
| *** gabys has joined #openstack-ironic | 20:19 | |
| *** irclogbot_3 has quit IRC | 20:21 | |
| *** JasonF has joined #openstack-ironic | 20:27 | |
| openstackgerrit | Merged openstack/ironic master: Follow-up to fix not exist deploy image of patch 592247 https://review.openstack.org/612880 | 20:28 |
| *** gabys has quit IRC | 20:45 | |
| *** gabys has joined #openstack-ironic | 20:45 | |
| rajinir | TheJulia,rloo,rpioso, We have converted all our 3rd party CI to use Python3. Ipmi builds are throwing errors. Has anyone seen this? | 20:48 |
| rajinir | https://www.irccloud.com/pastebin/AKUgLA8b/ | 20:48 |
| TheJulia | rajinir: you can't convert all of them, we need to be sure that it still works for python2 on some jobs to until we pull python2 support | 20:48 |
| TheJulia | I wonder if we've just not released a python3 version of python-ironic-inspector-client | 20:49 |
| rajinir | TheJulia: Other builds are green. Only the ipmi driver builds are throwing this error. I can revert the ipmi ones to python2 for now | 20:49 |
| *** gabys has quit IRC | 20:50 | |
| *** pcaruana has quit IRC | 20:50 | |
| *** e0ne has joined #openstack-ironic | 20:52 | |
| *** e0ne has quit IRC | 20:52 | |
| *** jcoufal_ has quit IRC | 21:03 | |
| *** openstack has quit IRC | 21:03 | |
| *** openstack has joined #openstack-ironic | 21:06 | |
| *** ChanServ sets mode: +o openstack | 21:06 | |
| rpioso | rajinir: +1 | 21:11 |
| rpioso | TheJulia: ^^^ | 21:11 |
| TheJulia | rajinir: That makes sense, but please also have at least one idrac test on python2 | 21:12 |
| rajinir | TheJulia: ok, will do | 21:13 |
| rpioso | rajinir, TheJulia: I suggest we keep an idrac BIOS boot mode test build or two at Python 2. After all, the future is UEFI boot mode, Redfish, and Python 3. My two cents. | 21:14 |
| rajinir | rpioso: +1 | 21:15 |
| TheJulia | I think that is reasonable, boot mode differences shouldn't be huge | 21:15 |
| rpioso | TheJulia: And if there's an issue, it be discovered quickly :) | 21:16 |
| rpioso | s/it/it'll/ | 21:16 |
| NobodyCam | Good Afternoon Boot'ers | 21:38 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: ipv6 ipxe BFV?!? https://review.openstack.org/613169 | 21:46 |
| TheJulia | good afternoon NobodyCam | 21:47 |
| * TheJulia suspects the patch she just pushed will explode amazingly | 21:47 | |
| *** bnemec has quit IRC | 21:55 | |
| * rpioso is a UEFI booter | 21:56 | |
| rpioso | NobodyCam: Hey! | 21:57 |
| rpioso | :) | 21:57 |
| NobodyCam | Afternoon TheJulia and rpioso | 21:57 |
| openstackgerrit | Julia Kreger proposed openstack/ironic master: WIP: ipv6 ipxe BFV?!? https://review.openstack.org/613169 | 22:07 |
| *** rh-jelabarre has quit IRC | 22:10 | |
| *** ElCoyote_ has joined #openstack-ironic | 22:12 | |
| *** sai_p_ has joined #openstack-ironic | 22:17 | |
| *** vabada has quit IRC | 22:35 | |
| *** vabada has joined #openstack-ironic | 22:37 | |
| *** baha has quit IRC | 22:41 | |
| *** mjturek has quit IRC | 22:41 | |
| *** rcernin has joined #openstack-ironic | 23:05 | |
| *** openstackgerrit has quit IRC | 23:20 | |
| *** jesusaur has quit IRC | 23:34 | |
| *** sthussey has quit IRC | 23:40 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!