| *** rpittau|afk is now known as rpittau | 07:47 | |
| mgoddard | #hashtag | 07:50 |
|---|---|---|
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/wallaby: Fix nova deployment failure when rabbitmq is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/803554 | 08:24 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/victoria: Do not run timesync checks on deployment host https://review.opendev.org/c/openstack/kolla-ansible/+/803555 | 08:24 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/victoria: Fix freezed spice console in horizon https://review.opendev.org/c/openstack/kolla-ansible/+/803556 | 08:25 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/ussuri: Fix freezed spice console in horizon https://review.opendev.org/c/openstack/kolla-ansible/+/803557 | 08:25 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/victoria: Fix nova deployment failure when rabbitmq is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/803558 | 08:27 |
| opendevreview | Mark Goddard proposed openstack/kolla-ansible stable/ussuri: Fix nova deployment failure when rabbitmq is disabled https://review.opendev.org/c/openstack/kolla-ansible/+/803559 | 08:27 |
| opendevreview | Verification of a change to openstack/kayobe master failed: Remove references to resolv.conf workaround https://review.opendev.org/c/openstack/kayobe/+/803459 | 09:46 |
| kevko | hi o/ | 10:09 |
| kevko | is there a way to build xy images in binary and for example one in source ..in one batch (i mean kolla build ....) | 10:10 |
| shyamb | Hi | 10:37 |
| shyamb | I want to get all eligible candidates of regular expression string. For ex: this string: 'compute[01:10].trilio.demo' | 10:37 |
| shyamb | candidates should be a list of string - ['compute01.trilio.demo', 'compute02.trilio.demo',...,'compute10.trilio.demo' ] | 10:38 |
| shyamb | Does python re module supports this? | 10:38 |
| shyamb | I am not able to find suitable function to do this | 10:38 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/773246 | 10:44 |
| ug_egon | I upgraded a (test) cluster from train to wallaby. Seems mostly fine but nova conductor is complaining: ERROR nova nova.exception.TooOldComputeService: Current Nova version does not support computes older than Victoria but the minimum compute service level in your system is 40 and the oldest supported service level is 52. Is that expected? | 11:21 |
| kevko | ug_egon: yeah, it is because kolla-ansible support only +1 version | 11:26 |
| kevko | you jumped +3 | 11:27 |
| kevko | from stein to ussuri it was OK because nova was RPC compatible .. | 11:27 |
| kevko | you had to upgrade at least victoria | 11:27 |
| ug_egon | OK, i see | 11:27 |
| kevko | i was testing upgrade from ussuri -> wallaby and found this also .. | 11:28 |
| ug_egon | so the db migrations don't work all the way? | 11:28 |
| kevko | if you upgrade placement and nova to victoria ..then all to wallaby ..it will be ok | 11:28 |
| ug_egon | OK, thanks! | 11:30 |
| kevko | ug_egon: but I am not sure it this will be working if you have already upgraded | 11:31 |
| ug_egon | Well, as it is just a test system, i could test ;-) | 11:32 |
| kevko | oh, then it will be working, I am sure | 11:32 |
| kevko | ug_egon: btw, could you restart all nova components on controllers ? | 11:32 |
| ug_egon | yes... will do | 11:37 |
| ug_egon | kevko Restarting all nova components (or all components) does not seem to have an effect. | 11:49 |
| kevko | ug_egon: i thought .. :/ | 11:49 |
| kevko | ug_egon: i would like to see how it can be fixed manually | 11:50 |
| kevko | ug_egon: but didn't have to time to investigate | 11:50 |
| kevko | yoctozepto: mgoddard: on some different environment (not openstack, not kolla) where OS is Ubuntu, it happened very annoying thing to us - security patch was released for docker package ..and Ubuntu OS's unattended upgrades downloaded and installed docker package which includes also restart docker service .. I think this should be forbidden in kolla-bootstrap ..or configurable .. that's dangerous i think ..what do you mean ? | 12:00 |
| kevko | (because I've already checked if kolla-ansible do something with unattended upgrades on Ubuntu - and it doesn't ) | 12:01 |
| yoctozepto | kevko: yeah, we can advise against unattended upgrades; and for all kinds of docker upgrades (attended too), I use live-restore in docker config to avoid the mega bounce | 12:02 |
| yoctozepto | though beware becasue it is quirky; docker seems not to track properly all resources it has | 12:02 |
| yoctozepto | I really root for podman support implementation | 12:03 |
| kevko | yoctozepto: but is it enough to advise somewhere in documentation ? or add some if ubuntu && unattended_updates == 'False': place some apt config which disabling it on host and otherwise ? | 12:04 |
| kevko | yoctozepto: podman +1 | 12:11 |
| kevko | yoctozepto: btw, do you have 5 minutes just to reply in https://review.opendev.org/c/openstack/kolla/+/802685 ? Thanks | 12:11 |
| luk4s | Afternoon all. I have noticed a problem with the use of symlinks in the kayobe environments kolla configs and was about to log an issue and submit a fix, but realised that there is already work happening on merging of the kolla configs. | 12:17 |
| luk4s | This will most likely make the symlink issue irrelevant. | 12:18 |
| yoctozepto | kevko: some unattended upgrades are fine; e.g., keeping sshd in perfect shape is nice | 12:19 |
| priteau | luk4s: could still be useful for the current release | 12:21 |
| yoctozepto | kevko: just defaulting all kolla/system/service users to /var/lib/<service> makes sense to me; but do we need overrides? I thought /var/lib/<service> is the default for system users | 12:21 |
| luk4s | What's the best course of action here - shall I wait for the kolla configs merge work to complete and re-visit the issue then? | 12:21 |
| priteau | luk4s: I would say share details on a new storyboard report | 12:21 |
| priteau | https://storyboard.openstack.org/#!/project/openstack/kayobe | 12:21 |
| luk4s | priteau, thanks - I proceed then (and and the issue to the storyboard). | 12:23 |
| yoctozepto | kevko: replied to you here and on both changes; open to questions | 12:27 |
| kevko | yoctozepto: replied also :) | 12:29 |
| yoctozepto | kevko: ok; I don't think it makes much sense to create the tcpdump user anyway; no idea what's it for; we aren't using tcpdump as a service | 12:31 |
| yoctozepto | kill it with fire; create all important users | 12:31 |
| kevko | yoctozepto: hmm, I think this is needed somewhere in masakari ..let me find it | 12:32 |
| kevko | or hacluster | 12:32 |
| yoctozepto | kevko: masakari calls tcpdump but it's not a service; it calls it as root | 12:34 |
| yoctozepto | masakari-hostmonitor to be exact | 12:34 |
| kevko | yoctozepto: yeah, sure, but tcpdump package is creating tcpdump user .. | 12:34 |
| kevko | https://github.com/openstack/masakari-monitors/blob/fe85b8debbd39de15e10e1c042f98fd771b3cfde/masakarimonitors/hostmonitor/host_handler/handle_host.py#L149 | 12:34 |
| kevko | so, if kolla's idea is create all users before package creates the user .. then it should be included in users ..as that user will be in system anyway | 12:36 |
| yoctozepto | yeah, I am aware, but we can ignore it; the packages might decide to install new users that we don't use - we don't care | 12:37 |
| yoctozepto | honestly, what's that user even for | 12:38 |
| yoctozepto | ah, the tcpdump binary drops to its perms to avoid running as full root | 12:39 |
| kevko | yoctozepto: well, we should care in general..because actually that is issue with hacluster .. because hacluster user was not defined in kolla image .. package was installed in base and also in openstack-base ..but with different UID .. | 12:39 |
| yoctozepto | well, the uid of that user is super irrelevant | 12:39 |
| yoctozepto | kevko: yes, but that's a different thing and that thing we want to fix | 12:40 |
| yoctozepto | meh, tcpdump only really relies on the fact this user will have almost no access to filesystem | 12:40 |
| yoctozepto | ignore it, really | 12:40 |
| kevko | yoctozepto: ok, what messagebus ? | 12:41 |
| kevko | i've just added what found in diff .. | 12:41 |
| yoctozepto | ignore too I guess, we are not running messagebus; I would assume some library pulls in some common stuff that creates this user | 12:41 |
| kevko | roger that | 12:42 |
| ug_egon | kevko: i can get servers to boot if i do a 'update services set version=52 where version=40;' on the nova DB. Nova does not like this though () . I'll destroy the test cluster and start fresh. | 12:55 |
| kevko | will be better .. | 12:55 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 12:59 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 12:59 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 12:59 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 13:03 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 13:03 |
| opendevreview | Radosław Piliszek proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 13:51 |
| opendevreview | Radosław Piliszek proposed openstack/kolla master: Add some system users to fix user uid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 13:51 |
| yoctozepto | kevko: I untangled the deps ^ this stack should fix the issue | 13:51 |
| yoctozepto | just need a few tweaks at the base level | 13:51 |
| yoctozepto | hashtags worky (hashtag:"partof-bugfix" OR hashtag:"bugfix") (status:open OR status:merged) | 13:54 |
| yoctozepto | ~> https://review.opendev.org/q/(hashtag:%2522partof-bugfix%2522+OR++hashtag:%2522bugfix%2522)+(status:open+OR+status:merged) | 13:54 |
| yoctozepto | now need to standardise | 13:55 |
| yoctozepto | but I see a potential here | 13:55 |
| kevko | yoctozepto: now i don't understand your comment .. | 14:01 |
| kevko | yoctozepto: you said you have no problem with default value /var/lib/service ..and now saying complicated without flexibility ? | 14:01 |
| kevko | yoctozepto: wait a minute ..have to reboot | 14:02 |
| kevko | here | 14:04 |
| yoctozepto | kevko: you add some steps to allow for configuration but not enough when you can just feed it in the command directly | 14:08 |
| yoctozepto | I will propose what I meant | 14:09 |
| kevko | ok .. | 14:09 |
| opendevreview | Radosław Piliszek proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 14:10 |
| yoctozepto | kevko: clear now? :-) | 14:10 |
| yoctozepto | I really think this is just making the default explicit | 14:11 |
| kevko | aaaaa | 14:12 |
| kevko | :D | 14:12 |
| opendevreview | Radosław Piliszek proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 14:12 |
| yoctozepto | updated commit message | 14:12 |
| yoctozepto | we can merge this; this is independent and functionally noop | 14:13 |
| kevko | hmm, how can I write unit test ? | 14:14 |
| yoctozepto | hmm, good question; there should be some already | 14:15 |
| kevko | honestly, I'm not good in it :D ..so I always trying to retrace some existing | 14:15 |
| kevko | but didn't find something for user | 14:15 |
| yoctozepto | hmm | 14:16 |
| yoctozepto | let me check | 14:16 |
| yoctozepto | yup, there are none | 14:21 |
| yoctozepto | so I can forgive you not writing the unit tests | 14:22 |
| yoctozepto | what else did I want there? | 14:22 |
| yoctozepto | probably a reno | 14:22 |
| kevko | yeah | 14:25 |
| kevko | let me do it | 14:25 |
| opendevreview | Merged openstack/kolla-ansible stable/wallaby: Do not run timesync checks on deployment host https://review.opendev.org/c/openstack/kolla-ansible/+/797584 | 14:37 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Config: allow to set group for user https://review.opendev.org/c/openstack/kolla/+/802663 | 14:50 |
| opendevreview | Michal Arbet proposed openstack/kolla master: base/deb: bump 'system' groups limit beyond Kolla ones https://review.opendev.org/c/openstack/kolla/+/802676 | 14:50 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 14:50 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 14:50 |
| kevko | yoctozepto: did I miss something ? | 14:50 |
| yoctozepto | kevko: what do you mean? | 14:53 |
| kevko | yoctozepto: reno added, added partial-bug and closes-bug to commits ..can u check | 14:54 |
| yoctozepto | kevko: 2x +2, 2x -1 | 14:56 |
| yoctozepto | perfectly balanced | 14:56 |
| kevko | :D | 14:57 |
| opendevreview | Radosław Piliszek proposed openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 15:00 |
| yoctozepto | kevko: anyhow, TIL: https://review.opendev.org/c/openstack/kolla/+/803617 | 15:01 |
| yoctozepto | this will need a rebase on your patches but I pushed not to forget already | 15:01 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Add some system users to fix user uid inconsistencies https://review.opendev.org/c/openstack/kolla/+/802671 | 15:05 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 15:05 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 15:05 |
| kevko | yoctozepto: done, reno added .. | 15:05 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Config: Set default home to /var/lib/{user} https://review.opendev.org/c/openstack/kolla/+/802685 | 15:07 |
| opendevreview | Michal Arbet proposed openstack/kolla master: Avoid the use of --force in groupadd https://review.opendev.org/c/openstack/kolla/+/803617 | 15:07 |
| kevko | yoctozepto: groupadd: GID '65534' already exists << :D :D | 15:35 |
| yoctozepto | kevko: interesting, where do we have it? | 15:35 |
| yoctozepto | so we had a hidden bug | 15:35 |
| kevko | yoctozepto: FROM quay.io/centos/centos:stream8 << | 15:35 |
| kevko | yoctozepto: for example https://zuul.opendev.org/t/openstack/build/e182b72ac9fb4f079255b14e85fb4557/log/kolla/build/000_FAILED_base.log | 15:35 |
| yoctozepto | ducking mongodb | 15:36 |
| yoctozepto | meh, we had a bug | 15:36 |
| kevko | :D | 15:36 |
| kevko | yoctozepto: why there is a comment ? # unused user (mongodb dropped), it should be removed from kolla no ? | 15:38 |
| kevko | it will small gap in uids sequence ..but it doesn't matter i guess | 15:39 |
| yoctozepto | kevko: we can reset the gid too | 15:41 |
| yoctozepto | it is probably some cult value | 15:41 |
| yoctozepto | because it overlaps with the nobody user | 15:41 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Use native fluent-logger instead of tail https://review.opendev.org/c/openstack/kolla-ansible/+/755775 | 15:50 |
| opendevreview | Verification of a change to openstack/kolla-ansible stable/ussuri failed: watcher: add missing become for copying configs https://review.opendev.org/c/openstack/kolla-ansible/+/803289 | 15:51 |
| kevko | yoctozepto: hmm, can't it be a problem | 15:56 |
| kevko | ? | 15:56 |
| yoctozepto | kevko: dropping is fine | 15:57 |
| opendevreview | Michal Arbet proposed openstack/kolla-ansible master: Use native fluent-logger instead of tail https://review.opendev.org/c/openstack/kolla-ansible/+/755775 | 15:58 |
| * yoctozepto off | 16:02 | |
| opendevreview | Merged openstack/kayobe master: Remove references to resolv.conf workaround https://review.opendev.org/c/openstack/kayobe/+/803459 | 16:04 |
| luk4s | priteau, https://storyboard.openstack.org/#!/story/2009102 | 16:41 |
| *** rpittau is now known as rpittau|afk | 16:41 | |
| luk4s | Let me know if its worth pursuing the fix | 16:42 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/773246 | 16:44 |
| opendevreview | Scott Shambarger proposed openstack/kolla master: monasca-thresh: Allow topology check and removal in storm https://review.opendev.org/c/openstack/kolla/+/792751 | 16:51 |
| opendevreview | Piotr Parczewski proposed openstack/kolla master: Remove Prometheus v1 image https://review.opendev.org/c/openstack/kolla/+/803636 | 17:51 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Remove support for Prometheus v1 https://review.opendev.org/c/openstack/kolla-ansible/+/803644 | 19:04 |
| opendevreview | Piotr Parczewski proposed openstack/kolla-ansible master: Remove support for Prometheus v1 https://review.opendev.org/c/openstack/kolla-ansible/+/803644 | 19:07 |
| parallax | dopamine shots | 19:16 |
| yoctozepto | always :-) | 19:19 |
| opendevreview | Seena Fallah proposed openstack/kolla-ansible master: tools/kolla-ansible: make configdir as an optional argument https://review.opendev.org/c/openstack/kolla-ansible/+/803648 | 20:02 |
| opendevreview | Verification of a change to openstack/kolla-ansible master failed: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/773246 | 20:03 |
| -opendevstatus- NOTICE: The Gerrit service on review.opendev.org is going down for a quick restart to adjust its database connection configuration, and should return to service momentarily | 20:03 | |
| opendevreview | Merged openstack/kolla-ansible master: Check config when checking the containers https://review.opendev.org/c/openstack/kolla-ansible/+/773246 | 21:01 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!