Thursday, 2021-11-25

opendevreviewGhanshyam proposed openstack/osc-placement master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/osc-placement/+/81920300:08
opendevreviewGhanshyam proposed openstack/os-vif master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/os-vif/+/81920400:08
opendevreviewGhanshyam proposed openstack/os-traits master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/os-traits/+/81920500:08
opendevreviewGhanshyam proposed openstack/placement master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/placement/+/81920600:08
opendevreviewGhanshyam proposed openstack/os-resource-classes master: Test all supported python version  https://review.opendev.org/c/openstack/os-resource-classes/+/81920700:16
opendevreviewGhanshyam proposed openstack/python-novaclient master: Updating python testing as per Yoga testing runtime  https://review.opendev.org/c/openstack/python-novaclient/+/81920800:16
*** EugenMayer9 is now known as EugenMayer00:56
mnasersdmitriev1: perhaps you can take the time to fix the merge conflict?03:21
*** tkajinam is now known as Guest674106:15
bauzashappy turkey day everyone08:50
nautikHello! I updated a merge request from a few months ago and it seems there is a "DellEMC PowerFlex CI" build failing with a message "EMC_PowerFlex_NOVA [...] : NOT_REGISTERED". Other builds (VMware, IBM...) succeeded.08:50
nautikis that expected or should I do something about it?08:50
nautikthe mr is https://review.opendev.org/c/openstack/nova/+/78107608:50
fricklernautik: that means that this 3rd party CI is offline. not related to your patch, nothing you can do about it08:56
fricklerexcept possibly trying to contact the CI operators, but that task should be on the nova team in general, not on you as patch author08:57
nautikok cool! thank you for the feedback, then this MR is ready for review09:07
nautik(not sure if I need to add some tags or do anything more to make it visible?)09:07
fricklernautik: just give it a couple of days, this week is also holiday season in some parts of the world. I there is no response after that, you could try pinging folks here again09:11
fricklers/I/If/09:12
nautikright, forgot that! Thank you and have a good day :)09:15
opendevreviewDmitrii Shcherbakov proposed openstack/nova master: [yoga] Support remote-managed SmartNIC DPU ports  https://review.opendev.org/c/openstack/nova/+/81211112:17
opendevreviewMerged openstack/nova master: Retry image download if it's corrupted  https://review.opendev.org/c/openstack/nova/+/81850313:05
sdmitriev1mnaser: good call, will look into that13:44
opendevreviewStanislav Dmitriev proposed openstack/nova stable/xena: Retry image download if it's corrupted  https://review.opendev.org/c/openstack/nova/+/81917913:55
opendevreviewStanislav Dmitriev proposed openstack/nova stable/wallaby: Retry image download if it's corrupted  https://review.opendev.org/c/openstack/nova/+/81918013:55
opendevreviewStanislav Dmitriev proposed openstack/nova stable/victoria: Retry image download if it's corrupted  https://review.opendev.org/c/openstack/nova/+/81918113:55
lyarwoodgibi: https://review.opendev.org/c/openstack/nova/+/818357 would you mind taking a look at this?14:09
gibisure14:09
gibidone14:11
gibithanks for updating our docs14:11
opendevreviewMerged openstack/nova stable/xena: Ensure MAC addresses characters are in the same case  https://review.opendev.org/c/openstack/nova/+/81688215:07
opendevreviewArtom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job  https://review.opendev.org/c/openstack/nova/+/81730315:52
opendevreviewArtom Lifshitz proposed openstack/nova master: DNM: Try disabling bind-time events  https://review.opendev.org/c/openstack/nova/+/81934915:52
*** artom_ is now known as artom15:53
opendevreviewArtom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job  https://review.opendev.org/c/openstack/nova/+/81730316:00
opendevreviewArtom Lifshitz proposed openstack/nova master: DNM: Try disabling bind-time events  https://review.opendev.org/c/openstack/nova/+/81934916:00
opendevreviewArtom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job  https://review.opendev.org/c/openstack/nova/+/81730316:21
opendevreviewArtom Lifshitz proposed openstack/nova master: DNM: Try disabling bind-time events  https://review.opendev.org/c/openstack/nova/+/81934916:21
opendevreviewGhanshyam proposed openstack/nova master: Updating tests with Yoga testing runtime  https://review.opendev.org/c/openstack/nova/+/81919416:56
opendevreviewGhanshyam proposed openstack/nova master: Updating tests with Yoga testing runtime  https://review.opendev.org/c/openstack/nova/+/81919417:38
opendevreviewMerged openstack/nova master: docs: Update libvirt distro support matrix for Xena  https://review.opendev.org/c/openstack/nova/+/81835717:40
EugenMayerWhat are the thoughs about moving secrets into the meta-data service which are shared e.g. by cluster nodes? Is this considered heavily unsecure? AFAIC each instance can only access it's own meta-data and cannot read any other instance meta-data, right? (Is this actually a nova or neutron question?)17:44
opendevreviewArtom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job  https://review.opendev.org/c/openstack/nova/+/81730318:16
opendevreviewArtom Lifshitz proposed openstack/nova master: DNM: Try disabling bind-time events  https://review.opendev.org/c/openstack/nova/+/81934918:16
sean-k-mooneyEugenMayer: am well you are partly correct18:16
sean-k-mooneyEugenMayer: ech isntance is only able to access its onw metadata 18:16
sean-k-mooneyhowever the metadata is also avaiable vai the api18:16
sean-k-mooneyso anywone in the project can access it via the api18:17
EugenMayeryes sure, every openstack admin cann access it .. but this one can also simply access the storage and read anything.18:17
sean-k-mooneywe also do not encypty the metaddat in any way in the nova db or when its sent to the instance18:17
EugenMayerI understand. It is plain text, send plain text18:18
sean-k-mooneyyep18:18
sean-k-mooneyso as an end user you can do this but you really shoudl use barbican18:18
sean-k-mooneyhttps://docs.openstack.org/barbican/latest/18:19
sean-k-mooneyEugenMayer: it provices and implementation fo a secure key manager which user and openstack can use18:19
EugenMayerinteresting - usually i would rather use vault18:19
sean-k-mooneyEugenMayer: it has a vault plugin18:20
sean-k-mooneyhttps://docs.openstack.org/barbican/latest/configuration/plugin_backends.html#enabling-multiple-barbican-backends18:21
EugenMayerwhat is the actually key USP of barbican compared to vault? Is there any auto-scoping of instances into "their namespace" or something like that? If it is just "if you have a speciifc token you can check in a secret KV for values you are interested in" i rather would use vault18:21
EugenMayeri see18:21
EugenMayersean-k-mooney: i know i repeat myself - but nevertheless, thank you very much!18:23
sean-k-mooneyEugenMayer: basically openstack a as a project has decalred that we will not manage securets in each project18:23
sean-k-mooneywe use castellan to provide a generic key manager https://github.com/openstack/castellan18:23
sean-k-mooneyand then barbican provides a secrets as a service api that endusers or sevices can use18:24
sean-k-mooneycastellan support barbican as a secret store and barbican support several backend to actully store the secrets18:24
EugenMayerI'am yet not sure i will need either of this. all secrets except this one are handled in k8s, this one is just to provision rke2. So i might stick to a chef-databag here since i use it to provision rke2 anyway18:25
sean-k-mooneyif you dont need to expose secrets as a service via openstack by the way castalan can also use vault directly18:25
sean-k-mooneyhttps://github.com/openstack/castellan/blob/master/castellan/key_manager/vault_key_manager.py18:25
EugenMayerinteresting, simple wrapper to read/write from the KV18:27
sean-k-mooneyhttps://docs.openstack.org/nova/latest/configuration/config.html#key_manager.backend you just ned to set that to valult and populate the related config options that are needed18:27
sean-k-mooneyEugenMayer: i dont know how much its used/tested18:28
sean-k-mooneyi dont often work on this part of the code but hoefully that helps18:29
EugenMayerI would not go the wrapper, arround the wrapper arround the wrapper way for this. If i need this, i will stick to vault directly IMHO. But it def. is good to know how things are handled and valued in the ecosystem18:29
EugenMayermeta-data is plaintext only, eventhough per instance, can be exploited if having enough API access (obviously). barbican to the rescue, will require a token to read from the storage, so knowledge required18:30
sean-k-mooneyEugenMayer: metadata will also show up in your debug logs in some cases18:30
opendevreviewArtom Lifshitz proposed openstack/nova master: Add nova-ovs-hybrid-plug job  https://review.opendev.org/c/openstack/nova/+/81730318:30
opendevreviewArtom Lifshitz proposed openstack/nova master: DNM: Try disabling bind-time events  https://review.opendev.org/c/openstack/nova/+/81934918:30
sean-k-mooneyso its really not intended for private stuff18:31
sean-k-mooneyEugenMayer: also just so you are aware instance metadta  is included in nova notificaitons18:32
sean-k-mooneyamqp is ment to be secured because it contains sensitive things but its just more reason not to store passwords/keys in it if you can avoid it18:33
EugenMayerunderstood, thank you!18:35
opendevreviewStephen Finucane proposed openstack/nova master: Deprecate the zvm driver  https://review.opendev.org/c/openstack/nova/+/81936518:50
opendevreviewStephen Finucane proposed openstack/nova master: Deprecate the powervm driver  https://review.opendev.org/c/openstack/nova/+/81936618:55
opendevreviewStephen Finucane proposed openstack/nova master: Deprecate the zvm driver  https://review.opendev.org/c/openstack/nova/+/81936518:56
artomZombie developer removing zombie code19:01
stephenfinmmm, brainz19:22
opendevreviewMerged openstack/nova master: db: Don't use legacy 'Row()' methods  https://review.opendev.org/c/openstack/nova/+/81774619:50

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