| *** tosky has quit IRC | 00:51 | |
| *** artom has quit IRC | 02:46 | |
| *** whoami-rajat_ has joined #openstack-sdks | 02:56 | |
| *** whoami-rajat_ is now known as whoami-rajat | 03:16 | |
| *** enriquetaso has quit IRC | 04:11 | |
| *** ysandeep|holiday is now known as ysandeep | 04:33 | |
| *** lifeless has quit IRC | 04:42 | |
| *** udesale has joined #openstack-sdks | 05:09 | |
| *** whoami-rajat has quit IRC | 05:28 | |
| *** evrardjp has quit IRC | 05:33 | |
| *** evrardjp has joined #openstack-sdks | 05:33 | |
| *** LinPeiWen has quit IRC | 06:02 | |
| *** LinPeiWen has joined #openstack-sdks | 06:13 | |
| *** ralonsoh has joined #openstack-sdks | 06:18 | |
| *** diablo_rojo has quit IRC | 06:36 | |
| *** lifeless has joined #openstack-sdks | 06:49 | |
| *** whoami-rajat_ has joined #openstack-sdks | 06:56 | |
| *** khomesh24 has joined #openstack-sdks | 06:59 | |
| *** slaweq has joined #openstack-sdks | 07:11 | |
| *** gtema has joined #openstack-sdks | 07:21 | |
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Container module for management Swift containers https://review.opendev.org/c/openstack/ansible-collections-openstack/+/779712 | 08:17 |
|---|---|---|
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Container module for management Swift containers https://review.opendev.org/c/openstack/ansible-collections-openstack/+/779712 | 08:26 |
| *** tosky has joined #openstack-sdks | 08:51 | |
| *** jpich has joined #openstack-sdks | 09:04 | |
| *** ysandeep is now known as ysandeep|lunch | 09:41 | |
| *** dtantsur|afk is now known as dtantsur | 09:51 | |
| *** jpich has quit IRC | 10:33 | |
| *** jpich has joined #openstack-sdks | 10:33 | |
| *** smcginnis has joined #openstack-sdks | 10:38 | |
| *** slaweq has quit IRC | 11:00 | |
| *** slaweq has joined #openstack-sdks | 11:02 | |
| *** udesale_ has joined #openstack-sdks | 11:04 | |
| *** udesale has quit IRC | 11:07 | |
| *** brinzhang0 has quit IRC | 11:11 | |
| *** jpich has quit IRC | 11:28 | |
| *** jpich has joined #openstack-sdks | 11:28 | |
| *** ysandeep|lunch is now known as ysandeep | 11:36 | |
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Container module for management Swift containers https://review.opendev.org/c/openstack/ansible-collections-openstack/+/779712 | 11:37 |
| *** enriquetaso has joined #openstack-sdks | 12:04 | |
| *** senrique_ has joined #openstack-sdks | 12:28 | |
| *** smcginnis has quit IRC | 12:28 | |
| *** enriquetaso has quit IRC | 12:28 | |
| *** artom has joined #openstack-sdks | 12:44 | |
| *** tkajinam has quit IRC | 12:54 | |
| *** smcginnis has joined #openstack-sdks | 13:05 | |
| openstackgerrit | Polina Gubina proposed openstack/ansible-collections-openstack master: Container module for management Swift containers https://review.opendev.org/c/openstack/ansible-collections-openstack/+/779712 | 13:19 |
| *** jpich has quit IRC | 13:31 | |
| *** jpich has joined #openstack-sdks | 13:31 | |
| mordred | gtema: so - I pushed up a change yesterday that was the result of me falling down a little rabbit hole looking at sdk in lgtm.com: https://lgtm.com/projects/g/openstack/openstacksdk/?mode=list | 13:51 |
| gtema | wow | 13:52 |
| mordred | many of the things are competely lame, and its warnings about multiple calls of constructors are not super accurate. but the duplicate code warnings seem to be reasonable | 13:52 |
| *** smcginnis has quit IRC | 13:52 | |
| mordred | (there's several other things where it's right and where I'm annoyed that pyflakes isn't flagging - like redefined variables and unused variables and stuff) | 13:52 |
| gtema | well, there is always place for tools to be different | 13:53 |
| mordred | yah | 13:54 |
| gtema | unrelated question to https://opendev.org/opendev/system-config/src/branch/master/playbooks/roles/base/server/templates/sshd_config.j2#L84 | 13:55 |
| gtema | I'm trying to reuse system-config pattern in my case and now digging the "undocumented" things | 13:56 |
| gtema | is the bastion_ipv4 (what is applied to all hosts behind bridge) the ip of the bridge itself? Or how otherwise bridge connected to hosts? | 13:56 |
| mordred | ip of bridge itself. | 13:59 |
| gtema | okay, and is the bridge behind the some form of the bastion? | 13:59 |
| mordred | nope. it is the bastion | 13:59 |
| gtema | cool, thanks | 13:59 |
| *** khomesh24 has quit IRC | 14:00 | |
| mordred | basically - it's where ansible is run from - so we allow it to connect to the remote hosts at root - but we only allow inbound root from bridge | 14:00 |
| gtema | was digging quite a while to figure out how the secrets themselves are provisioned to bridge | 14:00 |
| mordred | there's a git repo of secrets local to bridge that's managed manually there - it's the secretest part of the equation | 14:00 |
| gtema | and just to double check - they are provisioned under /etc/ansible/hosts/host_vars/bastion.o.o.yaml? | 14:01 |
| mordred | under /etc/ansible/hosts/host_vars/ and /etc/ansible/hosts/group_vars/ | 14:02 |
| gtema | cool | 14:02 |
| gtema | not the trivial usage pattern for the CD, but I clearly understand why | 14:02 |
| gtema | another infra question, how search.openstack.org is working? | 14:02 |
| mordred | yah - this is why our system-config jobs add bridge as an inventory host and the zuul job runs ansible on bridge in a shell task | 14:03 |
| gtema | I am willing to also implement searching | 14:03 |
| mordred | do you mean https://codesearch.opendev.org/ ? | 14:03 |
| gtema | nope, the search.openstack.org used by docs.openstack.org | 14:03 |
| mordred | I honestly don't know - I think the foundation might have set that up? fungi or clarkb might know | 14:04 |
| gtema | okay, thanks a lot anyway | 14:05 |
| gtema | btw, with https://review.opendev.org/c/openstack/openstacksdk/+/766783 I have figured out that we break ansible a lot | 14:05 |
| gtema | already clarified with the collection, that we will update all modules to enforce doing to_dict for whatever SDK returns to ansible | 14:06 |
| fungi | search.openstack.org does seem to be something related to the foundation-run openstack.org website, yes | 14:06 |
| gtema | as usual - Ansible is doing some weird things, but we can't really blame it | 14:06 |
| gtema | fungi - the question is basically which software is running there and how it indexes the content | 14:07 |
| fungi | i can ask the webdev folks, it's almost certainly outsourced to a third party to maintain | 14:07 |
| gtema | fungi - no worries. In that case it is most likely to be something more complex then I was supposing | 14:08 |
| gtema | mordred: I think I will shortly start a "r1" branch where we can complete switch of the cloud layer to proxy, so that cloud returns always resources, and not munches | 14:09 |
| fungi | gtema: what were you wanting to search with it? maybe they're willing to add something | 14:09 |
| gtema | and once it is complete we can merge it and cut a release, which is partly breaking interface (depending on your usage pattern) | 14:09 |
| gtema | fungi - no, I am "mimicing" OpenStack infra setup for our cloud and need to find a way to index html content in docs "portal" | 14:10 |
| fungi | aha, got it | 14:10 |
| gtema | just want to understand how it works | 14:11 |
| fungi | if it were a single sphix document, then sphinx's search capability would be an option | 14:11 |
| gtema | yeah, but we talk about multiple subprojects pushing their own content | 14:11 |
| fungi | but if it's a copy of the openstack project docs, that's all different projects so the indices aren't merged | 14:11 |
| fungi | right | 14:11 |
| gtema | ok, will think deeper on possibilities | 14:12 |
| gtema | thanks | 14:13 |
| fungi | classical approach would be to rely on one of the numerous open source solutions which spider your site content and create trigram indices, though you'd need to adjust the sphinx theme to embed the form field calling out to whatever index interface it provides | 14:13 |
| gtema | exact | 14:14 |
| *** smcginnis has joined #openstack-sdks | 14:28 | |
| *** whoami-rajat_ is now known as whoami-rajat | 14:40 | |
| *** ysandeep is now known as ysandeep|dinner | 15:00 | |
| *** artom has quit IRC | 15:04 | |
| *** artom has joined #openstack-sdks | 15:16 | |
| *** LinPeiWen has quit IRC | 16:00 | |
| *** ysandeep|dinner is now known as ysandeep | 16:18 | |
| *** Hybrid512 has quit IRC | 17:04 | |
| *** jpich has quit IRC | 17:19 | |
| *** udesale_ has quit IRC | 17:25 | |
| *** ralonsoh has quit IRC | 17:55 | |
| *** artom has quit IRC | 18:06 | |
| *** artom has joined #openstack-sdks | 18:08 | |
| *** dtantsur is now known as dtantsur|afk | 18:10 | |
| *** gtema has quit IRC | 18:21 | |
| *** gtema has joined #openstack-sdks | 18:24 | |
| *** smcginnis has quit IRC | 18:25 | |
| *** smcginnis has joined #openstack-sdks | 18:30 | |
| *** gtema has quit IRC | 19:02 | |
| yoctozepto | morning gentlefolks, a little emergency in here | 19:05 |
| yoctozepto | the newly released openstacksdk includes a breaking change | 19:06 |
| yoctozepto | at least for masakari | 19:06 |
| yoctozepto | (and it's not the one for masakari) | 19:06 |
| yoctozepto | this -> https://review.opendev.org/c/openstack/openstacksdk/+/776205 | 19:06 |
| yoctozepto | I don't know about other projects but all might be impacted in subtle ways | 19:06 |
| mordred | gtema: discovery in #zuul - api output can be different between list servers and get server - in this particular case, list shows a server stuck in building, but get doesn't show it existing at all. nodepool does the right thing because it always uses list and client-side filter - but the behavior can be confusing from OSC | 19:07 |
| mordred | yoctozepto: looking | 19:07 |
| yoctozepto | this patch essentially adds new props on listings | 19:07 |
| mordred | yoctozepto: I don't have enough context here, sorry. ... and poop, gtema isn't here | 19:08 |
| mordred | yoctozepto: why is that breaking things? | 19:08 |
| yoctozepto | because, at least masakari, has tests that compare things 1:1 | 19:08 |
| mordred | oh - that's a bug in masakari | 19:08 |
| yoctozepto | :D | 19:08 |
| yoctozepto | could be | 19:09 |
| mordred | the sdk interface contract does not consider adding properties to be a breaking change | 19:09 |
| mordred | it is always fair for us to add without warning. we will not, however, take properties away - that is a breaking change | 19:09 |
| mordred | so tests that test for 1:1 are bad tests | 19:09 |
| mordred | (sorry, I know that doesn't put you in an awesome position at this moment) | 19:10 |
| yoctozepto | ok, note taken | 19:10 |
| yoctozepto | haha, I have not written them | 19:10 |
| mordred | I know :P) | 19:10 |
| yoctozepto | I don't like the "functional" tests that we have | 19:10 |
| yoctozepto | they are like glorified unit tests that require full devstack deployment :D | 19:10 |
| mordred | gross | 19:11 |
| mordred | also - we have unit tests that test api structures in sdk - so masakari doesn't really need to also test that the sdk works | 19:11 |
| yoctozepto | yeah, it's more like half-bottomed thing that should be in tempest but otoh is still too basic to go there | 19:11 |
| * yoctozepto crying in despair | 19:12 | |
| yoctozepto | :D | 19:12 |
| yoctozepto | ok, so I will adapt these tests to make sense and not break | 19:13 |
| yoctozepto | and you know what happened if any other project comes crying their ci broke for no apparent reason :D | 19:13 |
| mordred | :) | 19:19 |
| mordred | yoctozepto: might be worth making a masakari functional test job that uses the functional tests in sdk (maybe adding more) and run it on both sdk and masakari changes - we do that for a few other services | 19:20 |
| yoctozepto | mordred: ack, that would likely make sense | 19:21 |
| yoctozepto | I think sdk does only unit tests for masakari atm | 19:21 |
| yoctozepto | I honestly can't wrap my head around the usefulness of the now-failing test | 19:26 |
| mordred | yoctozepto: what do you want to bet it's testing that the test tests what the test tests? | 19:33 |
| yoctozepto | WHA? :D | 19:35 |
| *** gtema has joined #openstack-sdks | 19:36 | |
| openstackgerrit | Merged openstack/ansible-collections-openstack master: security_group_rule: add support ipv6-icmp https://review.opendev.org/c/openstack/ansible-collections-openstack/+/778627 | 20:00 |
| yoctozepto | mordred: by the look at those test, you might be right that they would gladly belong in openstacksdk's functional ones | 20:12 |
| yoctozepto | mordred: I have proposed https://review.opendev.org/c/openstack/masakari/+/780319 - I would appreciate if you could take a look at the change as well as other functional tests around and advise me | 20:13 |
| openstackgerrit | Dylan Zapzalka proposed openstack/openstacksdk master: Added support for the Limits resource https://review.opendev.org/c/openstack/openstacksdk/+/780323 | 20:23 |
| *** gtema has quit IRC | 20:30 | |
| *** artom has quit IRC | 21:09 | |
| *** whoami-rajat has quit IRC | 21:13 | |
| *** smcginnis has quit IRC | 21:38 | |
| *** smcginnis has joined #openstack-sdks | 21:44 | |
| *** smcginnis has quit IRC | 21:53 | |
| *** mgariepy has quit IRC | 23:21 | |
| *** mgariepy has joined #openstack-sdks | 23:34 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!