| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2020-01-10 | |||
| 16:14:39 | efried | - Include your plugin in doc/requirements.txt, like: https://opendev.org/openstack/python-openstackclient/src/branch/master/doc/requirements.txt#L27 | |
| 16:14:39 | efried | Then in python-openstackclient you need to: | |
| 16:14:39 | efried | I think that's just standard fare for plugin-ness. But the important point is that the docs will be generated from the argparse helps defined in the classes' get_parser overrides. | |
| 16:14:39 | efried | ==> The right-hand side is the package:class that handles that. | |
| 16:14:39 | efried | ==> The left-hand side is the subcommand syntax, s/ /_/ | |
| 16:14:39 | efried | - Your plugin's setup.cfg needs to define entry_points for each subcommand in this spirit: https://opendev.org/openstack/python-octaviaclient/src/branch/master/setup.cfg#L24 | |
| 16:14:39 | efried | vkmc: Okay, using octavia as an example: | |
| 16:14:40 | efried | - Don't forget to add your new page ^ to the index https://opendev.org/openstack/python-openstackclient/raw/branch/master/doc/source/cli/plugin-commands/index.rst | |
| 16:14:40 | efried | - Set up a page with a title and any desired preamble text (preferably keep that minimal) with an autoprogram-cliff directive that references your entry point, like: https://opendev.org/openstack/python-openstackclient/raw/branch/master/doc/source/cli/plugin-commands/octavia.rst | |
| 16:14:41 | efried | I *think* that's it. | |
| 16:14:42 | efried | What would be really neat is if you could scribble this process into a doc of its own and publish it in python-openstackclient for future contributors... | |
| 16:16:01 | mordred | ++ | |
| 16:16:06 | vkmc | efried, yep, we definitely need to add this | |
| 16:16:09 | vkmc | I can do that | |
| 16:16:27 | efried | thanks! | |
| 16:16:38 | vkmc | thanks! | |
| 16:17:48 | johnsom | vkmc You can also ping me if you have questions. I authored the docs setup for octaviaclient and ironicclient. | |
| 16:18:06 | johnsom | Well, at least the auto generate bits | |
| 16:20:46 | vkmc | johnsom, oh nice, that would be helpful... in case we get stuck with anything there | |
| 16:22:39 | efried | phew, thanks johnsom -- cause all I did was framework stuff, no real clue how the docs themselves work :P | |
| 17:43:15 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Raise hacking to more recent 2.0.0 https://review.opendev.org/702019 | |
| 17:58:04 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Stop testing python2 https://review.opendev.org/686590 | |
| 18:01:09 | smcginnis | mordred: Restored and updated your patch, but let me know if there were other conversations around that that I am not aware of. ^^ | |
| 18:06:53 | smcginnis | dtroyer: Note that dropping py2 support requires a major version bump, so not sure if there are any other major-version-bumping-worthy changes in the works that we might want to get in along with that cycle goal change before doing an official release. ^ | |
| 18:08:51 | dtroyer | smcginnis: I don't have any waiting since we just did 4.0, there may be some that I have not seen yet. But that likely needs some work on the openstackclient repo to match, I think I saw some stuff going on about that earlier this week (sorry I wasn't able to actually follow along with that) | |
| 18:09:45 | smcginnis | I'm not aware either. Really just a heads up in case we had anything we were waiting on for an opportune time. ;) | |
| 18:10:53 | dtroyer | sure, thanks. I have a vague memory that mriedem did run across one or two small things, not sure where to find that now | |
| 18:11:40 | smcginnis | We can see how many major versions we can get over the next cycle or two. | |
| 18:13:41 | smcginnis | Oh, I totally missed https://review.opendev.org/#/c/695427/. Oh well, still a couple little things needed per the cycle goal instructions. | |
| 18:14:26 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Stop testing python2 https://review.opendev.org/686590 | |
| 18:17:01 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Stop testing python2 https://review.opendev.org/686590 | |
| 20:23:22 | openstackgerrit | Sean McGinnis proposed openstack/python-openstackclient master: Raise flake8-import-order version to latest https://review.opendev.org/702036 | |
| 21:36:26 | KeithMnemonic | smcginnis: Thank you very much for helping get the stable openstackclient branches working. it is very appreciated | |
| 21:37:18 | smcginnis | KeithMnemonic: No problem, thanks for getting a lot of that in place to start with. | |
| 21:37:40 | smcginnis | Hopefully we can get those stable branches cleaned up and jobs working again. | |
| 21:38:01 | smcginnis | Only a few outstanding patches, but most look like good fixes to get merged if we can. | |
| 21:39:20 | KeithMnemonic | i agree , it will be a much less stressful when just having to debug the code in the fix itself and not other external issues | |
| #openstack-sdks - 2020-01-11 | |||
| 10:10:32 | openstackgerrit | Artem Goncharov proposed openstack/openstacksdk master: Lay a foundation for the project cleanup https://review.opendev.org/700219 | |
| 17:59:48 | openstackgerrit | Merged openstack/openstackclient master: Constrain openstackclient-check-plugins run on master https://review.opendev.org/701293 | |
| 18:04:25 | openstackgerrit | Merged openstack/openstackclient master: Use python3 for openstackclient-check-plugins job https://review.opendev.org/701378 | |
| #openstack-sdks - 2020-01-12 | |||
| 13:13:43 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 13:40:29 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 14:26:14 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 16:18:39 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 16:19:34 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| 16:58:03 | openstackgerrit | Radosław Piliszek proposed openstack/js-openstack-lib master: WIP: New nodejs jobs https://review.opendev.org/702132 | |
| #openstack-sdks - 2020-01-13 | |||
| 08:22:15 | openstackgerrit | Vishakha Agarwal proposed openstack/python-openstackclient master: Adding options to user cli https://review.opendev.org/697444 | |
| 09:56:08 | openstackgerrit | Bence Romsics proposed openstack/python-openstackclient master: Add command: router add/remove route --route https://review.opendev.org/674325 | |
| 10:52:19 | gundalow | sshnaidm: Hi, I'm back now :) | |
| 11:05:06 | sshnaidm | gundalow, welcome back! | |
| 11:10:46 | sshnaidm | gundalow, will you be available this week for helping with ansible bits of moving OS modules? | |
| 11:11:34 | gundalow | Sure, is there a doc where we are listing the steps? | |
| 11:14:10 | sshnaidm | gundalow, I think we can use our etherpad for that: https://etherpad.openstack.org/p/openstack-ansible-modules | |
| 11:21:13 | gundalow | sshnaidm: perfect, thank you. Could you remind me if we decided on 1) namespace 2) Collection name 3) Host repo | |
| 11:28:39 | sshnaidm | gundalow, updated in etherpad | |
| 11:38:06 | sshnaidm | bbl | |
| 12:43:51 | sshnaidm | gundalow, I'd like to send a mail to Ansible devs/cores to ask them not to merge patches to OS modules w/o exceptional ack from specific OS cores. Where can I send it to? | |
| 12:44:10 | sshnaidm | gundalow, because we want to freeze them in current state now | |
| 14:53:05 | gundalow | hum, Ansibulbot will also automerge. I wonder if we can somehow make the PRs fail CI | |
| 15:51:11 | sshnaidm | gundalow, I think we can, need just to configure openstack job to fail always, but also I'd like to make people aware about it | |
| 15:51:24 | sshnaidm | gundalow, do you have kind of "ansible-dev" list..? | |
| 16:05:50 | KeithMnemonic | am i reading this failure correctly, that it also due to the thread on openstack-discuss about setuptools and python2? https://zuul.opendev.org/t/openstack/build/372bc8c66d794210aeb571491ed6e893 | |
| 16:25:27 | dtantsur | KeithMnemonic: everything is broken, yes. the infra team is about to work it around though. | |
| 16:28:55 | KeithMnemonic | thanks! | |
| 16:34:11 | mordred | should be fixed now | |
| 16:45:27 | KeithMnemonic | thanks mordred. Does anyone have any objections for me starting the cherry-pick to stein on this change while I am waiting for a WF+1? ? https://review.opendev.org/701299. That way i can start seeing if there are other issues on stable/stein. | |
| 16:45:47 | KeithMnemonic | or would you prefer i wait until it merges | |
| 16:48:45 | mordred | KeithMnemonic: go for it! stein has been a little shaky recently, so might as well go ahead and make sure things work | |
| 16:49:41 | KeithMnemonic | thanks morded | |
| 16:50:09 | KeithMnemonic | argh mordred: :-) | |
| 16:54:53 | mordred | :) | |
| 16:55:08 | dtantsur | note that if you press the cherry-pick button now, the patch will be missing the required "cherry picked from" line | |
| 16:55:52 | KeithMnemonic | got it thanks, there is a merge conflict so i need to do it by hand anyway | |
| 17:24:57 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Add unit tests and release note for dns_publish_fixed_ip https://review.opendev.org/702266 | |
| 18:24:18 | openstackgerrit | Merged openstack/python-openstackclient master: Raise hacking to more recent 2.0.0 https://review.opendev.org/702019 | |
| 18:24:20 | openstackgerrit | Merged openstack/python-openstackclient master: Raise flake8-import-order version to latest https://review.opendev.org/702036 | |
| 19:03:26 | efried | johnsom: Before I go digging, can you remind me where we are on the osc-lib business that was holding up https://review.opendev.org/#/c/692931/ ? | |
| 19:04:28 | efried | looks like https://review.opendev.org/#/c/693267/ is still open. | |
| 19:04:33 | johnsom | efried This is stuck: https://review.opendev.org/#/c/662864/ | |
| 19:04:41 | johnsom | On this: https://review.opendev.org/#/c/693267 | |
| 19:05:16 | efried | johnsom: is it just that we need the latter approved before the former can pick it up properly? | |
| 19:05:25 | efried | approved/released/constrainted/etc? | |
| 19:05:52 | johnsom | I think so, but it has been a long time, so the details are fuzzy | |
| 19:10:57 | efried | johnsom: Can we swap back in? Cause off the top I'm not sure I understand how https://review.opendev.org/#/c/662864/ needing https://review.opendev.org/#/c/693267 should hold up https://review.opendev.org/#/c/692931/ | |
| 19:14:12 | johnsom | efried Yeah, the whole issue was the addition of "enhanced_help" you did. It broke other pending patches and is a bit of an oddity. So, the discussion was if we revert the "enhanced_help" change or if we move forward with it and add it to osc-lib. | |
| 19:14:32 | efried | johnsom: but how does that relate to autogenerating docs for identity? | |
| 19:14:34 | johnsom | efried if we reverted, this https://review.opendev.org/#/c/692931/ should not merge either | |
| 19:16:01 | johnsom | efried that patch drops the versioning callout on the --tag options | |
| 19:16:28 | efried | tbc, ...31 doesn't do anything with enhance_help. If we revert that, we would have to revert down that (already merged) series as well. | |
| 19:16:40 | johnsom | Yep | |
| 19:16:41 | efried | if my delta makes the versioning of --tags wrong, that's its own issue. | |
| 19:16:48 | johnsom | It was a pretty nasty mess as I remember | |
| 19:19:32 | efried | johnsom: I'm rechecking to get a new docs build to be sure, but looking at the code, it should be the case that we're only adding the docs for --tags to the v3 doc. | |
| 19:19:35 | johnsom | It looks like Dean was ok with moving forward with the "enhanced_help" path, but it looks like the cores never confirmed that | |
| 19:19:37 | efried | which is what we want to do. | |
| 19:19:46 | efried | johnsom: "the cores" would be... Dean. | |
| 19:19:57 | johnsom | lol, well | |
| 19:20:40 | efried | I think dtroyer was waiting for you to confirm that that solution would work for you. | |
| 19:20:47 | efried | before +2ing | |
| 19:20:53 | johnsom | There are five folks on the list, but yeah, I understand this area is lacking at the moment | |