Monday, 2020-07-27

« Sunday, 2020-07-26 Index Tuesday, 2020-07-28 »
*** rcernin has quit IRC02:31
*** rcernin has joined #openstack-glance02:40
*** rcernin has quit IRC02:40
*** rcernin has joined #openstack-glance02:41
*** baojg has quit IRC03:39
*** baojg has joined #openstack-glance03:55
*** udesale has joined #openstack-glance04:07
*** ratailor has joined #openstack-glance04:07
*** m75abrams has joined #openstack-glance04:37
*** rcernin has quit IRC05:50
*** rcernin has joined #openstack-glance05:51
*** rosmaita has left #openstack-glance06:02
*** udesale has quit IRC06:08
*** udesale has joined #openstack-glance06:10
*** rcernin has quit IRC06:21
*** rcernin has joined #openstack-glance06:22
*** amoralej|off is now known as amoralej07:08
*** lpetrut has joined #openstack-glance07:10
*** rcernin has quit IRC07:28
*** ralonsoh has joined #openstack-glance07:53
*** nikparasyr has joined #openstack-glance08:25
*** rcernin has joined #openstack-glance08:30
*** rcernin has quit IRC08:35
*** rcernin has joined #openstack-glance09:25
*** rcernin has quit IRC09:30
*** tkajinam has quit IRC10:10
*** rcernin has joined #openstack-glance10:11
*** rcernin has quit IRC10:15
*** lpetrut has quit IRC10:17
*** baojg has quit IRC10:19
*** baojg has joined #openstack-glance10:21
openstackgerritLuigi Toscano proposed openstack/glance_store master: zuul: glance_store-src-ceph-tempest replaces a legacy job  https://review.opendev.org/74319110:54
*** rcernin has joined #openstack-glance11:23
*** rosmaita has joined #openstack-glance11:38
*** jungleboyj has joined #openstack-glance11:42
*** amoralej is now known as amoralej|lunch11:57
*** udesale_ has joined #openstack-glance12:00
*** udesale has quit IRC12:03
*** baojg has quit IRC12:10
*** baojg has joined #openstack-glance12:11
*** baojg has quit IRC12:26
*** evrardjp has quit IRC12:32
*** evrardjp has joined #openstack-glance12:32
*** evrardjp has quit IRC12:32
*** evrardjp has joined #openstack-glance12:33
*** baojg has joined #openstack-glance12:35
*** baojg has quit IRC12:48
*** baojg has joined #openstack-glance12:51
*** amoralej|lunch is now known as amoralej12:59
*** ratailor has quit IRC13:22
*** rcernin has quit IRC13:29
openstackgerritGhanshyam Mann proposed openstack/glance_store master: [goal] Migrate glance_store jobs to focal  https://review.opendev.org/74310913:33
abhishekksmcginnis, jokke, rosmaita kindly review when you are around, https://review.opendev.org/#/c/742810/13:35
abhishekkdansmith, around?14:01
dansmithwaiting on bluejeans14:01
rosmaitaabhishekk: start without me, will join when i can14:02
abhishekkrosmaita, ack14:03
*** lpetrut has joined #openstack-glance14:14
*** jv_ has joined #openstack-glance14:18
*** rcernin has joined #openstack-glance14:22
*** bhagyashris is now known as bhagyashris|away14:24
*** baojg has quit IRC14:34
*** hoonetorg has quit IRC14:35
*** baojg has joined #openstack-glance14:36
*** hoonetorg has joined #openstack-glance14:49
*** lpetrut has quit IRC14:49
*** rcernin has quit IRC14:56
*** m75abrams has quit IRC15:07
openstackgerritMerged openstack/glance master: Make glance-api able to do async tasks in WSGI mode  https://review.opendev.org/74206515:33
*** lpetrut has joined #openstack-glance15:42
openstackgerritGhanshyam Mann proposed openstack/glance_store master: [goal] Migrate glance_store jobs to focal  https://review.opendev.org/74310915:44
*** gyee has joined #openstack-glance15:47
*** nikparasyr has left #openstack-glance16:07
*** udesale_ has quit IRC16:21
*** rosmaita has quit IRC16:22
*** amoralej is now known as amoralej|off16:25
abhishekkdansmith, still around?16:53
dansmithyup16:53
abhishekkis it possible to remove depends on from patch https://review.opendev.org/#/c/742314/16:53
openstackgerritAbhishek Kekane proposed openstack/glance master: [Doc] Policy support to copy unowned images  https://review.opendev.org/74281016:54
*** rcernin has joined #openstack-glance16:54
dansmithabhishekk: yeah, but I just asked gmann if they could approve that dependent patch to avoid having to re-propose and wait for tests again16:55
abhishekkdansmith, that will also do16:55
abhishekkI just wanted to make sure this should be part of V2 milestone as I am going to tag it on Wednesday16:56
dansmithack, I'll just remove it, hang on16:56
abhishekkthanks16:56
openstackgerritDan Smith proposed openstack/glance master: Make image conversion use a proper python interpreter for prlimit  https://review.opendev.org/74231416:57
openstackgerritDan Smith proposed openstack/glance master: Make wsgi_app support graceful shutdown  https://review.opendev.org/74249316:57
openstackgerritDan Smith proposed openstack/glance master: DNM: Run tempest tests on WSGI stack  https://review.opendev.org/74288316:57
*** rcernin has quit IRC16:58
*** nicolasbock has joined #openstack-glance17:00
*** rosmaita has joined #openstack-glance17:03
*** ralonsoh has quit IRC17:23
*** ralonsoh has joined #openstack-glance17:23
*** baojg has quit IRC17:32
*** baojg has joined #openstack-glance17:34
*** jv_ has quit IRC17:36
gmanndansmith: web-download is import method not available and so test is skipped in https://review.opendev.org/#/c/742883/617:37
dansmithgmann: i just didn't enable it in that job17:38
dansmithgmann: if we unpatch the devstack change to null them out, I can remove that from the job17:39
dansmithgmann: https://review.opendev.org/#/c/742883/6/.zuul.yaml@24417:39
dansmithor we could fix the job17:39
dansmithI probably didn't put it in there because it didn't matter at the time and didn't want to type \" more times :)17:40
dansmithgmann: I have this up btw: https://review.opendev.org/#/c/742332/2/lib/glance17:41
dansmithto remove all these restrictions, which we can land soon as the glance wsgi stuff is mostly +2d17:41
gmanndansmith: ok,  lgtm, as we enable the import tests with GLANCE_USE_IMPORT_WORKFLOW flag so enabling import method for uwsgi is safe17:48
dansmithgmann: do we need to? once we merge the wsgi statck they can run in any config, so we should go based on the advertised methods via the api right?17:50
gmanndansmith: once wsgi stack merged then no, but till that migration we need as tempest has test now17:51
gmanndansmith: also can you remove the method restriction from job to see all tests running  - https://review.opendev.org/#/c/74288317:52
dansmithack17:52
openstackgerritDan Smith proposed openstack/glance master: DNM: Run tempest tests on WSGI stack  https://review.opendev.org/74288317:53
dansmithgmann: ^17:53
gmannthanks17:53
dansmithabhishekk: when is image.size set? seems like it's None during an import17:54
abhishekkdansmith, its set once image is imported to backend17:55
dansmithshouldn't we know the size once we have staged?17:55
dansmithis there another property I can look at?17:55
abhishekkdansmith not at the moment17:56
dansmithokay17:56
abhishekkbut I am working on plugin which will set virtual size to image before importing it to backend17:57
abhishekkhttps://review.opendev.org/74112117:57
dansmithoklay17:57
*** ralonsoh has quit IRC18:08
dansmithabhishekk: so, um, I'm testing something with import to a file store18:12
dansmithusing a 500M image, stage, and then import to a file store18:12
abhishekkok18:12
dansmithit *seems* like glance tries to read() the whole image into memory and then write it18:12
dansmithsurely that can't be true right?18:12
abhishekkthis is how it is done18:15
dansmithactually, I'm not sure why this would be any different with a file store vs. rbd or something else because it seems like the chunk size is determined by the source, which is the file in the staging area18:15
dansmithabhishekk: so if I need to import a 1TB image, I need 1.2TB of memory on the API worker?18:15
*** evrardjp has quit IRC18:21
abhishekkdansmith, it actually reads the file from the staging (i.e. it opens the file in memory) and then pass it to store18:21
dansmithabhishekk: right but the store is expecting an iterator, but it seems with a 500m file there is only ever one iteration, and a chunk of 500m in memory18:23
openstackgerritCyril Roelandt proposed openstack/glance master: Remove E711 and E712 from the ignorelist in tox.ini  https://review.opendev.org/74330218:24
dansmithokay I see where get_image_data_iter() is just returning open() for the file:// which I assume is the staging dir right?18:24
*** evrardjp has joined #openstack-glance18:24
abhishekkyes18:25
dansmithokay I see what's happening... open() is doing some input conversion and will read zeroed blocks as a single chunk18:27
*** lpetrut has quit IRC18:33
* abhishekk signing out for the day18:39
openstackgerritCyril Roelandt proposed openstack/python-glanceclient master: Remove F403, F812 and F821 from the ignorelist in tox.ini  https://review.opendev.org/74331518:47
*** baojg has quit IRC18:59
*** baojg has joined #openstack-glance19:00
*** jv_ has joined #openstack-glance19:34
*** rcernin has joined #openstack-glance21:53
*** rcernin has quit IRC21:58
*** rcernin has joined #openstack-glance21:59
openstackgerritMerged openstack/glance master: Make image conversion use a proper python interpreter for prlimit  https://review.opendev.org/74231422:01
openstackgerritMerged openstack/glance master: Make wsgi_app support graceful shutdown  https://review.opendev.org/74249322:04
openstackgerritDan Smith proposed openstack/glance master: WIP: Heartbeat the actual work of the task  https://review.opendev.org/74342622:07
openstackgerritDan Smith proposed openstack/glance master: WIP: Update task message during import  https://review.opendev.org/74342722:07
*** tkajinam has joined #openstack-glance22:52
*** hoonetorg has quit IRC23:06
*** hoonetorg has joined #openstack-glance23:19
« Sunday, 2020-07-26 Index Tuesday, 2020-07-28 »

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