Wednesday, 2023-06-07

opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible stable/yoga: Disable upgrade jobs after switching Xena to EM  https://review.opendev.org/c/openstack/openstack-ansible/+/88492810:45
noonedeadpunkThis has finally passed and ready for review https://review.opendev.org/c/openstack/openstack-ansible/+/884924 10:48
NeilHanlon👍12:34
NeilHanlonmornin'12:34
noonedeadpunko/12:37
opendevreviewDmitriy Rabotyagov proposed openstack/openstack-ansible-os_octavia master: Remove obsoleted provider drivers  https://review.opendev.org/c/openstack/openstack-ansible-os_octavia/+/88551916:02
jrosser`nova.exception.ImageNotAuthorized: Not authorized for image 8b68d0fd-6adb-4255-9d1e-67f8b892721a`16:23
jrosserthis is especially hard to debug :(16:23
jrosserglance logs seem really unhelpful even with debug16:24
mgariepyhmm fun16:25
jrossersomeone asked to make an application credential thats only allowed to create/delete instances16:26
jrosseromg turns out thats really hard16:26
mgariepydo you really have a solution for that ?16:26
mgariepyit does implies a lots or 3rd party api calls.. hahaha16:26
jrosserwell i am 90% of the way there16:26
jrosserexpect for ^ that error :)16:27
mgariepyif i read that right the app cred will have more rigth than *just* create/delete instances ?16:27
jrosserthis so far https://paste.opendev.org/show/bKBnfsgXHy1cYHBjL3Aw/16:28
mgariepynice.16:29
mgariepymethod patch  for glance?16:32
mgariepyhttps://bugzilla.redhat.com/show_bug.cgi?id=1382737 (PATCH stuff in this issue) 16:33
jrosserhmm seems haproxy log looks like a good place to see all these 401, as it also shows the path and http method16:34
jrosseroh cool i made it work16:43
jrosserhaproxy log is totally the only way to be able to construct the rules16:43
jrosseri have no idea whatsoever how end users are supposed to be able to use this16:44
NeilHanlonyou hire an openstack engineer of course!16:44
mgariepyalso fork the code a bit to add your magic sauce to it ;p16:47
mgariepyjrosser, what was missing ?16:50
jrosserGET of glance /v2/schemas/image16:51
jrosseri'm just sorting server delete out and i'll paste the whole rules16:51
jrossernova-compute seems to do that GET against glance16:51
mgariepyha ok16:52
jrosserthats the slightly absurd thing here that you have to allow all the internal API calls in the app cred too16:53
jrossereven though you never see them at the cli yourself16:53
mgariepyyeah.. juste like there is no delete --cascade to purge all the things .. 16:54
NeilHanlondo we already have some collection of rbac roles, like what you're setting up, jrosser? 16:55
jrosserno, i've tried to do this before and failed16:55
jrosserthis is the first time i've been persistent enough to understand / make it work16:55
jrosserdefinatly worth a documentation i think16:55
jrosserthe use case here is that someone is setting up a thing to use github ephemeral runners against our openstack16:56
* NeilHanlon is interested in this use case :D 16:56
jrosserso we want a limited token that it can use to just launch / destroy instances16:56
jrosserwe've forked the philips labs stuff16:56
jrosseri *think* it's this https://github.com/philips-labs/terraform-aws-github-runner16:57
jrosseri've not been involved in that part, but as i understand it there is some github actions <> AWS lambdas stuff, which we've extended to drive openstack instead of AWS16:57
jrossermgariepy: NeilHanlon this is what i have to create/delete now https://paste.opendev.org/show/bSDoK24YP70FNdUZfCUG/17:10
jrossernot sure the volumes stuff is actually needed - i added to what someone else made till it worked17:10
mgariepyi guess it depends if it booted from a volume or not.17:11
jrosseranyway, pretty please with that as it's something i've wanted to understand better for a while now17:12
jrosserjust regular app cred is far too powerful imho17:12
jrosserNeilHanlon: did you have an example cli to show building rocky9 + 64k page kernel?18:55
jrosser`DIB_RELEASE=9 DIB_ROCKY_KERNEL_64k=1 DIB_CLOUD_INIT_DATASOURCES=Ec2 DIB_CLOUD_INIT_GROWPART_DEVICES='["/"]' disk-image-create -a arm64 rocky-container vm block-device-efi cloud-init openssh-server cloud-init-datasources cloud-init-growpart -o baremetal-rocky-9-efi-amd64.qcow2`18:56
jrosser^ that doesnt get us a kernel on the grub menu18:56
NeilHanlonI think I had trouble passing 'arm64' when building and had to use 'aarch64' -- I didn't dig down enough to understand what was going wrong, though19:21
NeilHanlon`DIB_BOOT_LABEL=rocky DIB_RELEASE=9 DIB_ROCKY_KERNEL_64k=1 disk-image-create -a aarch64 -o rocky-arm64-64k.qcow2 vm rocky-container` is what I'd been doing19:21
jrosserhah i think i have a spurious amd64 in the image name there19:22
NeilHanlonit wasn't clear to me if/when/how arm64 is translated to/from aarch64, and what I should use in the package-installs.yaml here https://review.opendev.org/c/openstack/diskimage-builder/+/884452/3/diskimage_builder/elements/rocky-container/package-installs.yaml#1119:27
NeilHanlonI think because you're passing `-a arm64`, it doesn't match these pacakge install specifications unless it is `-a aarch64`19:28
NeilHanlontherefore, neither kernel nor kernel-64k are installed19:28
jrosserahha ok that makes sense19:29
opendevreviewMerged openstack/openstack-ansible master: Start 2023.2 (Bobcat) development  https://review.opendev.org/c/openstack/openstack-ansible/+/88492420:36

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