Thursday, 2020-10-29

*** tosky has quit IRC00:06
*** hamalq has quit IRC00:18
*** zzzeek has quit IRC00:42
*** zzzeek has joined #openstack-oslo00:45
*** brinzhang_ has joined #openstack-oslo01:31
*** brinzhang has quit IRC01:33
openstackgerritMerged openstack/os-api-ref master: Bump py37 to py38 in tox.ini  https://review.opendev.org/76016101:39
*** johnsom has quit IRC02:00
*** rpittau|afk has quit IRC02:00
*** TheJulia has quit IRC02:00
*** rpittau|afk has joined #openstack-oslo02:00
*** TheJulia has joined #openstack-oslo02:00
*** johnsom has joined #openstack-oslo02:01
*** vdrok has quit IRC02:02
*** vdrok has joined #openstack-oslo02:03
openstackgerritMerged openstack/oslo.policy master: Fix hacking min version to 3.0.1  https://review.opendev.org/75250402:32
*** dave-mccowan has quit IRC04:12
*** ralonsoh has joined #openstack-oslo06:47
*** sboyron has joined #openstack-oslo07:18
*** rcernin has quit IRC07:31
*** lbragstad has quit IRC08:04
*** lbragstad has joined #openstack-oslo08:04
*** hberaud has joined #openstack-oslo08:07
*** hberaud has quit IRC08:18
*** hberaud has joined #openstack-oslo08:19
*** rpittau|afk is now known as rpittau08:40
*** tosky has joined #openstack-oslo08:51
*** dtantsur|afk is now known as dtantsur08:58
*** rcernin has joined #openstack-oslo09:18
*** hberaud has quit IRC09:30
*** hberaud has joined #openstack-oslo09:32
*** dtantsur is now known as dtantsur|brb09:44
*** sum12 has quit IRC09:50
*** rcernin has quit IRC10:07
*** dhellmann has quit IRC10:23
*** zzzeek has quit IRC10:28
*** zzzeek has joined #openstack-oslo10:29
openstackgerritStephen Finucane proposed openstack/oslo.messaging stable/train: add min of 1 to rpc_conn_pool_size  https://review.opendev.org/75941010:42
openstackgerritMerged openstack/os-api-ref master: bump py37 to py38 in tox.ini  https://review.opendev.org/75742910:43
openstackgerritMerged openstack/os-api-ref master: Remove install unnecessary packages  https://review.opendev.org/75351210:45
*** rcernin has joined #openstack-oslo11:15
*** dhellmann has joined #openstack-oslo11:18
*** hberaud has quit IRC11:27
*** dave-mccowan has joined #openstack-oslo11:44
*** dave-mccowan has quit IRC11:48
*** hberaud has joined #openstack-oslo11:56
openstackgerritlikui proposed openstack/oslo.limit master: Update requirements URLs in tox config  https://review.opendev.org/76034511:56
*** dtantsur|brb is now known as dtantsur11:57
openstackgerritlikui proposed openstack/oslo.limit master: Update URLs in tox docs  https://review.opendev.org/76034512:04
openstackgerritMerged openstack/oslo.concurrency stable/ussuri: Monkey patch original current_thread _active in processutils  https://review.opendev.org/75702912:11
*** raildo has joined #openstack-oslo12:27
*** ianychoi__ is now known as ianychoi12:34
*** Luzi has joined #openstack-oslo12:53
*** rcernin has quit IRC13:43
*** hamalq has joined #openstack-oslo14:41
*** tkajinam has quit IRC15:05
*** takamatsu has quit IRC15:07
*** takamatsu has joined #openstack-oslo15:09
openstackgerritStephen Finucane proposed openstack/oslo.config master: types: Cleanup implementation of 'HostDomain' type  https://review.opendev.org/76038715:23
stephenfinhberaud: Can you take a look at that? ^15:23
stephenfinMaybe bnemec too? ^15:24
stephenfinAssuming I have that understood correctly, I'd like to backport that too. The implementation we have right now doesn't look correct (I don't think we want to allow IP addresses?)15:24
bnemecstephenfin: It doesn't allow ips, does it? The unit tests involving ips are all assertInvalids.15:28
stephenfinbnemec: They're invalid IPs15:28
bnemecNot all of them: self.assertInvalid("10.0.0.0")15:29
hberaudyep we expect ip too15:29
hberaudIIUC IPs will be be skipped, isn't?15:29
hberaud(by using this version of HostDomain)15:30
stephenfinbnemec: Sorry, crossed wires. I thought you were asking if the old version allowed IPs, and it does (valid ones)15:31
stephenfinand hberaud is saying that's intentional?15:31
bnemecAh, that doesn't seem right. The only change in behavior was supposed to be support for _.15:33
stephenfinWell, it depends on whether we're providing a new variant of the 'Hostname' type (hostnames only) or the 'HostAddress' type (hostnames or IP addresses)15:34
stephenfinI'd thought the request was for the former. If it's the latter, what's there is still a bit messy but is correct at least15:34
hberaudI think we want HostAddress with an extended hostname15:35
hberaudas user can pass IPs and extended hosts15:35
stephenfinhberaud: what was the use case for this again? Something specific, I assume?15:35
hberaudsec15:36
smcginnisI thought the main thing was being able to have "_" in the hostname.15:36
smcginnisWhich I still am not clear why we need a whole new class for that.15:36
stephenfinsmcginnis: Yeah, so did I15:36
stephenfinsmcginnis: https://tools.ietf.org/html/rfc191215:37
stephenfin"The presence15:37
stephenfin   of underscores in a label is allowed in [RFC 1033], except [RFC 1033]15:37
stephenfin   is informational only and was not defining a standard.  There is at15:37
stephenfin   least one popular TCP/IP implementation which currently refuses to15:37
stephenfin   talk to hosts named with underscores in them."15:37
stephenfinIt seems our 'Hostname' implementation is tied to the RFC 1033 definition15:37
stephenfinthe linux manpage for hostname(7) also suggests this implementation is correct https://man7.org/linux/man-pages/man7/hostname.7.html15:38
hberaudstephenfin: that was the original bug => https://bugzilla.redhat.com/show_bug.cgi?id=186894015:38
openstackbugzilla.redhat.com bug 1868940 in python-oslo-config "oslo_config not accepting undercore in the hostnames" [Medium,On_dev] - Assigned to mbollo15:38
stephenfinSo I do agree with bnemec that it should be a separate type, because there are clearly some things that say valid hostnames should _not_ include underscores15:39
hberaud+115:39
moguimardamani, did we get the oslo.utils release?16:02
hberaudmoguimar: yes https://review.opendev.org/#/c/760091/16:05
hberaudthanks damani16:05
moguimarcool, thanks guys o/16:05
hberaudu r welcome :)16:05
openstackgerritAde Lee proposed openstack/oslo.versionedobjects master: Replace md5 with oslo version  https://review.opendev.org/75615316:21
*** sboyron has quit IRC16:40
*** Luzi has quit IRC16:46
damanimoguimar, sorry yes16:53
*** rpittau is now known as rpittau|afk16:57
*** hamalq has quit IRC17:13
*** hamalq has joined #openstack-oslo17:14
*** ralonsoh has quit IRC17:15
openstackgerritAde Lee proposed openstack/oslo.versionedobjects master: Replace md5 with oslo version  https://review.opendev.org/75615317:31
openstackgerritRon Stone proposed openstack/openstackdocstheme master: Added styling for rST rubric directive.  https://review.opendev.org/76042517:35
*** dtantsur is now known as dtantsur|afk18:18
*** tosky has quit IRC18:29
*** hberaud has quit IRC19:00
*** ccamel has quit IRC19:35
openstackgerritAde Lee proposed openstack/tooz master: Replace md5 with oslo version  https://review.opendev.org/75643219:45
*** hberaud has joined #openstack-oslo19:54
*** zzzeek has quit IRC20:32
*** zzzeek has joined #openstack-oslo20:33
*** kgiusti has quit IRC20:46
*** kgiusti has joined #openstack-oslo20:47
*** camelCaser has joined #openstack-oslo21:13
*** kgiusti has left #openstack-oslo21:38
*** tosky has joined #openstack-oslo21:57
*** hberaud has quit IRC22:47
*** hamalq has quit IRC22:57
*** rcernin has joined #openstack-oslo22:58
*** rcernin has quit IRC23:00
*** rcernin has joined #openstack-oslo23:00

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