Friday, 2019-07-26

*** gyee has quit IRC00:19
*** btully has joined #openstack-horizon00:30
*** btully has quit IRC00:34
*** BjoernT has joined #openstack-horizon00:45
*** BjoernT has quit IRC00:47
*** BjoernT has joined #openstack-horizon00:48
*** BjoernT has quit IRC00:48
*** BjoernT has joined #openstack-horizon00:49
*** BjoernT has quit IRC00:49
*** BjoernT has joined #openstack-horizon00:49
*** BjoernT has quit IRC00:50
*** BjoernT has joined #openstack-horizon00:50
*** BjoernT has quit IRC00:50
*** BjoernT has joined #openstack-horizon00:51
*** BjoernT has quit IRC00:51
*** betherly has joined #openstack-horizon00:51
*** BjoernT has joined #openstack-horizon00:52
*** BjoernT has quit IRC00:52
*** BjoernT has joined #openstack-horizon00:54
*** BjoernT has quit IRC00:54
*** betherly has quit IRC00:56
*** betherly has joined #openstack-horizon01:13
*** betherly has quit IRC01:17
*** betherly has joined #openstack-horizon01:33
*** betherly has quit IRC01:38
*** betherly has joined #openstack-horizon01:54
*** betherly has quit IRC01:59
*** tinwood has quit IRC02:10
*** tinwood has joined #openstack-horizon02:12
*** betherly has joined #openstack-horizon02:15
*** betherly has quit IRC02:20
*** whoami-rajat has joined #openstack-horizon02:26
*** betherly has joined #openstack-horizon02:57
openstackgerritpengyuesheng proposed openstack/horizon master: After deleting the server group, jump to the correct page  https://review.opendev.org/64801302:58
*** betherly has quit IRC03:01
*** betherly has joined #openstack-horizon03:28
*** betherly has quit IRC03:33
*** betherly has joined #openstack-horizon03:48
*** betherly has quit IRC03:53
*** btully has joined #openstack-horizon04:07
*** betherly has joined #openstack-horizon04:09
*** btully has quit IRC04:12
*** betherly has quit IRC04:14
*** betherly has joined #openstack-horizon04:30
*** betherly has quit IRC04:35
*** BjoernT has joined #openstack-horizon04:58
*** betherly has joined #openstack-horizon05:01
*** BjoernT has quit IRC05:01
*** BjoernT has joined #openstack-horizon05:02
*** BjoernT has quit IRC05:02
*** betherly has quit IRC05:06
*** zbr has joined #openstack-horizon05:45
*** rcernin has quit IRC06:08
*** tesseract has joined #openstack-horizon07:03
*** awalende has joined #openstack-horizon07:19
*** tosky has joined #openstack-horizon07:34
*** pcaruana has joined #openstack-horizon07:44
openstackgerritOpenStack Proposal Bot proposed openstack/horizon master: Imported Translations from Zanata  https://review.opendev.org/67291308:35
*** e0ne has joined #openstack-horizon09:16
*** ChanServ sets mode: +o e0ne09:16
openstackgerritRadomir Dopieralski proposed openstack/horizon master: Display password_regex_description when password change fails  https://review.opendev.org/67292709:26
openstackgerritRadomir Dopieralski proposed openstack/horizon master: Display password_regex_description when password change fails  https://review.opendev.org/67292709:31
*** btully has joined #openstack-horizon09:34
*** btully has quit IRC09:38
*** priteau has joined #openstack-horizon09:53
openstackgerritStephen Crawley proposed openstack/horizon master: Manually registering 2 Dashboards breaks get_dashboards() (python3)  https://review.opendev.org/67044610:50
*** ksdean has quit IRC10:59
*** mvkr has joined #openstack-horizon11:16
*** btully has joined #openstack-horizon11:23
*** btully has quit IRC11:27
*** irclogbot_0 has quit IRC11:53
*** irclogbot_1 has joined #openstack-horizon11:54
*** ksdean has joined #openstack-horizon12:12
*** awalende has quit IRC12:31
*** mlycka has joined #openstack-horizon12:32
mlyckaamotoki: ping?12:32
*** waverider has joined #openstack-horizon12:34
amotokimlycka: hi12:46
mlyckaamotoki: Hey. Ivan (thanks btw) had my API thing rechecked by; (https://review.opendev.org/#/c/670513/) and it passed. You didn't happen to find a second to take a peek at it?12:48
amotokimlycka: it is in my usual list as usual but the list is long in horizon as the number of reviewers and patches are not balanced well and it takes time to rotate it :p12:52
amotokiI am not a full-time developer only for horizon, so the progress tends to be slow.12:52
mlyckaYeah, I figured, just thought I'd ask. Thanks.12:53
cdev3the CSV output in project->compute-> Overview does not take into account the fact that instances have been shelved, and that is different from "openstack hypervisor stats show". Should I open a bug for that?12:55
openstackgerritAkihiro Motoki proposed openstack/horizon master: Fix boot-from-volume test in project.instances.tests  https://review.opendev.org/67045612:58
openstackgerritAkihiro Motoki proposed openstack/horizon master: Do not always assume image_id in volume_image_metadata  https://review.opendev.org/66859512:58
*** jawad_axd has joined #openstack-horizon13:24
*** duochao has joined #openstack-horizon13:34
*** jawad_axd has quit IRC13:37
*** waverider has quit IRC13:39
*** FlorianFa has quit IRC13:58
amotokicdev3: it is based on non-admin API operation. Doesn't it match nova usage result?14:03
*** duochao_ has joined #openstack-horizon14:26
*** duochao has quit IRC14:26
*** duochao_ is now known as duochao14:26
cdev3@amotoki no it does not, shelved instances are not counted in NOVA ( they are snapshoted and turned off )14:32
*** duochao has quit IRC14:38
*** duochao has joined #openstack-horizon14:38
*** duochao_ has joined #openstack-horizon14:43
*** duochao has quit IRC14:44
*** duochao_ is now known as duochao14:44
*** duochao_ has joined #openstack-horizon14:49
*** duochao has quit IRC14:50
*** openstackgerrit has quit IRC14:51
*** duochao_ has quit IRC14:54
*** duochao has joined #openstack-horizon14:56
*** cmurphy is now known as cmorpheus15:25
*** mlycka has quit IRC15:43
*** duochao has quit IRC15:48
cdev3amotoki: no it does not, shelved instances are not counted in NOVA ( they are snapshoted and turned off )15:51
*** openstackgerrit has joined #openstack-horizon15:51
openstackgerritwuduochao proposed openstack/horizon master: Auto refresh, part 1  https://review.opendev.org/66061215:51
openstackgerritwuduochao proposed openstack/horizon master: Auto refresh, part 2  https://review.opendev.org/66698515:51
amotokicdev3: so, what do you want me to check?15:51
*** duochao has joined #openstack-horizon15:52
*** rdopiera has quit IRC15:54
*** gyee has joined #openstack-horizon15:59
amotokicdev3: In my understanding, the horizon CSV data depends on https://opendev.org/openstack/horizon/src/branch/master/openstack_dashboard/usage/base.py#L17816:00
amotokicdev3: I might be wrong though.16:01
amotokicdev3: if you think it is a problem not to match hypervisor stats and Horizon overview CSV data, I have no idea. Horizon can do nothing other than changing the API horizon uses. I am not sure what you see as a problem..16:02
*** e0ne has quit IRC16:12
*** ksdean has quit IRC16:14
cdev3amotoki: Should it use the same call that CLI uses ? /v2.1/5bd04b76d1db42988612e5f27170a40a/os-hypervisors/statistics. Which turns out to be lib/python2.7/site-packages/nova/db/api.py we find16:15
cdev316:15
cdev3def compute_node_statistics(context):16:15
cdev3    """Get aggregate statistics over all compute nodes.16:15
amotokicdev3: why should the CSV data match hypervisor stats?16:15
amotokicdev3: the API is not exposed to non-admin16:16
amotokicdev3: if you want the usage API of nova to match hypervisor stats, it should be discussed with nova team16:16
cdev3amotoki: the hypervisor stats do not count shelved instances, horizon overview considers shelved instances in the total number of used VMs16:18
amotokicdev3: Are you talking about the CSV data in the *Project* overview?16:18
cdev3amotoki: yes the csv data is reflective of he overview pie charts16:18
amotokicdev3: you are talking about *horizon* *horizon*.... but horizon depends on Nova API with non-admin token.16:19
amotokicdev3: if nova usage response does not consider shelved instances, horizon can do nothing.16:19
amotokicdev3: horizon does not call hypervisor stats. it calls nova usages API.16:19
amotokicdev3: so I wonder what you want actually....16:20
cdev3amotoki: yes, My question is SHOULD the number of VMs in use displayed match "openstack hypervisor stats show" If the CLI does not report shelved instances as using resources, then should the overview count them as in use?16:23
amotokicdev3: which CLI command are you talking about?16:24
cdev3amotoki: "openstack hypervisor stats show"16:24
amotokicdev3: the horizon project overview panel depends on non-admin user token, so how can horizon call "hypervisor stats" API for the project dashaboard? it requires admin token.16:25
amotokicdev3: you say "SHOULD", so it needs to be implementable.16:26
amotokicdev3: it looks like an improvement in nova side. My current suggestion is to propose nova to chage the nova usage API to match hypervisor stats....16:27
cdev3amotoki: As the admin user should the number of running_vms shown in the output of "openstack hypervisor stats show" be the same as"  {% trans "Active Instances:" %},{{ usage.summary.instances }} " found in .../admin/overview/templates/overview/usage.csv16:28
amotokicdev3: that's the reason I asked you whether you talked about Project->Overview CSV.16:29
amotokicdev3: if you talk about Admin->Overview or some panels under "Admin", your point makes sense to me.16:30
-amotoki- has had enough beer Friday night. hopefully someone take over the discussion. he16:30
-amotoki- is still debugging other issues though16:31
cdev3amotoki: ok np I did not know if they represented the same thing16:31
cdev3amotoki: if so and they should be the same then I'll report back and log bug otherwize I'll report that the represent different quntities16:33
amotokicdev3: so do you want "nova usages" and "nova hypervisor-stats" to match?16:33
amotoki"nova usages" is for non-admin users16:34
amotoki"nova hypervisor-stats" is for admin users who are aware of hypervisors.16:34
amotoki(i use command names of 'nova' CLI>16:34
amotokis/>/)/16:34
*** duochao has quit IRC16:35
amotokicdev3: anyway thanks for asking. the basic policy of horizon implementation is to use a token of a logged-in user to display all contents.16:37
*** duochao has joined #openstack-horizon16:37
cdev3I think Active Instances in the CSV should reflect the number of running instances.16:37
cdev3leme check it16:38
amotokicdev3: no worries a lot. I heard the similar thing several times. it looks like we need to clarify what are included in Project->Overview->CSV data.16:40
cdev3amotoki: nova list shows all instances, running or shelved. nova hypervisor-list, shows running_vms with out including shelved instances. So it look like it to me that the overview is showing the same as nova list and not nova hypervisor-list16:41
cdev3amotoki: so if all the instances are shelved or all the instances are running the pie chare will show the same16:42
cdev3becaue it reflects the COUNt of VMs not the usage16:43
-amotoki- does not have enough experience on shelved instances (as openstack clouds he operated disabled shelved instances) :-(16:43
amotokicdev3: so the problem looks like about pie charts for resources quotas and what are reported to users.16:44
amotokiinconsistency should be fixed/improved in information that horizon UI provides.16:45
cdev3amotoki: the pie chart and the CSV file share the same datum. I just cant put a pie chart into bugzilla, only text  :-)16:45
amotokicdev3: bugzilla? horizon uses launchpad16:45
cdev3for my internal bug tracking16:46
cdev3amotoki: ok then I'll report back that the pie chart represents nova list, and log a launchpad bug that it should represent nova hypervisor-stats16:48
amotokicdev3: could you file a bug to provide more detail context on what happens? I am a bit confused with your statement that CSV data does not match what hypervisor-stats returns. I would like to clarify what kind of inconsistency happens from the point of view of regular user (i.e. non-admin user).16:49
cdev3amotoki: yes I will provide full details in a bug, thx!16:50
amotokicdev3: from our discussion I see there are several rooms both in nova and horizon sides.16:50
amotokis/several rooms/several rooms to be improved/16:50
cdev3YA, does this reflect what is in the inventory or what is actually running16:51
amotokiperhaps we can start it from what we can see in non-admin APIs.16:52
*** duochao has quit IRC16:52
cdev3my testing is as admin user, Let me see what I see as nonadmin and I'll put that in the bug16:53
amotokicdev3: thanks. As operators, testing with both non-admin and admin users is important :)17:00
amotokiespecially on how non-admin (i.e. regular) users see.17:01
*** tesseract has quit IRC17:54
*** kplant has joined #openstack-horizon18:55
*** vishalmanchanda has quit IRC19:05
*** BjoernT has joined #openstack-horizon19:51
*** BjoernT has quit IRC19:55
*** kplant has quit IRC20:00
*** btully has joined #openstack-horizon20:27
*** btully has quit IRC20:31
*** priteau has quit IRC20:53
*** whoami-rajat has quit IRC22:06
*** gyee has quit IRC22:39
*** gyee has joined #openstack-horizon22:55
*** tosky has quit IRC23:31

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