Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2020-07-24
19:43:44 dansmith cool
19:44:15 melwitt great, the WIP patch just failed
19:44:25 melwitt [errno 110] error connecting to the cluster wtf
19:46:27 dansmith maybe that config made it fail to start?
19:47:05 dansmith no ceph logs
19:47:54 dansmith hrm, don't even see it set the ini,
19:47:59 dansmith so maybe it didn't even get that far
19:50:44 melwitt yeah must be unless it's a fluke coincidence that ceph totally bombed this time
19:51:41 dansmith I think it must be a fluke bombing because it didn't run that line
19:53:01 dansmith ah here we go: 2020-07-24 18:32:36.045 | /opt/stack/devstack-plugin-ceph/devstack/lib/ceph: line 429: (24G: value too great for base (error token is "24G")
19:55:46 melwitt ah
19:55:59 dansmith hacky fix
20:05:20 dansmith melwitt: oops, should have put the bug on that, sorry
20:05:24 dansmith but it'll need cleanup
20:05:35 melwitt ah yeah
20:05:54 melwitt I was just thinking, I think all ceph jobs in openstack are broken over this, not just ours
20:06:50 dansmith potentially, but like I said, my job may be running slower or have different behaviors
20:07:26 dansmith actually
20:07:40 melwitt yeah. I was thinking of replying to mention other ceph jobs may be affected too. I see the older version 14.2.2 being pulled in openstack/tempest, for example
20:07:44 dansmith heh, I just saw a glance failure on the plain ceph job which is the same novalidhost
20:07:48 dansmith so I think yeah.
20:07:57 dansmith yeah
20:08:14 dansmith this is from one of my glance patches: https://e81e6b81331830d4903c-5acdef5dc10478cee5291df1596ec66a.ssl.cf1.rackcdn.com/742065/9/check/devstack-plugin-ceph-tempest-py3/292fd21/testr_results.html
20:09:17 melwitt ah yeah. and this is the tempest job failure I was looking at https://1ca2ee7583d21788b1d8-42b9b3ca9891e58d539431fcfb5b799d.ssl.cf2.rackcdn.com/742836/2/check/devstack-plugin-ceph-tempest-py3/547fab7/testr_results.html
20:09:27 melwitt (I picked a recent patch proposed to the tempest repo)
20:09:37 dansmith cool
20:09:56 dansmith that's awesome because of two things:
20:10:02 dansmith 1. I didn't break stuff
20:10:13 dansmith 2. I get to steal the credit from sean-k-mooney for fixing more people!
20:10:31 melwitt lol awww
20:10:45 dansmith dang, now I better give him honorable mention in the commit message ;P
20:11:08 melwitt hey, I found the default config value too
20:11:42 melwitt after that he blew past me finding how/where it was used
20:12:11 dansmith heh
20:12:51 dansmith he gave me a line to copy/paste, so...
20:13:44 melwitt yeah, that's what it's all about
#openstack-nova - 2020-07-25
03:08:32 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of hosts policies https://review.opendev.org/743013
03:20:06 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Introduce scope_types in hosts policy https://review.opendev.org/743014
04:10:41 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in hosts policies https://review.opendev.org/743015
05:02:45 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.PY2 and six.PY3 https://review.opendev.org/726512
05:03:06 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.add_metaclass https://review.opendev.org/726565
05:03:18 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.reraise https://review.opendev.org/726898
05:03:34 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.moves https://review.opendev.org/727224
05:03:48 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.iteritems/itervalues/iterkeys https://review.opendev.org/727757
05:12:37 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.byte2int/int2byte https://review.opendev.org/727777
05:13:03 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.binary_type/integer_types/string_types https://review.opendev.org/728094
05:13:24 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (1/2) https://review.opendev.org/728109
05:13:41 openstackgerrit Takashi Natsume proposed openstack/nova master: Remove six.text_type (2/2) https://review.opendev.org/728117
21:20:04 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add new default roles in hosts policies https://review.opendev.org/743015
21:55:51 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add test coverage of extensions policies https://review.opendev.org/743045
22:09:22 openstackgerrit Ghanshyam Mann proposed openstack/nova master: Add scope and new default roles in extensions policies https://review.opendev.org/743046
#openstack-nova - 2020-07-26
18:39:27 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-live-migration: Add encrypted volume to BFV evacuation test https://review.opendev.org/743067
#openstack-nova - 2020-07-27
01:53:15 openstackgerrit Keigo Noha proposed openstack/nova master: Change default num_retries for glance to 3 https://review.opendev.org/740389
07:52:32 bauzas good morning Nova
08:16:55 stephenfin o/
08:33:25 kashyap (Not that this kind of info be mentioned openly on a publicly logged channel. Where is my good sense...)
08:49:26 openstackgerrit Tony Su proposed openstack/nova master: Provider Config File: YAML file loading and schema validation https://review.opendev.org/673341
08:49:45 openstackgerrit Luyao Zhong proposed openstack/nova master: vpmem enhancement for resource cleanup https://review.opendev.org/714653
08:54:22 tony_su alex_xu: stephenfin: I updated the first patch for Provider Config File accordingly and answered all comments.. It would be grateful for you to spare some time to make a new round of review. Thanks.
08:54:55 tony_su I am going on the remaining patches ...
09:33:12 stephenfin tony_su: Hey o/ I've left a new review on that. Much better, but it still needs some work. Let me know if anything is unclear :)
09:57:16 stephenfin sean-k-mooney: did ye get to the bottom of the failing ceph-multistore job last week?
09:59:15 stephenfin lyarwood: Care to look at this again and possibly fast approve in gibi's absence (a trivial merge conflict resulted in loss of votes) https://review.opendev.org/#/c/742651/
10:03:30 brinzhang stephenfin: there is a patch about nova-cyborg internaction, hope you can review https://review.opendev.org/#/c/716186/
10:04:28 brinzhang stephenfin: gibi has +2 for that, and sean-k-mooney is one of the modifiers.
10:11:18 stephenfin brinzhang: I can try but realistically I know very little about Cyborg, so I don't know how much help I'll be. bauzas or similar might be a better shout
10:11:24 lyarwood stephenfin: ./me looks
10:12:55 brinzhang stephenfin: thanks, I will ask bauzas and dansmith give some help too.
10:14:06 brinzhang bauzas, dansmith: hope you can review the nova-cyborg interaction patch, and there is the cyborg operatcion support patch (evacuate, shelve/unshelve patch), thanks
10:16:21 lyarwood stephenfin: https://review.opendev.org/#/c/742961/ - re ceph-multistore, that's the fix.
10:25:34 sean-k-mooney stephenfin: yes we swapped to the bluestore backend and created a 10G backing file inside the normally backing file and were just running out of space
10:25:40 stephenfin lyarwood: thanks
10:26:22 sean-k-mooney stephenfin: so now we are setting the undocumented test only config option in ceph to set it to loopback device -4g
10:26:25 sean-k-mooney so 20G
10:26:32 sean-k-mooney that seam to solve the issue
10:27:24 stephenfin makes sense
10:27:25 sean-k-mooney brinzhang: ill review the cyborg patch again now
10:27:40 lyarwood sean-k-mooney: https://github.com/openstack/devstack/blob/e799770f59dfa81c78599bc4313503815b210b52/.zuul.yaml#L343 - any idea if we can move that into the ceph plugin .zuul.yaml?
10:28:26 lyarwood I'd like to add /etc/ceph/ to the list but adding it in devstack feels wrong
10:28:38 sean-k-mooney that is a good question i think i have done that before once sec
10:29:25 sean-k-mooney https://opendev.org/x/networking-ovs-dpdk/src/branch/master/.zuul.yaml#L27-L28
10:29:33 sean-k-mooney so yes i think we can do it like this
10:29:58 sean-k-mooney so it could be moved to the base ceph job instead
10:30:16 lyarwood cool thanks
10:31:02 brinzhang sean-k-mooney: cool, thanks
10:31:25 tosky lyarwood: iirc zuul_copy_output is a dictionary so you can extend it, but it makes sense to add that to devstack-plugin-ceph jobs
10:32:04 sean-k-mooney tosky: right you can add more keys but not remove keys set by parent jobs
10:32:26 lyarwood ack cool I'll add it and remove it from the original with a depends-on now
11:12:40 jsuchome lyarwood: hi, once you have time, please take a look at https://review.opendev.org/#/c/574301/ again, it's rebased on top of recently merged change
11:34:52 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-live-migration: Add encrypted volume to BFV evacuation test https://review.opendev.org/743067
11:35:32 lyarwood jsuchome: ack will do
11:36:54 openstackgerrit Lee Yarwood proposed openstack/nova stable/train: libvirt: Handle VIR_ERR_DEVICE_MISSING when detaching devices https://review.opendev.org/742415
11:46:30 sean-k-mooney brinzhang_: i thinik the current iteration of https://review.opendev.org/#/c/716186/17 is good
11:47:09 sean-k-mooney bauzas: dansmith can ye take a look if ye have time ^
12:09:55 openstackgerrit Merged openstack/nova master: scheduler: Default request group to None https://review.opendev.org/742651
12:12:24 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Further usage of new server helpers https://review.opendev.org/743204
12:16:17 bauzas sean-k-mooney: ack, will try
12:16:24 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add type hints for 'nova.compute.api' https://review.opendev.org/741623
12:16:25 openstackgerrit Stephen Finucane proposed openstack/nova master: tests: Duplicate '_test_resize' to '_test_migrate' https://review.opendev.org/743206
12:16:25 openstackgerrit Stephen Finucane proposed openstack/nova master: api: Add 'nova.compute.API.migrate' https://review.opendev.org/741624

Earlier   Later