| opendevreview | Pierre-Samuel Le Stang proposed openstack/glance-specs master: [APIImpact] Add a new glance-download import method https://review.opendev.org/c/openstack/glance-specs/+/836132 | 14:14 |
|---|---|---|
| opendevreview | Pierre-Samuel Le Stang proposed openstack/glance master: [WIP] Implement glance-download internal plugin https://review.opendev.org/c/openstack/glance/+/840318 | 14:48 |
| gmann | pdeore: please join here https://meet.google.com/qax-tkne-dmk | 15:05 |
| jbernard | can glance reject uploads of a particular format, say only allow raw and reject qcow2, for example? | 17:47 |
| dansmith | jbernard: https://github.com/openstack/glance/blob/a42fda92dc8fe160bea1dd4d22cb1135fab89022/glance/common/config.py#L88-L103 | 18:13 |
| dansmith | I think that actually changes the jsonschema to make those values not acceptable in payloads | 18:13 |
| dansmith | which is kinda .. icky, but I imagine it will do what you're asking | 18:13 |
| jbernard | interesing, thanks for the link | 18:14 |
| jbernard | im looking at https://bugs.launchpad.net/cinder/+bug/1970115 | 18:14 |
| jbernard | and trying to determine if the patch is fixing a symptom and that be handled elsewhere, or if this is something we want in it current form | 18:14 |
| jbernard | s/and that/that should/ | 18:15 |
| dansmith | jbernard: image conversions in glance are opt-in, fwiw | 18:15 |
| jbernard | dansmith: good to know, thanks | 18:15 |
| dansmith | I definitely sympathize with mnaser's desire to be able to disable things like that | 18:15 |
| mnaser | jbernard: there is a scenario where you might want to have both qcow2 and raw images though | 18:16 |
| mnaser | if someone doesnt do boot from volume, they can use qcow2 just fine and it wont kill our control plane | 18:16 |
| jbernard | im thinking that too, as much as i dont like more config options, this one seems sound | 18:17 |
| mnaser | but if they're going to do boot from volume, its' fair that an operator says "make sure you got the right type of image to do that, because i dont want to burn my cpu cycles" | 18:17 |
| jbernard | mnaser: i added comments to the review | 18:18 |
| mnaser | jbernard: thanks for that :) i think rico is probably off but they'll look at it tomorrow | 18:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!