Monday, 2018-10-15

*** tetsuro has joined #openstack-blazar00:35
*** tetsuro has quit IRC03:57
*** masahito has joined #openstack-blazar04:04
*** tetsuro has joined #openstack-blazar05:50
*** asmita has joined #openstack-blazar06:16
asmitamasahito: Hi, I want your suggestion on this bug: https://bugs.launchpad.net/blazar/+bug/179750206:37
openstackLaunchpad bug 1797502 in Blazar "missing 'before_end_date' value in lease response" [Undecided,New] - Assigned to Neha Alhat (nehaalhat)06:37
*** masahito has quit IRC06:40
*** nehaalhat has joined #openstack-blazar06:40
*** masahito has joined #openstack-blazar06:41
masahitoasmita: sorry, disconnected. What problem are you hitting?06:42
asmitamasahito: I want your suggestion on this bug: https://bugs.launchpad.net/blazar/+bug/1797502, If we decide to add 'before_end_date' in lease response. then we can go by 1. Updating migration script to add column for 'before_end_date' in lease table06:49
openstackLaunchpad bug 1797502 in Blazar "missing 'before_end_date' value in lease response" [Undecided,New] - Assigned to Neha Alhat (nehaalhat)06:49
asmita2. fetch value of 'before_end_date' from events table and show it in lease response06:50
asmitamasahito: or we can go with 'microversioning thing' like other projects(nova, cinder) have implemented.06:52
masahitoI'm not clear the bug report. I can see the before_end_lease time in the response.06:58
*** tetsuro has quit IRC07:01
*** tetsuro has joined #openstack-blazar07:01
*** tetsuro has quit IRC07:02
asmitamasahito: Actually like 'start_date' and 'end_date' there is no value for 'before_end_date'. We can see time for all events in response but still  'start_date' and 'end_date' are shown explicitly and same are stored in DB.'07:05
asmitaIn DB value for 'before_end_date' is not getting stored.07:05
masahitoI got the point.07:09
masahitoThe point is 'before_end_date' key should be a first key in the LEASE object, right?07:10
asmitaYes07:11
masahitoIMO, each date info should be managed in one place, not both of lease table and event table because it causes miss-matching between the lease raw and the event raw.07:14
masahitoSo it's better to either of deleting start/end_date in the lease table or deleting the time in the event column.07:16
masahitoFor the API side, we don't have microversioning now.07:17
masahitoIt's okay to add the new key without microversioning if others agree the change.07:18
asmitamasahito: ok, so now we need to  manage each date info in one place, right?07:20
asmitaand I haven't understood this: miss-matching between the lease raw and the event raw07:24
*** masahito has quit IRC08:05
*** tetsuro has joined #openstack-blazar09:02
*** masahito has joined #openstack-blazar09:14
masahitoasmita: yes. It's better to manage the info in one place.09:14
masahitoIn case of the info is managed in two or more places, a developer would create a bug. For example, A developer only modifies either of them to add a new feature even though they need to modify all of the codes.09:20
asmitamasahito: Okay ,got your point.Will check.09:25
asmitaThanks09:25
*** masahito has quit IRC10:16
*** asmita has quit IRC10:17
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Trivial refactor in instance_plugin.py  https://review.openstack.org/61048511:40
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Add create/delete reservation provider  https://review.openstack.org/58462512:30
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Add create/delete resource class  https://review.openstack.org/57807312:30
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Add update/delete resource inventory  https://review.openstack.org/57863712:30
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Create and delete reservation provider  https://review.openstack.org/58532312:30
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Use placement inventory for instance reservation  https://review.openstack.org/58474412:30
openstackgerritTetsuro Nakamura proposed openstack/blazar master: Adds subnormal path for placement client  https://review.openstack.org/61049712:30
*** tetsuro has quit IRC12:32
openstackgerritPierre Riteau proposed openstack/blazar master: Allow terminating a lease in ERROR state  https://review.openstack.org/60415914:08
*** openstackgerrit has quit IRC15:47
*** openstackgerrit has joined #openstack-blazar15:48
openstackgerritMerged openstack/blazar master: Allow terminating a lease in ERROR state  https://review.openstack.org/60415917:21
openstackgerritPierre Riteau proposed openstack/blazar master: Allow leases to transition to ERROR status when events fail  https://review.openstack.org/58989818:00
openstackgerritPierre Riteau proposed openstack/blazar master: Allow terminating leases in ERROR state when start_lease failed  https://review.openstack.org/61067818:00
*** openstackgerrit has quit IRC21:56
*** openstackgerrit has joined #openstack-blazar21:57
*** openstackgerrit has quit IRC21:58
*** openstackgerrit has joined #openstack-blazar22:00
*** openstackgerrit has quit IRC22:12
*** openstackgerrit has joined #openstack-blazar22:13
*** openstackgerrit has quit IRC22:43

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!