| opendevreview | Thomas Goirand proposed openstack/puppet-ironic master: Allow standalone mode for Debian. https://review.opendev.org/c/openstack/puppet-ironic/+/917715 | 14:25 |
|---|---|---|
| opendevreview | Thomas Goirand proposed openstack/puppet-ironic master: Allow standalone mode for Debian. https://review.opendev.org/c/openstack/puppet-ironic/+/917715 | 14:29 |
| opendevreview | Thomas Goirand proposed openstack/puppet-ironic master: Allow standalone mode for Debian. https://review.opendev.org/c/openstack/puppet-ironic/+/917715 | 14:30 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: task_flow: Ensure backend package is installed https://review.opendev.org/c/openstack/puppet-octavia/+/917753 | 15:24 |
| zigo | tkajinam: Hi there! I'm currently upgrading all of my puppet-openstack packages, and I am very annoyed by this patch: | 15:28 |
| zigo | https://review.opendev.org/c/openstack/puppet-tempest/+/900621 | 15:28 |
| zigo | How can I make it work with my packaged version of tempest that has its config file in /etc/tempest/tempest.conf ? | 15:28 |
| tkajinam | zigo, I don't think using that file in /etc is the right choice really, as the module creates a workspace | 15:32 |
| zigo | tkajinam: Well, I have debian specific patches to force using /etc/tempest. | 15:33 |
| tkajinam | if you want then we can probably add a flag not to create a workspace | 15:33 |
| zigo | I assure you that /etc/tempest is the correct FHS location ! :P | 15:33 |
| tkajinam | using system-wide config for tempest is not how tempest is supposed to be used afaik | 15:33 |
| zigo | But that's still how I do, because that's the correct thing to do in packages. | 15:34 |
| zigo | I'm currently reverting your patch, in an attempt to restore my patch on top... | 15:34 |
| zigo | But definitively, that's not the way to go. | 15:34 |
| zigo | I'd prefer to write something correct, and included upstream in puppet-tempest. | 15:34 |
| zigo | I'll see what I can come with. | 15:34 |
| zigo | Thanks. | 15:35 |
| tkajinam | use workspace > something correct | 15:35 |
| tkajinam | that's again what tempest expects | 15:35 |
| tkajinam | tempest in nature allows you to create multiple workspaces and maintaing config file per workspace | 15:35 |
| tkajinam | tempest is not a daemon but a test tool which is more similar to CLI. I think using local config, instead of system wide one, makes sense for tempest | 15:36 |
| zigo | The way I did, is that you just install the tempest package, and run tempest_debian_shell_wrapper. | 15:37 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: task_flow: Ensure backend package is installed https://review.opendev.org/c/openstack/puppet-octavia/+/917753 | 15:37 |
| zigo | No param: runs all tests, but what's blacklisted in /etc/tempest/exclude.conf. | 15:37 |
| zigo | Params: run the tests sent as regex as $1. | 15:37 |
| tkajinam | so it voids the capability of tempest which can run tests over multiple clouds | 15:37 |
| tkajinam | as you maintain only a single config and use it for any run | 15:38 |
| zigo | I don't see the point of having workspace, and I'm not really interested in it, I'd say. | 15:38 |
| zigo | Typically, I install tempest in a VM. | 15:38 |
| tkajinam | ok if you want to pursue that path then we may need an option to disable workspace | 15:40 |
| tkajinam | in a correct manner instead of just creating unused one | 15:40 |
| zigo | Yeah. | 15:41 |
| zigo | I'll see later, I'm reverting your patch in a debian/patches patch, and validate upgrading to all puppet-openstack 24.0.0 first. | 15:41 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: task_flow: Ensure backend package is installed https://review.opendev.org/c/openstack/puppet-octavia/+/917753 | 15:41 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-oslo master: coordination: Install python3-kazoo when zookeeper is used https://review.opendev.org/c/openstack/puppet-oslo/+/917759 | 15:49 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: task_flow: Ensure backend package is installed https://review.opendev.org/c/openstack/puppet-octavia/+/917753 | 17:05 |
| opendevreview | Takashi Kajinami proposed openstack/puppet-octavia master: task_flow: Ensure backend package is installed https://review.opendev.org/c/openstack/puppet-octavia/+/917753 | 18:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!