Tuesday, 2021-08-10

*** rpittau|afk is now known as rpittau06:15
opendevreviewHervé Beraud proposed openstack/oslo.cache master: Add socket keepalive creation from config  https://review.opendev.org/c/openstack/oslo.cache/+/80371906:28
opendevreviewHervé Beraud proposed openstack/oslo.cache master: Add retry mechanisms to oslo.cache  https://review.opendev.org/c/openstack/oslo.cache/+/80374706:28
opendevreviewHervé Beraud proposed openstack/oslo.cache master: Add retry context creation from config  https://review.opendev.org/c/openstack/oslo.cache/+/80374906:28
*** mtreinish_ is now known as mtreinish06:40
opendevreviewMerged openstack/oslo.reports master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/oslo.reports/+/79472609:44
opendevreviewMerged openstack/oslo.messaging master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/oslo.messaging/+/79471609:48
opendevreviewMerged openstack/oslo.i18n master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/oslo.i18n/+/79471709:49
opendevreviewMerged openstack/oslo.versionedobjects master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/oslo.versionedobjects/+/79473009:51
opendevreviewMerged openstack/oslo.i18n master: Switch testing to Xena testing runtime  https://review.opendev.org/c/openstack/oslo.i18n/+/79322809:51
opendevreviewMerged openstack/oslo.privsep master: Changed minversion in tox to 3.18.0  https://review.opendev.org/c/openstack/oslo.privsep/+/79472510:19
*** geguileo is now known as Guest381212:30
*** geguileor is now known as geguileo12:48
hberaudzzzeek: o/ do you mind to have another look to https://github.com/sqlalchemy/dogpile.cache/pull/205/13:20
zzzeekhberaud: yeah will get to taht tdy13:25
hberaudzzzeek: ty13:25
opendevreviewMerged openstack/oslo.log master: Replace deprecated arguments of RequestContext  https://review.opendev.org/c/openstack/oslo.log/+/80185514:27
opendevreviewMerged openstack/oslo.db master: tests: Clean up base test  https://review.opendev.org/c/openstack/oslo.db/+/80256314:28
opendevreviewMerged openstack/oslo.versionedobjects master: Do not patch inspect.argspec directly  https://review.opendev.org/c/openstack/oslo.versionedobjects/+/80101614:33
opendevreviewMerged openstack/oslo.versionedobjects master: Replace deprecated inspect.getargspec  https://review.opendev.org/c/openstack/oslo.versionedobjects/+/80092614:34
opendevreviewMerged openstack/oslo.db master: tests: Enfeeble 'oslo_db.tests.utils.BaseTestCase'  https://review.opendev.org/c/openstack/oslo.db/+/80256414:38
opendevreviewMerged openstack/oslo.db master: tests: Use common base class  https://review.opendev.org/c/openstack/oslo.db/+/80256514:38
opendevreviewMerged openstack/oslo.db master: tests: Enable SADeprecationWarning warnings  https://review.opendev.org/c/openstack/oslo.db/+/80256614:38
opendevreviewMerged openstack/oslo.db master: utils: Deprecate sqlalchemy-migrate-related functions  https://review.opendev.org/c/openstack/oslo.db/+/80256714:38
opendevreviewMerged openstack/oslo.db master: tests: Enable SQLAlchemy 2.0 deprecation warnings  https://review.opendev.org/c/openstack/oslo.db/+/80256814:38
opendevreviewMerged openstack/oslo.db master: Remove legacy calling style of select()  https://review.opendev.org/c/openstack/oslo.db/+/80256914:38
opendevreviewMerged openstack/oslo.db master: Remove use of MetaData.bind argument  https://review.opendev.org/c/openstack/oslo.db/+/80257014:39
opendevreviewMerged openstack/oslo.db stable/wallaby: Added handler for mysql 8.0.19 duplicate key error update  https://review.opendev.org/c/openstack/oslo.db/+/79876014:57
opendevreviewMerged openstack/oslo.db stable/wallaby: Followup of "Added handler for mysql 8.0.19 duplicate key error update"  https://review.opendev.org/c/openstack/oslo.db/+/79876314:57
*** ralonsoh_ is now known as ralonsoh15:22
*** rpittau is now known as rpittau|afk16:31
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't pass strings to Connection.execute()  https://review.opendev.org/c/openstack/oslo.db/+/80257116:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Add missing bind argument to calls  https://review.opendev.org/c/openstack/oslo.db/+/80257216:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of insert.values parameter  https://review.opendev.org/c/openstack/oslo.db/+/80257316:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of update.values parameter  https://review.opendev.org/c/openstack/oslo.db/+/80257416:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Update import of declarative_base()  https://review.opendev.org/c/openstack/oslo.db/+/80257516:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of update.whereclause parameter  https://review.opendev.org/c/openstack/oslo.db/+/80257616:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of Table.autoload parameter  https://review.opendev.org/c/openstack/oslo.db/+/80257716:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't use plain string SQL statements  https://review.opendev.org/c/openstack/oslo.db/+/80257816:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't use dict-style attribute accesses  https://review.opendev.org/c/openstack/oslo.db/+/80257916:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't use the 'Row.keys()' method  https://review.opendev.org/c/openstack/oslo.db/+/80258016:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of Engine.scalar()  https://review.opendev.org/c/openstack/oslo.db/+/80258116:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Remove unnecessary warning filter  https://review.opendev.org/c/openstack/oslo.db/+/80258216:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of Executable.execute method  https://review.opendev.org/c/openstack/oslo.db/+/80258316:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't pass kwargs to connection.execute()  https://review.opendev.org/c/openstack/oslo.db/+/80258416:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't call mapper() outside of declarative registry  https://review.opendev.org/c/openstack/oslo.db/+/80258516:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Replace use of 'Engine.execute()'  https://review.opendev.org/c/openstack/oslo.db/+/80258616:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Don't rely on implicit autocommit  https://review.opendev.org/c/openstack/oslo.db/+/80258716:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: Remove use of Session.begin.subtransactions flag  https://review.opendev.org/c/openstack/oslo.db/+/80258816:45
opendevreviewStephen Finucane proposed openstack/oslo.db master: WIP: tests: Enable SAWarning warnings  https://review.opendev.org/c/openstack/oslo.db/+/80258916:45
stephenfinzzzeek: Respun those patches to account for the changes to the MetaData.bind patch. A few might need you to revisit them due to (reasonably trivial) merge conflicts16:49
zzzeekstephenfin: will look16:51
zzzeekstephenfin: i dont see 'merge conflict" for any at https://review.opendev.org/q/topic:%22sqlalchemy-20%22+(status:open%20OR%20status:merged) should I ?16:53
stephenfinSorry, no, I meant I resolved the merge conflicts so you may need to re-review some patches that you'd previously +2s16:53
stephenfin*+2d16:53
zzzeekoh ok16:53
zzzeekstephenfin: some of these patches will only work with SQLAlhcemy 1.4 as min version17:00
zzzeekstephenfin: we aren't ready for that yet, right?17:00
zzzeekstephenfin: it's likely wise to keep the "1.3 compatible" and "not 1.3 compat" patches in a separate series17:00
stephenfinHmm, any reason not to other than the fact I haven't bumped the minimum version in requirements.txt yet?17:01
zzzeekstephenfin: taht's the main reason but also theres a global requirekments.txt somehwere right?  how does this relate to that ?17:25
zzzeekstephenfin: im not sure about upper constraints either, that seems to move around17:27
zzzeekstephenfin: good news 1.4 is at least in upper constraints now https://github.com/openstack/requirements/blob/master/upper-constraints.txt18:03
opendevreviewJohn Eckersberg proposed openstack/oslo.messaging master: Revert "Disable AMQP 1.0 SSL unit tests"  https://review.opendev.org/c/openstack/oslo.messaging/+/80367021:04
opendevreviewJohn Eckersberg proposed openstack/oslo.messaging master: amqp1: Do not reuse _socket_connection on reconnect  https://review.opendev.org/c/openstack/oslo.messaging/+/80350821:04
opendevreviewJohn Eckersberg proposed openstack/oslo.messaging master: amqp1: re-organize TestFailover to be reused by TestSSL  https://review.opendev.org/c/openstack/oslo.messaging/+/80414121:04

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