| *** dims has joined #openstack-glance | 00:01 | |
| *** obedmr has quit IRC | 00:06 | |
| *** chlong has quit IRC | 00:08 | |
| *** dramakri has quit IRC | 00:12 | |
| *** r-daneel has quit IRC | 00:13 | |
| openstackgerrit | Wayne Okuma proposed openstack/glance-specs: CLI Support For Tag Metadata Definitions https://review.openstack.org/179674 | 00:17 |
|---|---|---|
| *** dramakri has joined #openstack-glance | 00:20 | |
| *** dramakri has quit IRC | 00:21 | |
| *** tsekiyama has quit IRC | 00:26 | |
| *** ducttape_ has joined #openstack-glance | 00:31 | |
| *** ducttape_ has quit IRC | 00:40 | |
| *** dims has quit IRC | 00:50 | |
| *** dims has joined #openstack-glance | 00:58 | |
| *** MentalRay has quit IRC | 01:10 | |
| *** jasondotstar has quit IRC | 01:12 | |
| *** ducttape_ has joined #openstack-glance | 01:14 | |
| *** annashen has joined #openstack-glance | 01:17 | |
| *** Longgeek has joined #openstack-glance | 01:18 | |
| openstackgerrit | wangxiyuan proposed openstack/glance: Add new function 'show' in version.py https://review.openstack.org/196575 | 01:21 |
| *** annashen has quit IRC | 01:22 | |
| *** mtanino has quit IRC | 01:22 | |
| *** jasondotstar has joined #openstack-glance | 01:25 | |
| *** chlong has joined #openstack-glance | 01:30 | |
| *** ducttape_ has quit IRC | 01:32 | |
| openstackgerrit | Fei Long Wang proposed openstack/python-glanceclient: Use API v2 as default https://review.openstack.org/193931 | 01:47 |
| *** Longgeek has quit IRC | 01:51 | |
| *** harshs has quit IRC | 01:51 | |
| *** mingdang1 has joined #openstack-glance | 01:56 | |
| *** ajayaa has joined #openstack-glance | 01:58 | |
| *** skoar has quit IRC | 02:03 | |
| *** arnaud____ has joined #openstack-glance | 02:13 | |
| *** skoar has joined #openstack-glance | 02:15 | |
| *** annashen has joined #openstack-glance | 02:18 | |
| *** arnaud____ has quit IRC | 02:21 | |
| *** jskinner has joined #openstack-glance | 02:22 | |
| *** annashen has quit IRC | 02:22 | |
| *** MentalRay has joined #openstack-glance | 02:23 | |
| *** spzala has quit IRC | 02:24 | |
| *** skoar has quit IRC | 02:42 | |
| openstackgerrit | Wayne Okuma proposed openstack/glance: Metadef ORM classes should use a common delete for consistency https://review.openstack.org/122910 | 02:48 |
| *** kebray has joined #openstack-glance | 02:56 | |
| *** arnaud____ has joined #openstack-glance | 02:57 | |
| *** kebray has quit IRC | 02:57 | |
| *** kebray has joined #openstack-glance | 02:58 | |
| openstackgerrit | Wayne Okuma proposed openstack/glance: Metadef ORM classes should use a common delete for consistency https://review.openstack.org/122910 | 03:01 |
| *** wayne__ has quit IRC | 03:04 | |
| *** skoar has joined #openstack-glance | 03:15 | |
| *** Longgeek has joined #openstack-glance | 03:18 | |
| *** annashen has joined #openstack-glance | 03:19 | |
| *** zxen has quit IRC | 03:20 | |
| *** [1]shalq has joined #openstack-glance | 03:20 | |
| *** zxen has joined #openstack-glance | 03:20 | |
| *** shalq has quit IRC | 03:22 | |
| *** [1]shalq is now known as shalq | 03:22 | |
| *** annashen has quit IRC | 03:23 | |
| *** Longgeek has quit IRC | 03:29 | |
| *** julim has joined #openstack-glance | 03:31 | |
| *** MentalRay has quit IRC | 03:34 | |
| *** MentalRay has joined #openstack-glance | 03:34 | |
| *** dims has quit IRC | 03:39 | |
| *** arnaud____ has quit IRC | 03:54 | |
| *** skoar has left #openstack-glance | 04:00 | |
| *** zxen has quit IRC | 04:01 | |
| *** zxen has joined #openstack-glance | 04:01 | |
| *** harshs has joined #openstack-glance | 04:10 | |
| openstackgerrit | wangxiyuan proposed openstack/python-glanceclient: Fix exception message in SessionClient https://review.openstack.org/198617 | 04:12 |
| openstackgerrit | wangxiyuan proposed openstack/python-glanceclient: Fix exception message in Http.py https://review.openstack.org/198617 | 04:13 |
| *** arnaud__ has joined #openstack-glance | 04:14 | |
| *** julim has quit IRC | 04:19 | |
| *** annashen has joined #openstack-glance | 04:19 | |
| *** jskinner has quit IRC | 04:22 | |
| *** annashen has quit IRC | 04:24 | |
| *** dims has joined #openstack-glance | 04:40 | |
| *** dims_ has joined #openstack-glance | 04:41 | |
| *** dims has quit IRC | 04:45 | |
| *** dims_ has quit IRC | 04:46 | |
| *** ajayaa has quit IRC | 04:47 | |
| openstackgerrit | Fei Long Wang proposed openstack/python-glanceclient: Use API v2 as default https://review.openstack.org/193931 | 04:50 |
| *** MentalRay has quit IRC | 04:55 | |
| *** [1]shalq has joined #openstack-glance | 05:11 | |
| *** shalq has quit IRC | 05:13 | |
| *** [1]shalq is now known as shalq | 05:13 | |
| *** exploreshaifali has joined #openstack-glance | 05:17 | |
| *** annashen has joined #openstack-glance | 05:21 | |
| *** e0ne has joined #openstack-glance | 05:22 | |
| *** annashen has quit IRC | 05:25 | |
| *** Longgeek has joined #openstack-glance | 05:26 | |
| *** ajayaa has joined #openstack-glance | 05:28 | |
| *** exploreshaifali has quit IRC | 05:28 | |
| *** nlevinki has joined #openstack-glance | 05:38 | |
| *** harshs has quit IRC | 05:43 | |
| *** e0ne has quit IRC | 05:44 | |
| *** ig0r_ has joined #openstack-glance | 05:52 | |
| *** Longgeek has quit IRC | 05:52 | |
| *** harshs has joined #openstack-glance | 05:53 | |
| *** gberginc has joined #openstack-glance | 05:54 | |
| *** MentalRay has joined #openstack-glance | 05:54 | |
| *** ig0r__ has quit IRC | 05:55 | |
| *** Longgeek has joined #openstack-glance | 05:56 | |
| *** arnaud__ has quit IRC | 06:01 | |
| *** gberginc has quit IRC | 06:04 | |
| *** MentalRay has quit IRC | 06:04 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Imported Translations from Transifex https://review.openstack.org/197206 | 06:06 |
| *** groen692 has joined #openstack-glance | 06:13 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance_store: Imported Translations from Transifex https://review.openstack.org/194510 | 06:19 |
| *** annashen has joined #openstack-glance | 06:21 | |
| *** annashen has quit IRC | 06:26 | |
| *** belmoreira has joined #openstack-glance | 06:43 | |
| *** fifieldt has joined #openstack-glance | 06:43 | |
| *** sgotliv has quit IRC | 06:45 | |
| openstackgerrit | Sudipta Biswas proposed openstack/glance: Return 404 for images in pending_delete status https://review.openstack.org/177326 | 06:54 |
| *** e0ne has joined #openstack-glance | 06:57 | |
| *** e0ne has quit IRC | 07:04 | |
| *** gberginc has joined #openstack-glance | 07:17 | |
| flaper87 | jokke_: https://review.openstack.org/#/c/177326/ -2'd this to avoid accidental merges (although there's not even a +2) | 07:19 |
| *** annashen has joined #openstack-glance | 07:22 | |
| *** belmoreira has quit IRC | 07:26 | |
| *** annashen has quit IRC | 07:27 | |
| *** harshs has quit IRC | 07:32 | |
| *** kebray has quit IRC | 07:33 | |
| *** e0ne has joined #openstack-glance | 07:36 | |
| *** jistr has joined #openstack-glance | 07:37 | |
| *** dims has joined #openstack-glance | 07:43 | |
| *** dims has quit IRC | 07:48 | |
| *** sgotliv has joined #openstack-glance | 07:52 | |
| *** e0ne is now known as e0ne_ | 08:01 | |
| *** exploreshaifali has joined #openstack-glance | 08:02 | |
| *** e0ne_ is now known as e0ne | 08:03 | |
| *** chlong has quit IRC | 08:13 | |
| *** annashen has joined #openstack-glance | 08:23 | |
| flaper87 | jokke_: any chance you can get Stuart to review this: https://review.openstack.org/#/c/154695/ | 08:27 |
| *** annashen has quit IRC | 08:28 | |
| *** gberginc has quit IRC | 08:46 | |
| *** gberginc has joined #openstack-glance | 08:54 | |
| *** krykowski has joined #openstack-glance | 09:05 | |
| *** e0ne is now known as e0ne_ | 09:11 | |
| openstackgerrit | Merged openstack/glance: Added test for "delete image member for public image". https://review.openstack.org/192085 | 09:18 |
| openstackgerrit | Merged openstack/glance: Fix wrong check when create image without data. https://review.openstack.org/198206 | 09:19 |
| *** boris-42 has quit IRC | 09:22 | |
| *** annashen has joined #openstack-glance | 09:24 | |
| *** e0ne_ is now known as e0ne | 09:24 | |
| *** pbourke has quit IRC | 09:27 | |
| *** pbourke has joined #openstack-glance | 09:27 | |
| *** annashen has quit IRC | 09:29 | |
| openstackgerrit | Merged openstack/glance: Functional of the HTTPclient was put in own method https://review.openstack.org/198310 | 09:29 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/glance: Updated from global requirements https://review.openstack.org/186114 | 09:31 |
| *** dims has joined #openstack-glance | 09:44 | |
| *** dims_ has joined #openstack-glance | 09:45 | |
| *** dims__ has joined #openstack-glance | 09:46 | |
| *** dims has quit IRC | 09:48 | |
| *** dims_ has quit IRC | 09:50 | |
| *** dims__ has quit IRC | 09:50 | |
| openstackgerrit | Kamil Rykowski proposed openstack/glance: Healthcheck Middleware https://review.openstack.org/148595 | 09:52 |
| *** e0ne is now known as e0ne_ | 10:07 | |
| *** e0ne_ is now known as e0ne | 10:21 | |
| *** annashen has joined #openstack-glance | 10:25 | |
| *** annashen has quit IRC | 10:30 | |
| *** sgotliv has quit IRC | 10:30 | |
| *** exploreshaifali has quit IRC | 10:32 | |
| *** GeetikaBatra has joined #openstack-glance | 10:33 | |
| jokke_ | flaper87: thanks | 10:40 |
| jokke_ | flaper87: flagged that to Stuart | 10:40 |
| *** dims has joined #openstack-glance | 10:46 | |
| *** dims has quit IRC | 10:51 | |
| *** GeetikaBatra_ has joined #openstack-glance | 11:06 | |
| openstackgerrit | Darja Shakhray proposed openstack/glance_store: Fix expired token while upload chunks https://review.openstack.org/199049 | 11:07 |
| *** Longgeek has quit IRC | 11:08 | |
| *** GeetikaBatra has quit IRC | 11:08 | |
| *** exploreshaifali has joined #openstack-glance | 11:09 | |
| *** GeetikaBatra has joined #openstack-glance | 11:09 | |
| *** sgotliv has joined #openstack-glance | 11:10 | |
| *** delatte has quit IRC | 11:11 | |
| *** GeetikaBatra_ has quit IRC | 11:11 | |
| *** arnaud__ has joined #openstack-glance | 11:15 | |
| *** e0ne is now known as e0ne_ | 11:25 | |
| *** GeetikaBatra has quit IRC | 11:25 | |
| *** annashen has joined #openstack-glance | 11:26 | |
| *** sgotliv has quit IRC | 11:26 | |
| *** GeetikaBatra has joined #openstack-glance | 11:29 | |
| *** annashen has quit IRC | 11:30 | |
| *** delatte has joined #openstack-glance | 11:31 | |
| *** mclaren has joined #openstack-glance | 11:35 | |
| *** GeetikaBatra is now known as GB21 | 11:35 | |
| mclaren | flwang: hi, is it ok if we take a look at fixing bug 1439808? (It's currently assigned to you) | 11:36 |
| openstack | bug 1439808 in Glance "Updating an image by adding or replacing 'id' returns a 200 response" [Undecided,New] https://launchpad.net/bugs/1439808 - Assigned to Fei Long Wang (flwang) | 11:36 |
| *** e0ne_ is now known as e0ne | 11:45 | |
| *** GeetikaBatra has joined #openstack-glance | 11:45 | |
| *** Longgeek has joined #openstack-glance | 11:45 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Fix an issue with broken test on ci https://review.openstack.org/199061 | 11:48 |
| *** GB21 has quit IRC | 11:48 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 11:53 |
| *** jasondotstar has quit IRC | 11:57 | |
| *** dims has joined #openstack-glance | 12:02 | |
| *** jaypipes has joined #openstack-glance | 12:02 | |
| *** arnaud___ has joined #openstack-glance | 12:04 | |
| *** chlong has joined #openstack-glance | 12:06 | |
| *** arnaud__ has quit IRC | 12:07 | |
| *** dims has quit IRC | 12:07 | |
| *** mingdang1 has quit IRC | 12:08 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 12:09 |
| openstackgerrit | Pawel Koniszewski proposed openstack/glance: update req https://review.openstack.org/199072 | 12:13 |
| *** ajayaa has quit IRC | 12:15 | |
| openstackgerrit | Pawel Koniszewski proposed openstack/glance: update req https://review.openstack.org/199072 | 12:15 |
| *** sgotliv has joined #openstack-glance | 12:18 | |
| *** gberginc has quit IRC | 12:21 | |
| *** arnaud___ has quit IRC | 12:23 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 12:26 |
| *** annashen has joined #openstack-glance | 12:27 | |
| *** ajayaa has joined #openstack-glance | 12:31 | |
| *** annashen has quit IRC | 12:31 | |
| *** edmondsw has joined #openstack-glance | 12:38 | |
| *** MentalRay has joined #openstack-glance | 12:41 | |
| *** MentalRay has quit IRC | 12:41 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 12:49 |
| *** krykowski has quit IRC | 12:59 | |
| *** krykowski_ has joined #openstack-glance | 12:59 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 13:03 |
| *** changbl has quit IRC | 13:08 | |
| *** jskinner has joined #openstack-glance | 13:08 | |
| *** haomaiw__ has quit IRC | 13:08 | |
| *** jasondotstar has joined #openstack-glance | 13:09 | |
| *** jasondot_ has joined #openstack-glance | 13:10 | |
| *** jasondot_ has quit IRC | 13:13 | |
| *** jasondotstar has quit IRC | 13:14 | |
| *** jasondotstar has joined #openstack-glance | 13:14 | |
| *** mingdang1 has joined #openstack-glance | 13:15 | |
| *** exploreshaifali has quit IRC | 13:15 | |
| *** mingdang_ has joined #openstack-glance | 13:16 | |
| *** dims has joined #openstack-glance | 13:19 | |
| *** mingdang1 has quit IRC | 13:20 | |
| *** bdossant has joined #openstack-glance | 13:21 | |
| *** haomaiwa_ has joined #openstack-glance | 13:22 | |
| *** julim has joined #openstack-glance | 13:24 | |
| *** annashen has joined #openstack-glance | 13:27 | |
| *** mtanino has joined #openstack-glance | 13:31 | |
| *** spzala has joined #openstack-glance | 13:31 | |
| *** annashen has quit IRC | 13:32 | |
| *** boris-42 has joined #openstack-glance | 13:33 | |
| *** e0ne is now known as e0ne_ | 13:37 | |
| *** Ctina has joined #openstack-glance | 13:39 | |
| *** e0ne_ is now known as e0ne | 13:39 | |
| *** jasondotstar has quit IRC | 13:40 | |
| *** ccrouch has joined #openstack-glance | 13:42 | |
| *** annegentle has joined #openstack-glance | 13:42 | |
| *** haomaiwa_ has quit IRC | 13:42 | |
| *** ajayaa has quit IRC | 13:45 | |
| *** ccrouch has left #openstack-glance | 13:45 | |
| openstackgerrit | Niall Bunting proposed openstack/glance: Make 'id' a read only property for v2 https://review.openstack.org/199104 | 13:48 |
| *** haomaiwang has joined #openstack-glance | 13:48 | |
| openstackgerrit | Chris St. Pierre proposed openstack/glance: Purge dead file-backed scrubber queue code https://review.openstack.org/161051 | 13:54 |
| *** r-daneel has joined #openstack-glance | 13:56 | |
| *** annegentle has quit IRC | 13:57 | |
| *** sigmavirus24_awa is now known as sigmavirus24 | 13:57 | |
| *** Ctina has quit IRC | 14:00 | |
| *** Ctina has joined #openstack-glance | 14:00 | |
| sigmavirus24 | nikhil_k: driver's meeting? | 14:01 |
| *** annegentle has joined #openstack-glance | 14:02 | |
| *** ccrouch has joined #openstack-glance | 14:02 | |
| nikhil_k | sigmavirus24: a bit late today | 14:05 |
| sigmavirus24 | No worries | 14:05 |
| sigmavirus24 | flaper87: you around? | 14:06 |
| flaper87 | sigmavirus24: yes | 14:06 |
| sigmavirus24 | :D | 14:06 |
| *** annegentle has quit IRC | 14:07 | |
| *** mingdang_ has quit IRC | 14:09 | |
| *** pbourke has quit IRC | 14:14 | |
| *** krykowski_ has quit IRC | 14:14 | |
| *** pdb has joined #openstack-glance | 14:20 | |
| *** chlong has quit IRC | 14:25 | |
| *** ducttape_ has joined #openstack-glance | 14:27 | |
| *** krykowski has joined #openstack-glance | 14:27 | |
| *** annashen has joined #openstack-glance | 14:28 | |
| nikhil_k | yee | 14:30 |
| nikhil_k | who's around? | 14:30 |
| jokke_ | hi there ... so backport question | 14:30 |
| nikhil_k | flaper87: sigmavirus24 jokke_ ^ | 14:30 |
| jokke_ | how do ye feel about https://review.openstack.org/#/c/156580/ && https://review.openstack.org/#/c/156573/ | 14:31 |
| sigmavirus24 | hi | 14:31 |
| flaper87 | jokke_: +1 for both | 14:31 |
| flaper87 | nikhil_k: here | 14:31 |
| sigmavirus24 | So | 14:31 |
| nikhil_k | hey hey! | 14:31 |
| jokke_ | flaper87: well it's both or neither | 14:31 |
| sigmavirus24 | eh | 14:31 |
| sigmavirus24 | jokke_: so https://review.openstack.org/#/c/156573/ seems bad to me because it's a breaking API change | 14:32 |
| jokke_ | they need to go in hand in hand | 14:32 |
| *** annegentle has joined #openstack-glance | 14:32 | |
| sigmavirus24 | AH yeah | 14:32 |
| flaper87 | wait, breaking change? | 14:32 |
| sigmavirus24 | the glance one solves the backwards compat problems for glance_store | 14:32 |
| flaper87 | ah phew | 14:32 |
| sigmavirus24 | flaper87: yeah, raising an exception where previously you never raised one is a breaking API change | 14:32 |
| sigmavirus24 | jokke_: is right that they need to go hand in hand | 14:32 |
| jokke_ | yes that's my problem there | 14:32 |
| sigmavirus24 | Technically you could do the glance without the glance_store one =P | 14:33 |
| sigmavirus24 | glance one also has a bit of refactoring in it =/ | 14:33 |
| *** annashen has quit IRC | 14:33 | |
| flaper87 | I think I missed that one | 14:33 |
| flaper87 | but yeah, if it's a breaking change then nope | 14:33 |
| sigmavirus24 | So | 14:33 |
| sigmavirus24 | in Kilo, is anyone else using glance_store? | 14:33 |
| jokke_ | it's breaking the api between glance and glance-store ... meaning that we need to push in both if we want them and that needs to be taken into account if upgrading stable/kilo | 14:33 |
| nikhil_k | Yeah, let's avoid single approvals | 14:33 |
| sigmavirus24 | nikhil_k: gerrit can be configured to enforce that =P | 14:34 |
| nikhil_k | It can be tricky to catch these all by yourself given we have more than one job/prj these days | 14:34 |
| sigmavirus24 | Also I had +2'd that before | 14:34 |
| nikhil_k | sigmavirus24: that's a good idea. can we indicate branches for that? | 14:34 |
| sigmavirus24 | nikhil_k: yes I believe so | 14:34 |
| nikhil_k | we don't want to block stable/* basically | 14:34 |
| sigmavirus24 | Or | 14:35 |
| sigmavirus24 | Yeah | 14:35 |
| sigmavirus24 | Who is our stable maintenance team these days? | 14:35 |
| sigmavirus24 | Zhi Yan Liu, jokke_, you (nikhil_k), flaper87 and who else? | 14:35 |
| *** haypo has joined #openstack-glance | 14:35 | |
| sigmavirus24 | flaper87: is global stable maint, right? | 14:35 |
| nikhil_k | that's it | 14:35 |
| *** nlevinki has quit IRC | 14:35 | |
| haypo | hi. can someone please review my 2 last python 3 patches for glance_store? https://review.openstack.org/#/c/184603/ & https://review.openstack.org/#/c/184601/ | 14:35 |
| jokke_ | global is included in each project stable_maint group | 14:35 |
| jokke_ | iiuc | 14:36 |
| nikhil_k | and yes | 14:36 |
| flaper87 | sigmavirus24: yes, global | 14:36 |
| haypo | they already got a +2 from Ian Cordasco | 14:36 |
| sigmavirus24 | haypo: I am the real Ian Cordasco =P | 14:36 |
| sigmavirus24 | ;) | 14:36 |
| haypo | sigmavirus24: thanks for your review! | 14:36 |
| nikhil_k | mystery solved | 14:36 |
| sigmavirus24 | You're welcome! | 14:36 |
| sigmavirus24 | haypo: thanks for doing the hard work :D | 14:37 |
| nikhil_k | this should be on the next sherlock :P | 14:37 |
| haypo | i'm waiting for a release for glance_store compatible with py3 to continue | 14:37 |
| sigmavirus24 | So jokke_ seems to be doing a lot of this on his own from what I've seen | 14:37 |
| sigmavirus24 | Do we need more stable maintainers too? | 14:37 |
| sigmavirus24 | haypo: yeah I know | 14:37 |
| sigmavirus24 | haypo: os-ansible-deployment was angry at me for approving lifeless's patch with the -e git+.../glance_store.git requirement line | 14:38 |
| nikhil_k | sigmavirus24: we don't regulate that. ttx and doug would need to be asked | 14:38 |
| sigmavirus24 | nikhil_k: glance_store release or stable maintainers? | 14:38 |
| nikhil_k | stable | 14:38 |
| sigmavirus24 | okay | 14:38 |
| nikhil_k | and glance_store was being handled by flaper87 and doug these days | 14:38 |
| jokke_ | sigmavirus24: I'm happy following up the stables ... and I get well enough reviews when I poke these guys for them ;) | 14:38 |
| sigmavirus24 | heh | 14:39 |
| jokke_ | the stable is bit tricky and I'd rather not add more maintainers unless people actually pick up the work and show that they understand what the stable is about | 14:39 |
| sigmavirus24 | jokke_: yeah I get that | 14:39 |
| sigmavirus24 | I just feel bad that you're doing what seems to be 98% of the work (if not 99) | 14:39 |
| jokke_ | thanks ... that seems to be the global trend on stable ;) | 14:40 |
| nikhil_k | we need more recruits for glance and I'm working on a couple these days | 14:40 |
| nikhil_k | should we continue the drivers' discussion? | 14:41 |
| jokke_ | sigmavirus24: I'm more than happy to take the help on bug targeting ... so you guys are on the drivers which means that you're on the release management group I'm not in, which means that you can actually target those bigs I'm tagging to the stable branches | 14:41 |
| jokke_ | /bigs/bugs/ | 14:42 |
| haypo | sigmavirus24: if we cannot get these two glance_store patches merged, maybe we may disable python3 to unblock glance? | 14:42 |
| haypo | (to fix -e in requirements) | 14:43 |
| *** annegentle has quit IRC | 14:43 | |
| nikhil_k | haypo: it's a non-voting gate job | 14:43 |
| sigmavirus24 | jokke_: when you think something should be targetted, nominate it and I'll approve it. If I don't approve it right away, ping me in IRC and I will. I trust your judgment implicitly | 14:43 |
| sigmavirus24 | haypo: the -e is not a big deal | 14:43 |
| nikhil_k | haypo: so what sigmavirus24 just said | 14:43 |
| jokke_ | sigmavirus24: thnx | 14:43 |
| sigmavirus24 | haypo: I'm honestly not very concerned by it being there. os-ansible-deployment is just testing against master of all OS projects and so it got annoyed when its requirements parsing went sideways | 14:44 |
| sigmavirus24 | I'm a core on os-ansible-deployment and warned them when I +2'd that change | 14:44 |
| sigmavirus24 | so | 14:44 |
| sigmavirus24 | I blame them for not properly heeding my warning =P | 14:44 |
| haypo | nikhil_k: "it's a non-voting gate job" which job? py34 is voting in glance | 14:45 |
| sigmavirus24 | nikhil_k: I believe haypo is correct about it being voting in both glance and glance_store | 14:45 |
| nikhil_k | in that case, nvm. I thought you mentioned glance_store | 14:45 |
| haypo | the -e issue is in glance | 14:45 |
| nikhil_k | py34 is non-voting glance_store | 14:45 |
| *** krykowski has quit IRC | 14:47 | |
| haypo | nikhil_k: correct | 14:48 |
| jokke_ | ok, so 3 backports to glance_store stable/kilo, not too bad | 14:48 |
| nikhil_k | haypo: on my list for today. | 14:48 |
| *** ajayaa has joined #openstack-glance | 14:49 | |
| *** krykowski has joined #openstack-glance | 14:49 | |
| jokke_ | that's actually my next question ... do we want to bring stable/kilo to py3 compliance ... I wouldn't mind at all | 14:49 |
| *** nlevinki has joined #openstack-glance | 14:50 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 14:51 |
| haypo | nikhil_k: tox -e py34 fails because S3 & vmware, so it doesn't make sense to make the py34 gate voting yet | 14:52 |
| haypo | nikhil_k: in the meanwhile i proposed a patch to fix tox -e py34, but it's a compromise if my two patches for S3 & vmware are not merged | 14:52 |
| etoews | for importing an image via the v2 api, is the flow that you first import an image (POST /tasks) and then update the image with the properties it needs (PATCH/images/{image_id})? | 14:52 |
| haypo | nikhil_k: i would prefer to get these two patches merged instead, so we don't need tricks for tox -e py34 ;) | 14:52 |
| *** GeetikaBatra_ has joined #openstack-glance | 14:52 | |
| *** ducttape_ has quit IRC | 14:53 | |
| *** GeetikaBatra has quit IRC | 14:55 | |
| nikhil_k | haypo: that makes sense. these need reviews I understand. | 14:56 |
| nikhil_k | etoews: you can pass properties as a part of the task body | 14:56 |
| nikhil_k | etoews: are you using devstack tasks? | 14:56 |
| nikhil_k | or rackspace? | 14:56 |
| etoews | both | 14:57 |
| nikhil_k | etoews: I need to verify devstack, we had some stuff broken and some peeps fixed it so, that being the ideal state may be true in reality. | 14:57 |
| nikhil_k | may not be* | 14:57 |
| haypo | sigmavirus24: would you like to modify tests of the xattr image cache to mock atime? -- https://bugs.launchpad.net/glance/+bug/1471243 | 14:57 |
| openstack | Launchpad bug 1471243 in Glance "Test on the xattr image cache fails because of noatime" [Medium,Confirmed] | 14:57 |
| nikhil_k | that sounds good haypo | 14:58 |
| sigmavirus24 | haypo: yeah | 14:59 |
| sigmavirus24 | haypo: to be fair, I really think we did a bad job with those drivers | 14:59 |
| sigmavirus24 | I want to write a spec to fix how we handle image-cache drivers as I indicated in that comment | 15:00 |
| haypo | sigmavirus24: hum, what do you mean? it's a design issue in the driver to rely on atime? | 15:00 |
| sigmavirus24 | But a short term fix makes sense | 15:00 |
| sigmavirus24 | haypo: yeah and for it not to disable itself when atime isn't present | 15:00 |
| nikhil_k | sigmavirus24: may you can chat with hemanthm as he was planning on working some on such things | 15:00 |
| sigmavirus24 | We have similar behaviour for glance_store drivers that they are not allowed to be used when a dependency is missing | 15:00 |
| sigmavirus24 | nikhil_k: ah that's good to know | 15:00 |
| nikhil_k | GeetikaBatra_: around? | 15:01 |
| sigmavirus24 | haypo: it's also very common for noatime to be set for performance benefits and atime is just a bad idea in general | 15:01 |
| sigmavirus24 | haypo: also, if you look at the comment, we really don't even use atime for anything | 15:01 |
| sigmavirus24 | And it's part of the deprecated v1 api | 15:01 |
| sigmavirus24 | So | 15:01 |
| haypo | sigmavirus24: "we really don't even use atime for anything" well, i don't understand the bug. my concern is just that my patch got a -1 :) | 15:02 |
| *** annegentle has joined #openstack-glance | 15:04 | |
| *** obedmr has joined #openstack-glance | 15:04 | |
| sigmavirus24 | haypo: yeah no worries | 15:06 |
| sigmavirus24 | I'm just miniranting again | 15:06 |
| sigmavirus24 | It's been that kind of morning =P | 15:06 |
| *** annegentle has quit IRC | 15:13 | |
| *** e0ne is now known as e0ne_ | 15:16 | |
| *** dims has quit IRC | 15:16 | |
| *** kebray has joined #openstack-glance | 15:17 | |
| *** nlevinki has quit IRC | 15:19 | |
| *** tsekiyama has joined #openstack-glance | 15:20 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: (WIP) Artifacts client listing support https://review.openstack.org/198450 | 15:20 |
| *** annegentle has joined #openstack-glance | 15:21 | |
| *** tpeoples has joined #openstack-glance | 15:21 | |
| *** e0ne_ is now known as e0ne | 15:26 | |
| *** shalq has quit IRC | 15:27 | |
| *** shalq has joined #openstack-glance | 15:28 | |
| *** annashen has joined #openstack-glance | 15:29 | |
| openstackgerrit | Niall Bunting proposed openstack/glance: Make 'id' a read only property for v2 https://review.openstack.org/199104 | 15:29 |
| *** changbl has joined #openstack-glance | 15:31 | |
| *** annashen has quit IRC | 15:34 | |
| *** dims has joined #openstack-glance | 15:43 | |
| *** bdossant has quit IRC | 15:44 | |
| *** annegentle has quit IRC | 15:56 | |
| *** GeetikaBatra_ is now known as GB21 | 15:59 | |
| *** kebray has quit IRC | 15:59 | |
| *** jistr has quit IRC | 16:00 | |
| *** mgarza_ has joined #openstack-glance | 16:00 | |
| *** kebray has joined #openstack-glance | 16:01 | |
| *** aswadr has joined #openstack-glance | 16:02 | |
| *** annegentle has joined #openstack-glance | 16:03 | |
| *** jistr has joined #openstack-glance | 16:07 | |
| *** annegentle has quit IRC | 16:10 | |
| *** haomaiw__ has joined #openstack-glance | 16:16 | |
| *** haomaiwang has quit IRC | 16:18 | |
| *** ajayaa has quit IRC | 16:20 | |
| *** hogepodge has quit IRC | 16:21 | |
| *** krykowski has quit IRC | 16:23 | |
| *** annashen has joined #openstack-glance | 16:28 | |
| *** e0ne has quit IRC | 16:33 | |
| *** jistr has quit IRC | 16:35 | |
| *** exploreshaifali has joined #openstack-glance | 16:35 | |
| *** groen692 has quit IRC | 16:38 | |
| *** haypo has quit IRC | 16:39 | |
| *** haypo has joined #openstack-glance | 16:40 | |
| *** annegentle has joined #openstack-glance | 16:40 | |
| *** haypo has left #openstack-glance | 16:41 | |
| *** annegentle has quit IRC | 16:44 | |
| *** Longgeek has quit IRC | 16:47 | |
| *** harshs has joined #openstack-glance | 16:53 | |
| *** ducttape_ has joined #openstack-glance | 17:04 | |
| *** aswadr has quit IRC | 17:07 | |
| *** aix has quit IRC | 17:08 | |
| *** e0ne has joined #openstack-glance | 17:09 | |
| *** ajayaa has joined #openstack-glance | 17:10 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Fix an issue with broken test on ci https://review.openstack.org/199061 | 17:13 |
| *** e0ne is now known as e0ne_ | 17:18 | |
| *** david-lyle has quit IRC | 17:19 | |
| *** e0ne_ is now known as e0ne | 17:21 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Fix an issue with broken test on ci https://review.openstack.org/199061 | 17:23 |
| *** e0ne is now known as e0ne_ | 17:26 | |
| *** e0ne_ is now known as e0ne | 17:27 | |
| *** ducttape_ has quit IRC | 17:29 | |
| *** annegentle has joined #openstack-glance | 17:30 | |
| *** e0ne is now known as e0ne_ | 17:33 | |
| *** annegentle has quit IRC | 17:35 | |
| *** e0ne_ is now known as e0ne | 17:39 | |
| *** TravT has quit IRC | 17:44 | |
| *** TravT_ has joined #openstack-glance | 17:44 | |
| *** Longgeek has joined #openstack-glance | 17:53 | |
| *** ducttape_ has joined #openstack-glance | 17:53 | |
| *** TravT_ has quit IRC | 17:56 | |
| *** TravT has joined #openstack-glance | 17:56 | |
| *** mgarza_ has quit IRC | 18:00 | |
| *** TravT_ has joined #openstack-glance | 18:01 | |
| *** harshs has quit IRC | 18:02 | |
| *** TravT has quit IRC | 18:02 | |
| *** julim_ has joined #openstack-glance | 18:03 | |
| *** annashen has quit IRC | 18:03 | |
| *** Longgeek has quit IRC | 18:04 | |
| *** ajayaa has quit IRC | 18:04 | |
| *** changbl has quit IRC | 18:05 | |
| *** julim has quit IRC | 18:05 | |
| *** annashen has joined #openstack-glance | 18:05 | |
| *** mgarza_ has joined #openstack-glance | 18:06 | |
| *** hogepodge has joined #openstack-glance | 18:11 | |
| *** harshs has joined #openstack-glance | 18:12 | |
| *** e0ne is now known as e0ne_ | 18:17 | |
| *** GB21 has quit IRC | 18:19 | |
| *** hogepodge has quit IRC | 18:20 | |
| *** annashen has quit IRC | 18:23 | |
| *** annashen has joined #openstack-glance | 18:24 | |
| *** andymccr has quit IRC | 18:25 | |
| *** GeetikaBatra has joined #openstack-glance | 18:26 | |
| *** annashen has quit IRC | 18:28 | |
| *** mgarza_ has quit IRC | 18:38 | |
| *** e0ne_ is now known as e0ne | 18:39 | |
| *** mgarza_ has joined #openstack-glance | 18:45 | |
| *** e0ne has quit IRC | 18:50 | |
| *** bpoulos has joined #openstack-glance | 18:52 | |
| *** dims has quit IRC | 18:53 | |
| *** GeetikaBatra_ has joined #openstack-glance | 18:54 | |
| *** annegentle has joined #openstack-glance | 18:56 | |
| *** GeetikaBatra has quit IRC | 18:56 | |
| *** obedmr has left #openstack-glance | 18:57 | |
| *** dims has joined #openstack-glance | 18:58 | |
| *** GeetikaBatra_ has quit IRC | 18:59 | |
| *** hogepodge has joined #openstack-glance | 19:00 | |
| *** kebray has quit IRC | 19:02 | |
| *** harlowja_ has joined #openstack-glance | 19:02 | |
| *** harlowja has quit IRC | 19:03 | |
| *** annashen has joined #openstack-glance | 19:06 | |
| *** dramakri has joined #openstack-glance | 19:08 | |
| *** harshs has quit IRC | 19:08 | |
| *** ayoung is now known as ayoung-afk | 19:13 | |
| *** mgarza_ has quit IRC | 19:14 | |
| *** TravT_ is now known as TravT | 19:17 | |
| *** changbl has joined #openstack-glance | 19:22 | |
| *** mgarza has joined #openstack-glance | 19:24 | |
| *** harshs has joined #openstack-glance | 19:30 | |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Glance v3 client basic methods https://review.openstack.org/192261 | 19:35 |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Glance v3 client initial commit https://review.openstack.org/189860 | 19:35 |
| openstackgerrit | Mike Fedosin proposed openstack/python-glanceclient: Glance v3 client methods for work with blobs https://review.openstack.org/197970 | 19:35 |
| *** kebray has joined #openstack-glance | 19:40 | |
| lifeless | sigmavirus24: hi | 19:41 |
| lifeless | sigmavirus24: the answer there is to get rid of the -d asap | 19:41 |
| lifeless | erm | 19:41 |
| lifeless | -e asap | 19:41 |
| sigmavirus24 | lifeless: No I know | 19:42 |
| sigmavirus24 | We still need to approve Victor's last two patches | 19:42 |
| sigmavirus24 | Then we can cut a release of some sort to get that fixed | 19:42 |
| sigmavirus24 | it's on us | 19:42 |
| lifeless | sigmavirus24: you could just do a release accepting that its still not ready for py3, but better than it was ;) | 19:43 |
| lifeless | sigmavirus24: e.g. there's nothing blocking the release AFAICT | 19:43 |
| sigmavirus24 | True. I can't cut releases though =P | 19:44 |
| *** ajayaa has joined #openstack-glance | 19:45 | |
| *** dims has quit IRC | 19:48 | |
| *** kebray has quit IRC | 19:50 | |
| *** david-lyle has joined #openstack-glance | 19:50 | |
| *** ayoung-afk is now known as ayoung | 19:52 | |
| *** e0ne has joined #openstack-glance | 19:54 | |
| *** wayne__ has joined #openstack-glance | 19:58 | |
| *** mgarza has quit IRC | 20:00 | |
| *** Longgeek has joined #openstack-glance | 20:01 | |
| *** mgarza_ has joined #openstack-glance | 20:03 | |
| *** kebray has joined #openstack-glance | 20:04 | |
| *** chlong has joined #openstack-glance | 20:06 | |
| lifeless | sigmavirus24: I can | 20:10 |
| lifeless | sigmavirus24: it just needs a request to the releases team to make it | 20:10 |
| * sigmavirus24 doesnt' want to step on nikhil_k's toes | 20:13 | |
| *** kebray has quit IRC | 20:17 | |
| *** kebray has joined #openstack-glance | 20:18 | |
| *** spzala has quit IRC | 20:19 | |
| nikhil_k | sigmavirus24: lifeless : I guess we can cut the release after victor's changes are merged? | 20:20 |
| *** Ctina has quit IRC | 20:20 | |
| lifeless | nikhil_k: its up to you. You can always do two releases | 20:20 |
| lifeless | one now to fix requirements.txt | 20:20 |
| lifeless | one in a week or whenever victor's changes are merged. | 20:20 |
| sigmavirus24 | Those changes should be merged soonish | 20:21 |
| sigmavirus24 | they're in good shape | 20:22 |
| sigmavirus24 | they both have +2s | 20:22 |
| sigmavirus24 | just need one more core to review them | 20:22 |
| nikhil_k | lifeless: that works. we can have another one next week. thanks. | 20:23 |
| nikhil_k | I see sigmavirus24's point, may be we can wait until tomorrow? | 20:23 |
| nikhil_k | lifeless: ^ | 20:24 |
| sigmavirus24 | nikhil_k: we can wait as long as we need | 20:24 |
| lifeless | nikhil_k: ok so -> #openstack-relmgr-office and we'll do a release. Once you and sigmavirus24 arm wrestle each other. | 20:24 |
| *** Longgeek has quit IRC | 20:24 | |
| sigmavirus24 | lol | 20:24 |
| lifeless | doing a release is about 2m work today | 20:24 |
| lifeless | its cheap | 20:24 |
| sigmavirus24 | I'd be afraid to arm wrestle nikhil_k | 20:24 |
| nikhil_k | :P | 20:24 |
| lifeless | I'm not affected by the ansible fallout | 20:24 |
| sigmavirus24 | He's 7ft tall and has the body of adonis I hear | 20:24 |
| lifeless | so I don't have a dog in the fight | 20:24 |
| sigmavirus24 | lifeless: yeah, they also already fixed their requirement parsing stuff so | 20:25 |
| sigmavirus24 | it's not a big deal | 20:25 |
| nikhil_k | and I'm in the middle of a very grumpy and process related meeting | 20:25 |
| nikhil_k | :P | 20:25 |
| sigmavirus24 | as I said, I warned them | 20:25 |
| sigmavirus24 | They didn't heed it so "Not my division" | 20:25 |
| lifeless | ansible manually parses requirements? ewwwwwwww | 20:25 |
| nikhil_k | sigmavirus24: lifeless: it's fine with me either way. | 20:26 |
| *** dramakri has quit IRC | 20:26 | |
| sigmavirus24 | lifeless: not ansible | 20:26 |
| sigmavirus24 | openstack-ansible builds wheels from source | 20:26 |
| sigmavirus24 | it does some requirements parsing to handle odd corner cases to be safe | 20:27 |
| *** dramakri has joined #openstack-glance | 20:27 | |
| sigmavirus24 | And like I said, I don't think it's much of a big deal at this point | 20:27 |
| nikhil_k | just mentioned in #openstack-relmgr-office | 20:28 |
| nikhil_k | that cutting a release should be fine today given the patches may or may not be merged | 20:28 |
| *** ducttape_ has quit IRC | 20:30 | |
| lifeless | sigmavirus24: but but but pip does it all | 20:31 |
| lifeless | sigmavirus24: Should I look or will I be sad? | 20:32 |
| sigmavirus24 | lifeless: you'll be a little sad | 20:32 |
| sigmavirus24 | and pip wasn't handling it well before | 20:32 |
| sigmavirus24 | it's also more complicated than you think | 20:32 |
| sigmavirus24 | we build then once, put them behind nginx in a container on the container network so the installs are repeatable etc. | 20:33 |
| lifeless | sigmavirus24: so sure, 'pip wheel -w mywheelhouse -f mywheelhouse directory' is the stock recipe for that | 20:33 |
| lifeless | sigmavirus24: I'd like to know more about these complications (given I've been tramping all through this space for a while now...) | 20:34 |
| sigmavirus24 | to be fair, I think we ran into the complications long before you started doing that and never looked back | 20:36 |
| *** jamielennox is now known as jamielennox|away | 20:41 | |
| *** e0ne has quit IRC | 20:44 | |
| *** jasondotstar has joined #openstack-glance | 20:50 | |
| *** ducttape_ has joined #openstack-glance | 20:51 | |
| *** jamielennox|away is now known as jamielennox | 20:52 | |
| *** jskinner has quit IRC | 20:54 | |
| *** exploreshaifali has quit IRC | 21:00 | |
| *** jdurgin2 has joined #openstack-glance | 21:01 | |
| *** jdurgin has quit IRC | 21:02 | |
| *** julim_ has quit IRC | 21:04 | |
| *** dims has joined #openstack-glance | 21:13 | |
| *** delatte has quit IRC | 21:14 | |
| *** jasondotstar has quit IRC | 21:16 | |
| *** dramakri has quit IRC | 21:16 | |
| *** kebray has quit IRC | 21:16 | |
| *** dramakri has joined #openstack-glance | 21:17 | |
| *** annegentle has quit IRC | 21:18 | |
| *** bpoulos has left #openstack-glance | 21:19 | |
| *** kebray has joined #openstack-glance | 21:21 | |
| openstackgerrit | Fei Long Wang proposed openstack/python-glanceclient: Use API v2 as default https://review.openstack.org/193931 | 21:25 |
| *** dims has quit IRC | 21:30 | |
| *** dims has joined #openstack-glance | 21:35 | |
| *** dims has quit IRC | 21:35 | |
| *** Guest7393 has joined #openstack-glance | 21:35 | |
| lifeless | sigmavirus24: maybe its time ? link me to what it does? | 21:41 |
| sigmavirus24 | openstack-ansible deployment or the requirements wheel builder and such? | 21:43 |
| *** sgotliv has quit IRC | 21:48 | |
| openstackgerrit | Wayne Okuma proposed openstack/glance: Glance metadef tables need unique constraints. https://review.openstack.org/195820 | 21:49 |
| *** Longgeek has joined #openstack-glance | 21:51 | |
| lifeless | sigmavirus24: the thing that broke | 21:51 |
| lifeless | which was related to wheel building etc | 21:52 |
| sigmavirus24 | https://github.com/cloudnull/yaprt | 21:52 |
| sigmavirus24 | lifeless: ^ | 21:53 |
| lifeless | that has so many bugs | 21:56 |
| lifeless | I've lost count already | 21:56 |
| lifeless | (e.g. it's not using PEP-440 version comparisons) | 21:56 |
| *** mgarza_ has quit IRC | 21:57 | |
| lifeless | and it looks like it leaks files on error. IMBW there - speed reading | 21:57 |
| *** annegentle has joined #openstack-glance | 21:57 | |
| *** Longgeek has quit IRC | 21:57 | |
| *** ducttape_ has quit IRC | 21:58 | |
| lifeless | its missing the -f to avoid redundant rebuilds of wheels | 21:58 |
| sigmavirus24 | lifeless: file bugs yo =P | 21:59 |
| lifeless | its going to be missing wheels now already, because of setup.cfg | 21:59 |
| lifeless | meh | 21:59 |
| lifeless | okok I migt | 21:59 |
| *** changbl has quit IRC | 21:59 | |
| lifeless | let me finish paging enough in | 21:59 |
| *** mgarza has joined #openstack-glance | 21:59 | |
| lifeless | I'm really just procrastinating on this git review | 22:00 |
| lifeless | this - https://github.com/cloudnull/yaprt/blob/master/yaprt/wheel_builder.py#L347 - is terrifying | 22:00 |
| sigmavirus24 | heh | 22:02 |
| lifeless | yah, its broken through and through | 22:05 |
| lifeless | sigmavirus24: https://github.com/cloudnull/yaprt/issues/3 | 22:15 |
| *** pennerc has joined #openstack-glance | 22:18 | |
| *** pennerc has quit IRC | 22:19 | |
| lifeless | sigmavirus24: 0.7.0 ok for glance-store? | 22:20 |
| lifeless | sigmavirus24: (we're in #openstack-relmgr-office doing the release right now) | 22:21 |
| cloudnull | ping @ lifeless. hows it ? | 22:26 |
| lifeless | hi cloudnull, all good | 22:27 |
| *** jdurgin2 has quit IRC | 22:27 | |
| cloudnull | i was reading through your issue you posted. | 22:27 |
| *** jdurgin has joined #openstack-glance | 22:27 | |
| *** annegentle has quit IRC | 22:29 | |
| cloudnull | we never hi an issue with the -e in the requirements for glance though I brought it up to sigmavirus24 the other day questioning why it was required at all. | 22:29 |
| cloudnull | lifeless: do you have an example of wheels that we may be missing from not directly indexing items out of setup.cfg and from setup.py? | 22:32 |
| *** edmondsw has quit IRC | 22:33 | |
| cloudnull | also we are respecting all of the markers for python deps from requirements. and creating a requirements file that has a collection of all markers, is there more marker support a that its missing ? | 22:34 |
| cloudnull | atleast the env markers from pip 6+ | 22:34 |
| cloudnull | example complete requirements.txt file http://rpc-repo.rackspace.com/os-releases/master/build_reqs.txt | 22:35 |
| cloudnull | i'd love to work on making this all better and pick your brain if you have a chance. | 22:35 |
| lifeless | as a for instance | 22:39 |
| lifeless | https://review.openstack.org/#/c/184328/ | 22:39 |
| lifeless | we're moving to having no text files for requirements at all | 22:39 |
| lifeless | outside of global-requirements | 22:40 |
| lifeless | the markers thing - it depends on your goal | 22:40 |
| lifeless | AIUI the goal is to build a collection of wheels for deployment elsewhere | 22:40 |
| lifeless | and for that, you need to make sure that the environment you build them in is an exact match. Preserving the markers is indeed necessary and perhaps its all ok. | 22:40 |
| lifeless | the concern I had is that you may not be making the environmen tbe an exact match | 22:41 |
| lifeless | (exact goes down to even point releases of python, architecture, linux distro vendor) | 22:41 |
| *** andymccr has joined #openstack-glance | 22:46 | |
| *** chlong has quit IRC | 22:46 | |
| cloudnull | so is glance looking to remove https://github.com/openstack/glance/blob/master/requirements.txt entirelly infavor of loading everything into setup.cfg for PBR ? | 22:46 |
| cloudnull | if so how would you sugest one might read the requirements generated via PBR without having to install and or look for imports ? does PBR have a facility for that ? | 22:52 |
| lifeless | thats the long term direction as we consolidate on pip's behaviour and semantics | 22:52 |
| lifeless | you create an egg_info and introspect that | 22:52 |
| lifeless | thats the defined interface today that pip users. | 22:52 |
| lifeless | uses | 22:52 |
| lifeless | (and one of the reasons I'm querying why you do more than just 'pip wheel ...' | 22:53 |
| lifeless | one pip command should be able to build all the wheels for a deployment of all of openstack in one hit | 22:53 |
| *** annashen has quit IRC | 22:54 | |
| cloudnull | ah, there are a few issues with simply running that, in the past that what I had done. but, several projects, namely heat, isnt keeping some of the subdirectory plugins updated which causes PBR bomb. We've raised that issue with them but theres little interest in maintining those contrib items. | 22:55 |
| cloudnull | also pip wheel (last i checked) failed on subdirectory packages from git | 22:55 |
| lifeless | they're under pressure upstream to make those requirements all be surfaced at the top | 22:55 |
| lifeless | and are working on it | 22:55 |
| lifeless | cloudnull: pip wheel shouldn't fail there, as long as the repo can actually be checked out | 22:55 |
| cloudnull | it would parse it, but ignore the subdirectory and rebuild the main repo | 22:56 |
| lifeless | cloudnull: there was a bug where someone complained that pip couldn't install a thing that couldn't be checked out due to submodules. | 22:56 |
| lifeless | we closed that one :) | 22:56 |
| jokke_ | lifeless: are you suggesting that we expect each deploy to install whole global requirements on each component node or how that system is supposed to track the per project requirements? | 22:56 |
| lifeless | cloudnull: uhm, the submodule in this case is a different project? | 22:56 |
| cloudnull | something like git+https://github.com/openstack/heat@6f948fbf0cf960e55e122720b1b429fbe20d442d#egg=extraroute&subdirectory=contrib/extraroute | 22:56 |
| * jokke_ probably missed something eyeing this discussion in the middle of the night | 22:57 | |
| lifeless | jokke_: once the wheels are built, they have a *very* well defined metadata you can query | 22:57 |
| cloudnull | pip wheel git+https://github.com/openstack/heat@6f948fbf0cf960e55e122720b1b429fbe20d442d#egg=extraroute&subdirectory=contrib/extraroute will clone checkout and built | 22:57 |
| lifeless | jokke_: so you can traverse their transitive dependencies really easily | 22:57 |
| cloudnull | but will not build the extraout dubdirectory | 22:57 |
| lifeless | cloudnull: so thats a heat bug, since that is broken for everyone | 22:57 |
| lifeless | cloudnull: and as I said, they're fixing it AIUI. | 22:58 |
| jokke_ | lifeless: aha! That makes more sense now ;) | 22:58 |
| cloudnull | i think thats a pip bug. its the same behavior on anything thats using the subdirectory syntax | 22:58 |
| jokke_ | I knew I missed something | 22:58 |
| lifeless | where fixing it == folding contrib into the real tree and getting rid of the spurious differences | 22:58 |
| lifeless | openstack infra /never/ supported multiple artifacts per source tree | 22:59 |
| cloudnull | same thing with "git+https://github.com/holland-backup/holland@v1.0.10#egg=holland.backup.xtrabackup&subdirectory=plugins/holland.backup.xtrabackup" | 22:59 |
| cloudnull | there are a few of them that we were attempting to use before. | 22:59 |
| lifeless | cloudnull: please file a bug about that subdirectory thing | 22:59 |
| cloudnull | will do | 22:59 |
| lifeless | cloudnull: in cases where its meant to work, I agree, pip should make a wheel of the subdir | 23:00 |
| lifeless | the heat one was broken in several ways since forever | 23:00 |
| cloudnull | it was | 23:00 |
| cloudnull | that was just the first example i had on hand :) | 23:00 |
| *** r-daneel has quit IRC | 23:00 | |
| lifeless | jokke_: looping back | 23:07 |
| lifeless | jokke_: I'd build one directory of wheels | 23:07 |
| lifeless | jokke_: copy that everywhere | 23:08 |
| lifeless | jokke_: and pip install $service as appropriate | 23:08 |
| lifeless | jokke_: the total code size is small compared to a server / container whatever | 23:08 |
| lifeless | jokke_: and the avoidance of having to second guess version resolution etc is a significant reduction in complexity | 23:08 |
| lifeless | jokke_: you probably also want to start consuming upper-constraints.txt | 23:09 |
| lifeless | jokke_: which will insulate you from unexpected releases tht break things | 23:09 |
| *** andymccr has quit IRC | 23:09 | |
| *** andymccr has joined #openstack-glance | 23:10 | |
| cloudnull | we've found creating the sets of wheel to be super useful. | 23:11 |
| *** annegentle has joined #openstack-glance | 23:12 | |
| cloudnull | so lifeless when i run `python setup.py egg_info` using the ref from your patchset https://review.openstack.org/#/c/184328/ i get oslo.db.egg-info and its requirements in a kind of ini format. http://cdn.pasteraw.com/mgark3b1t5l9ej5q68bodnwa0gj2l5o however without that patch i get a proper egg_info for glance http://cdn.pasteraw.com/bw3ao86mw070iveu1274pnxpznx2bcc | 23:18 |
| *** andymccr has quit IRC | 23:18 | |
| cloudnull | is that expected in this case ? | 23:18 |
| cloudnull | nevermind im an idiot . . . | 23:19 |
| cloudnull | it working as expecte.d | 23:19 |
| cloudnull | but the quetion of the ini format remains. i've not seen that in a requires before. | 23:21 |
| *** dramakri has quit IRC | 23:26 | |
| cloudnull | looking more into that, it seems thats perfectly valid syntax so, guess nothing to see there either. | 23:29 |
| lifeless | yah | 23:30 |
| lifeless | there's machinery in pip to handle all of this | 23:30 |
| lifeless | which is why I keep coming back to 'use pip', it sounds like there is just one bug you need fixed | 23:31 |
| cloudnull | looks like I can replace a lot in using the more modern version of pip, when it started we were running with pip 156. | 23:32 |
| lifeless | yeah | 23:32 |
| lifeless | pip hath not stood still :) | 23:32 |
| cloudnull | granted we using 7.1.0 now but it seems I need to revise some things | 23:32 |
| cloudnull | im checking to see if the subdir build process is still broken, if so ill raise an issue and have most of thse bits refacoted in the morning :) | 23:33 |
| *** annegentle has quit IRC | 23:34 | |
| *** Longgeek has joined #openstack-glance | 23:44 | |
| lifeless | cool | 23:48 |
| lifeless | the glue you have to deal with non-setuptools projects may still be needed | 23:48 |
| lifeless | but we could in principle try that within pip | 23:49 |
| lifeless | I'd want to talk to dstufft about it | 23:49 |
| *** Longgeek has quit IRC | 23:49 | |
| *** jasondotstar has joined #openstack-glance | 23:51 | |
| sigmavirus24 | lifeless: I for one continue to be against turning pip into a library =P | 23:54 |
| lifeless | sigmavirus24: huh? non-sequitor | 23:54 |
| lifeless | sigmavirus24: so, did you want glance-store released? we halted while you and nikhil_k were afk | 23:54 |
| sigmavirus24 | Let me pop int here | 23:55 |
| *** kebray has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!