Wednesday, 2020-09-09

*** LinPeiWen has joined #openstack-sdks01:23
*** ricolin_ has joined #openstack-sdks01:31
*** enriquetaso has quit IRC02:27
*** johnavp1989 has left #openstack-sdks03:57
*** evrardjp has quit IRC04:33
*** evrardjp has joined #openstack-sdks04:33
*** gtema has joined #openstack-sdks06:11
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: [WIP] Add additional compute flavor operations  https://review.opendev.org/75007206:15
*** gtema has quit IRC06:23
openstackgerritMerged openstack/osc-lib master: Update master for stable/victoria  https://review.opendev.org/75033406:25
*** ralonsoh has joined #openstack-sdks06:34
*** gtema has joined #openstack-sdks06:39
*** slaweq has joined #openstack-sdks06:50
openstackgerritMerged openstack/openstacksdk master: Update master for stable/victoria  https://review.opendev.org/75033807:25
*** jpena|off is now known as jpena07:30
*** nightmare_unreal has joined #openstack-sdks07:32
openstackgerritArtem Goncharov proposed openstack/cliff master: Revert "Capturing argparse errors due to problem with cmd2"  https://review.opendev.org/75058507:35
openstackgerritArtem Goncharov proposed openstack/cliff master: Enable osc-functional-devstack-tips tests  https://review.opendev.org/75058807:47
openstackgerritArtem Goncharov proposed openstack/cliff master: Enable osc-functional-devstack-tips job  https://review.opendev.org/75058807:49
openstackgerritjayaditya gupta proposed openstack/python-openstackclient master: Fix --image-property option in 'create server'  https://review.opendev.org/74640507:50
*** tosky has joined #openstack-sdks07:57
*** jpich has joined #openstack-sdks07:58
*** slaweq has quit IRC08:20
*** slaweq has joined #openstack-sdks08:29
*** nikparasyr has joined #openstack-sdks08:43
*** slaweq has quit IRC08:49
*** lxkong has joined #openstack-sdks09:28
openstackgerritjayaditya gupta proposed openstack/python-openstackclient master: Fix --image-property option in 'create server'  https://review.opendev.org/74640509:28
*** gtema has quit IRC09:28
*** gtema has joined #openstack-sdks09:31
openstackgerritArtem Goncharov proposed openstack/cliff master: Enable osc-functional-devstack-tips job  https://review.opendev.org/75058809:34
*** lxkong has quit IRC09:34
openstackgerritArtem Goncharov proposed openstack/cliff master: Revert "Capturing argparse errors due to problem with cmd2"  https://review.opendev.org/75058509:34
*** dtantsur|afk is now known as dtantsur09:35
*** ricolin_ has quit IRC09:38
openstackgerritArtem Goncharov proposed openstack/openstacksdk master: Add additional compute flavor operations  https://review.opendev.org/75007209:52
*** ralonsoh has quit IRC10:01
*** ralonsoh has joined #openstack-sdks10:02
*** jpich has quit IRC10:09
*** jpich has joined #openstack-sdks10:09
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: [WIP] Switch compute flavors from novaclient/direct to SDK  https://review.opendev.org/75015110:30
*** gtema has quit IRC10:59
*** mgariepy has quit IRC11:06
*** gtema has joined #openstack-sdks11:26
*** jpena is now known as jpena|lunch11:30
*** mgariepy has joined #openstack-sdks12:09
*** thiagop has joined #openstack-sdks12:16
gtemadtroyer, mordred, dtantsur is anybody here? I need a short "consultation"12:30
* dtantsur is here12:30
gtemahopefully both other would be able to read/participate...12:30
gtemaSDK is normalizing resource attribute names, while OSC used to return whatever is on the server side12:31
gtemaI prepared a patch to switch flavor operations to use SDK (https://review.opendev.org/#/c/750151/)12:31
gtemaand here comes the question, what should we do with the fact, that osc and SDK use different names12:32
gtemasimply changing osc output might break users12:32
*** jpena|lunch is now known as jpena12:32
gtemaduplicating data (sdk and server-side names) might be an option, but maybe also confusing12:33
gtemaideas?12:34
dtantsurI think our to_dict calls have a way to return server values?12:35
dtantsurI agree that changing OSC is less than ideal12:35
gtemayes, but it's either one or another12:35
gtemato stick forever to server names (while goal of SDK is to normalize things) is also not good12:36
gtemamaybe at some point do a major releasee switching all to SDK names?12:37
gtemathis will however first need everything uses SDK12:37
dtantsurexactly12:43
gtemawhat's also not to forget it is easier to use osc output with sdk names (is_disabled vs OS-FLV-DISABLED:disabled)12:44
gtemaI mean if you for example want to wrap it in script or so12:44
gtemaso would you prefer leaving server side names or having both?12:55
*** enriquetaso has joined #openstack-sdks12:56
openstackgerritDoug Hellmann proposed openstack/cliff master: change help action to use its own exception for exit  https://review.opendev.org/75064813:05
dmsimardI'm looking at potentially implementing novaclient's live-migration-abort in openstackclient, thought it'd be easy enough: https://github.com/openstack/python-novaclient/blob/003ac57d9af74aa4658a7bf6cc6b6b3bafa58c11/novaclient/v2/server_migrations.py#L70-L8013:16
dmsimardso I guess what I need is somewhere inside "compute_client.servers" but I'm not sure how to do the right delete call ? I see compute_client.servers.delete used for deleting servers themselves and compute_client.servers.delete_meta for properties13:17
gtemadmismard, you don't need to send delete call. Currently compute_client is novaclient13:20
gtemaso you need to get ServerMigrationsManager from novaclient and directly invoke live_migration_abort (as far as I can see what novaclient does)13:21
dmsimardyeah actually I just realized compute_client.migrations is a thing13:23
gtemaI suggest you have a look in compute.v2.server.ResizeRevert and do thing similarly13:23
dmsimardResizeRevert has a handy function for it, server.revert_resize() but I don't see anything like that for aborting migrations :/ http://paste.openstack.org/raw/797641/13:30
dmsimardoptimistically trying server.migrations.live_migration_abort doesn't work either: 'MigrationManager' object has no attribute 'live_migration_abort'13:31
dmsimardsorry I'm kind of a noob :D13:31
gtemaI was more meaning type of invocation: you use find to get your migration passing it novaclient13:31
gtemaand when you get the migration itself (instance of ServerMigrationsManager) you call abort directly on it13:32
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: Switch compute flavors from novaclient/direct to SDK  https://review.opendev.org/75015113:34
fricklergtema: dtantsur: mordred: regarding https://review.opendev.org/750585 , as that is blocking osc currently, I'd vote to revert now and fix later, but I'm uncertain about the freeze status, does that have any impact? do we need a bug report attached in order to gain a FFE?13:42
gtemahonestly I have no clue about that (https://review.opendev.org/#/c/750648/ should also help to get this fixed)13:44
dtantsurfrickler: I think both patches in question are bug fixes, and are not affected by *our* FFE, but you may check with the release team re release freeze13:44
dtantsurand requirements freeze13:44
dtantsuri.e. I think you'll need an FFE from them, but not from us13:44
*** vishakha has joined #openstack-sdks13:47
gtemahonestly I am a bit confused here: cliff is cycle-with-interim. Last release for victoria is 3.4.0, which doesn't have this bug yet13:52
gtemain victoria branch this is not existing either13:52
gtemaso perhaps we do not need any FFE if we do not do release at the moment. Next release must contain then bad patch and fix13:53
*** slaweq has joined #openstack-sdks14:03
dmsimardgtema: maybe I'm missing something obvious -- I can find the migration object but there's no delete or abort method available: http://paste.openstack.org/raw/797644/14:10
*** ricolin_ has joined #openstack-sdks14:11
*** ricolin_ has quit IRC14:13
gtemadmsimard, I guess the problem is that you use "migrations" instead of "server_migrations"14:16
fricklergtema: oh, so this only affects -tips testing, that's good. also dhellman's fix works for me, so we should be able to go forward with that14:16
gtemafrickler - yes, I think it should be fine this way. We land the fix and do not trigger any release so far14:17
dmsimardgtema: oh you're right! <314:23
openstackgerritArtem Goncharov proposed openstack/cliff master: Enable osc-functional-devstack-tips job  https://review.opendev.org/75058814:27
*** outbrito has joined #openstack-sdks14:40
gtemaI'm going to approve cliff fix unless somebody objects14:42
*** thiagop has quit IRC14:43
*** outbrito is now known as thiagop14:51
openstackgerritZane Bitter proposed openstack/cliff master: Exit gracefully on Ctrl-C  https://review.opendev.org/74806014:53
*** nikparasyr has left #openstack-sdks15:02
*** slaweq has quit IRC15:04
*** slaweq has joined #openstack-sdks15:15
*** slaweq has quit IRC15:20
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: Switch compute flavors from novaclient/direct to SDK  https://review.opendev.org/75015115:49
*** aj_mailing has joined #openstack-sdks16:19
*** jpich has quit IRC16:21
*** mgariepy has quit IRC16:23
openstackgerritMerged openstack/cliff master: change help action to use its own exception for exit  https://review.opendev.org/75064816:24
gtemayeah, and the terrifying email came in :(16:25
*** nightmare_unreal has quit IRC16:49
*** dtantsur is now known as dtantsur|afk16:55
*** slaweq has joined #openstack-sdks16:56
openstackgerritArtem Goncharov proposed openstack/python-openstackclient master: Switch openstack console log show operation to use OpenStackSDK  https://review.opendev.org/75073817:03
*** mgariepy has joined #openstack-sdks17:07
*** gtema has quit IRC17:10
*** jpena is now known as jpena|off17:11
*** slaweq has quit IRC17:12
*** slaweq has joined #openstack-sdks17:16
*** slaweq has quit IRC17:21
*** gtema has joined #openstack-sdks17:41
*** gtema has quit IRC17:46
openstackgerritZane Bitter proposed openstack/cliff master: Exit gracefully on Ctrl-C  https://review.opendev.org/74806018:03
*** ralonsoh has quit IRC18:05
*** LinPeiWen has quit IRC18:36
*** slaweq has joined #openstack-sdks19:17
*** tosky has quit IRC19:37
*** gtema has joined #openstack-sdks19:42
*** gtema has quit IRC19:55
*** openstackgerrit has quit IRC20:17
*** slaweq has quit IRC20:51
*** slaweq has joined #openstack-sdks20:54
*** slaweq has quit IRC20:59
*** thiagop has quit IRC21:12
*** slaweq has joined #openstack-sdks21:26
*** openstackgerrit has joined #openstack-sdks21:27
openstackgerritGhanshyam Mann proposed openstack/python-openstackclient master: [goal] Migrate testing to Focal  https://review.opendev.org/75081421:27
*** slaweq has quit IRC21:31
openstackgerritGhanshyam Mann proposed openstack/os-service-types master: Fix pep8 testing for ubuntu focal  https://review.opendev.org/75081521:51
*** gtema has joined #openstack-sdks21:51
openstackgerritGhanshyam Mann proposed openstack/os-client-config master: Fix l-c testing for ubuntu focal  https://review.opendev.org/75081621:52
openstackgerritGhanshyam Mann proposed openstack/requestsexceptions master: Fix pep8 testing for ubuntu focal  https://review.opendev.org/75081721:53
gmannsdk team ^^ these are fixes you might be seeing as tox base jobs are running on Focal now21:54
openstackgerritGhanshyam Mann proposed openstack/python-openstackclient master: [goal] Migrate testing to Focal  https://review.opendev.org/75081421:55
*** gtema has quit IRC21:56
*** vishakha has quit IRC22:26
openstackgerritGhanshyam Mann proposed openstack/osc-lib master: Fix l-c testing for ubuntu focal  https://review.opendev.org/75083022:26
openstackgerritGhanshyam Mann proposed openstack/openstacksdk master: Fix l-c testing for ubuntu focal  https://review.opendev.org/75083122:28
openstackgerritMerged openstack/cliff master: Exit gracefully on Ctrl-C  https://review.opendev.org/74806022:53
*** enriquetaso has quit IRC23:13
*** gtema has joined #openstack-sdks23:52
*** gtema has quit IRC23:57

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