Tuesday, 2016-11-01

*** tovin07 has joined #openstack-vitrage01:44
*** yujunz has joined #openstack-vitrage02:07
*** yujunz is now known as yujunz[away]02:28
*** yujunz[away] is now known as yujunz02:42
*** yujunz is now known as yujunz[away]02:42
*** yujunz[away] is now known as yujunz02:42
*** yujunz is now known as yujunz[away]02:43
*** yujunz[away] is now known as yujunz02:43
*** yujunz has quit IRC03:29
*** yujunz has joined #openstack-vitrage03:30
*** yujunz is now known as yujunz[away]03:47
*** yujunz[away] is now known as yujunz03:47
*** yujunz is now known as yujunz[away]03:48
*** yujunz[away] is now known as yujunz04:14
*** yujunz is now known as yujunz[away]04:15
*** yujunz[away] has quit IRC04:18
*** yujunz has joined #openstack-vitrage05:34
*** yujunz is now known as yujunz[away]06:19
*** yujunz[away] is now known as yujunz06:32
*** yujunz is now known as yujunz[away]06:43
*** yujunz[away] is now known as yujunz06:43
*** yujunz is now known as yujunz[away]08:20
*** yujunz[away] is now known as yujunz08:23
*** yujunz is now known as yujunz[away]09:00
*** yujunz[away] is now known as yujunz09:17
*** yujunz is now known as yujunz[away]09:41
*** yujunz[away] is now known as yujunz10:02
*** tovin07 has quit IRC10:06
*** yujunz has quit IRC10:06
*** yujunz has joined #openstack-vitrage10:06
*** yujunz has quit IRC10:14
*** yujunz has joined #openstack-vitrage10:18
*** yujunz has quit IRC10:24
openstackgerritNoam Bloom proposed openstack/vitrage-dashboard: Generify template show view  https://review.openstack.org/39166310:28
openstackgerritMerged openstack/vitrage-dashboard: Generify template show view  https://review.openstack.org/39166310:43
*** yujunz has joined #openstack-vitrage10:46
*** yujunz has quit IRC10:47
*** yujunz has joined #openstack-vitrage11:03
*** yujunz has quit IRC11:06
*** yujunz has joined #openstack-vitrage11:15
*** yujunz has quit IRC11:18
*** yujunz has joined #openstack-vitrage11:31
*** yujunz has quit IRC11:33
*** yujunz has joined #openstack-vitrage11:34
*** yujunz has quit IRC11:34
*** yujunz has joined #openstack-vitrage11:35
*** yujunz has quit IRC11:36
*** yujunz has joined #openstack-vitrage11:42
*** yujunz has quit IRC12:22
*** yujunz has joined #openstack-vitrage13:32
openstackgerritOpenStack Proposal Bot proposed openstack/python-vitrageclient: Updated from global requirements  https://review.openstack.org/38638213:35
openstackgerritOpenStack Proposal Bot proposed openstack/vitrage: Updated from global requirements  https://review.openstack.org/38714713:37
*** elisha_r has joined #openstack-vitrage13:40
*** eyalb has joined #openstack-vitrage13:41
*** yujunz has quit IRC14:01
mrungeeyalb, hey, following up on https://bugzilla.redhat.com/show_bug.cgi?id=134298715:42
openstackbugzilla.redhat.com bug 1342987 in Package Review "Review Request: openstack-vitrage - OpenStack RCA (Root Cause Analysis) Engine" [Unspecified,Assigned] - Assigned to mrunge15:42
mrungeit looks like I missed keystoneauth1 indeed15:42
mrungenevertheless, it builds with my changes in a mock environment15:43
mrungeeyalb, but for tests there is the vitrageclient required15:43
eyalbbut its for packging not for building am I right ?15:50
eyalbwe use the client in tempest15:51
mrungeuhm, I'd love to execute the unit tests during package build15:51
mrungeactually at the end of build process, just to make sure, the build passes the tests15:51
mrungeeyalb, there is no real difference between packaging and building15:52
mrungedoes that make any sense to you?15:53
eyalbwell there is BuildRequires and there is Requires15:53
mrungeright.15:54
mrungebuildrequires is everything required during build15:54
eyalbfor example for generating the the conf file we need some moudles15:54
eyalbso you must have it in buildrequires15:54
mrungethen it's missing in build requires15:54
mrungeyes15:54
eyalbbut we dont need the client for the build process15:55
mrungeI didn't say my spec is complete15:55
eyalbjust when installing the tests packages15:55
mrungedon't you use the client for running tests?15:55
eyalbyes but only if you install the tests rpm15:55
mrungein my spec, tests are being executed as part of the build15:56
eyalbthen you need the dependency for the client15:56
mrungeright15:56
eyalbbut usually in production you will install the client rpm and the vitrage rpm15:56
mrungesince the client is not available as rpm yet, we're currently blocked by it15:56
mrungeright15:56
eyalbI dont think that we are blocked15:57
mrungesee: https://github.com/mrunge/openstack-vitrage/blob/master/openstack-vitrage.spec#L200-L20215:57
eyalbbecause you can build the rpm no ?15:57
mrungewell...15:57
mrungeyes you can15:57
mrungeeyalb, from looking at your spec file, it looks like you packaged it for 0.2.0, right?15:58
eyalbyes15:59
mrungebut now vitrage had a bit progress since then15:59
eyalbwell I can change the version of the dependencies15:59
eyalbit should be align with the master no ?15:59
mrungethat's why I was trying it a bit more and found more additional dependencies15:59
mrungeideally yes16:00
eyalbok16:00
eyalbbut it changes all the time16:00
eyalbsometimes every week16:00
mrungeI would try it with latest release for now16:00
eyalbok thanks16:00
mrungeand then we'll adjust for delorean16:00
mrungeand yes, it's expected to break from time to time, that happens to other projects as well16:00
mrungeevery time when a new dependency was added16:01
eyalbI also added a package review for the dashboard16:01
mrungeoh, I missed that so far16:01
eyalbif you can look at it it will be great16:01
mrungeyes sure16:01
mrungedo you have a bz for it?16:01
eyalbyes16:02
mrungeor is it not that ready yet?16:02
eyalbhttps://bugzilla.redhat.com/show_bug.cgi?id=139060816:02
openstackbugzilla.redhat.com bug 1390608 in Package Review "Review Request: vitrage-dashboard Horizon plugin for vitrage" [Unspecified,New] - Assigned to hguemar16:02
mrungeok, thanks, will have a look asap16:03
eyalbcool thanks16:03
mrungefor reference, the client is under review here: https://bugzilla.redhat.com/show_bug.cgi?id=137978616:04
openstackbugzilla.redhat.com bug 1379786 in Package Review "Review Request: python-vitrageclient - Python bindings to the Vitrage API" [Unspecified,Assigned] - Assigned to mrunge16:04
mrungemaybe I should ping the submitter to look at the changes16:04
eyalbthat would be great16:09
*** elisha_r has quit IRC16:15
*** eyalb has quit IRC16:19
*** pava has quit IRC23:07
*** pava has joined #openstack-vitrage23:07

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