*** boris-42 has joined #openstack-ceilometer | 00:02 | |
*** jruano has quit IRC | 00:08 | |
*** promulo__ has joined #openstack-ceilometer | 00:15 | |
*** ildikov has quit IRC | 00:17 | |
*** shardy_ has joined #openstack-ceilometer | 00:18 | |
*** promulo_ has quit IRC | 00:19 | |
*** shardy has quit IRC | 00:19 | |
*** htruta_ has joined #openstack-ceilometer | 00:22 | |
*** shardy_ has quit IRC | 00:23 | |
*** shardy has joined #openstack-ceilometer | 00:23 | |
*** ildikov has joined #openstack-ceilometer | 00:25 | |
*** _cjones_ has quit IRC | 00:39 | |
*** jruano has joined #openstack-ceilometer | 01:03 | |
*** ddieterl_ has joined #openstack-ceilometer | 01:08 | |
*** ddieterly has quit IRC | 01:08 | |
*** jruano has quit IRC | 01:09 | |
*** jdandrea has quit IRC | 01:11 | |
*** r-mibu has quit IRC | 01:26 | |
*** r-mibu has joined #openstack-ceilometer | 01:26 | |
*** jruano has joined #openstack-ceilometer | 01:28 | |
*** ankita_wagh has quit IRC | 01:37 | |
*** thorst has joined #openstack-ceilometer | 01:38 | |
*** jruano has quit IRC | 01:41 | |
*** changbl has joined #openstack-ceilometer | 01:43 | |
*** jruano has joined #openstack-ceilometer | 02:00 | |
openstackgerrit | Ryota MIBU proposed openstack/aodh: Refactor alarm scenario tests (RuleCombination) https://review.openstack.org/204336 | 02:00 |
---|---|---|
*** ankita_wagh has joined #openstack-ceilometer | 02:03 | |
*** julim has joined #openstack-ceilometer | 02:03 | |
*** ankita_wagh has quit IRC | 02:07 | |
*** ankita_wagh has joined #openstack-ceilometer | 02:08 | |
*** jruano has quit IRC | 02:13 | |
*** jruano has joined #openstack-ceilometer | 02:17 | |
*** jruano has quit IRC | 02:17 | |
*** cchang has joined #openstack-ceilometer | 02:19 | |
*** thorst has quit IRC | 02:20 | |
*** dikonoor has joined #openstack-ceilometer | 02:31 | |
*** deepthi has joined #openstack-ceilometer | 02:32 | |
jasonamyers | I hope devstack gets burned in a fire one day.... | 02:40 |
*** jwcroppe has joined #openstack-ceilometer | 02:42 | |
*** dikonoor has quit IRC | 02:42 | |
*** julim has quit IRC | 02:53 | |
openstackgerrit | liusheng proposed openstack/python-ceilometerclient: Add support for client redirecting to aodh endpoint if available https://review.openstack.org/202938 | 02:57 |
*** cmyster has quit IRC | 03:29 | |
*** exploreshaifali has joined #openstack-ceilometer | 03:29 | |
*** promulo__ has quit IRC | 03:30 | |
*** cmyster has joined #openstack-ceilometer | 03:30 | |
*** cmyster has joined #openstack-ceilometer | 03:30 | |
*** promulo_ has joined #openstack-ceilometer | 03:30 | |
*** thorst has joined #openstack-ceilometer | 03:31 | |
*** ankita_w_ has joined #openstack-ceilometer | 03:34 | |
*** promulo_ has quit IRC | 03:35 | |
*** h00327910_ has quit IRC | 03:35 | |
*** promulo_ has joined #openstack-ceilometer | 03:36 | |
*** exploreshaifali has quit IRC | 03:36 | |
*** ankita_wagh has quit IRC | 03:37 | |
*** promulo_ has quit IRC | 03:40 | |
*** promulo_ has joined #openstack-ceilometer | 03:41 | |
*** thorst has quit IRC | 03:45 | |
*** jruano has joined #openstack-ceilometer | 03:46 | |
*** exploreshaifali has joined #openstack-ceilometer | 03:49 | |
*** cchang has quit IRC | 03:54 | |
*** Kennan2 has joined #openstack-ceilometer | 04:14 | |
*** Kennan has quit IRC | 04:15 | |
openstackgerrit | Ryota MIBU proposed openstack/aodh: Refactor alarm scenario tests (RuleGnocchi) https://review.openstack.org/204370 | 04:16 |
*** jruano has quit IRC | 04:31 | |
*** boris-42 has quit IRC | 04:32 | |
*** yprokule has joined #openstack-ceilometer | 04:44 | |
openstackgerrit | Merged openstack/ceilometer: Sort metric data before grouping and processing https://review.openstack.org/201444 | 04:46 |
*** htruta_ has quit IRC | 04:51 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ceilometer: Updated from global requirements https://review.openstack.org/204387 | 04:54 |
openstackgerrit | Jason Myers proposed openstack/ceilometer: WIP Instance Caching https://review.openstack.org/204388 | 04:54 |
openstackgerrit | Jason Myers proposed openstack/ceilometer: WIP Instance Caching https://review.openstack.org/204388 | 04:56 |
*** exploreshaifali has quit IRC | 04:57 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ceilometerclient: Updated from global requirements https://review.openstack.org/204416 | 04:59 |
*** exploreshaifali has joined #openstack-ceilometer | 05:06 | |
*** cmyster has quit IRC | 05:20 | |
*** ankita_wagh has joined #openstack-ceilometer | 05:38 | |
openstackgerrit | Ryota MIBU proposed openstack/aodh: Refactor alarm scenario tests (RuleThreshold) https://review.openstack.org/204439 | 05:39 |
*** ankita_w_ has quit IRC | 05:40 | |
*** ViswaV has joined #openstack-ceilometer | 05:41 | |
*** ViswaV_ has joined #openstack-ceilometer | 05:53 | |
*** ViswaV has quit IRC | 05:53 | |
*** ViswaV_ has quit IRC | 05:55 | |
*** ViswaV has joined #openstack-ceilometer | 05:56 | |
*** _nadya_ has joined #openstack-ceilometer | 05:56 | |
*** ankita_wagh has quit IRC | 06:04 | |
*** vishwanathj has quit IRC | 06:17 | |
*** exploreshaifali has quit IRC | 06:20 | |
*** ankita_wagh has joined #openstack-ceilometer | 06:21 | |
*** cmyster has joined #openstack-ceilometer | 06:26 | |
*** lsmola has joined #openstack-ceilometer | 06:29 | |
*** boris-42 has joined #openstack-ceilometer | 06:29 | |
*** shardy_ has joined #openstack-ceilometer | 06:31 | |
*** shardy has quit IRC | 06:32 | |
*** shardy_ has quit IRC | 06:36 | |
*** shardy has joined #openstack-ceilometer | 06:37 | |
*** tshtilma has joined #openstack-ceilometer | 06:37 | |
*** ViswaV has quit IRC | 06:38 | |
*** afazekas_ has joined #openstack-ceilometer | 06:39 | |
*** jwcroppe has quit IRC | 06:46 | |
*** jwcroppe has joined #openstack-ceilometer | 06:47 | |
*** jwcroppe has quit IRC | 06:51 | |
*** ViswaV has joined #openstack-ceilometer | 06:51 | |
*** jwcroppe has joined #openstack-ceilometer | 07:00 | |
*** _nadya_ has quit IRC | 07:04 | |
*** ankita_wagh has quit IRC | 07:06 | |
*** tshtilma has quit IRC | 07:17 | |
*** ViswaV has quit IRC | 07:21 | |
*** ildikov has quit IRC | 07:22 | |
-openstackstatus- NOTICE: zuul is undergoing an emergency restart. Jobs will be re-queued but some events may be lost. | 07:22 | |
*** tshtilma has joined #openstack-ceilometer | 07:25 | |
*** afazekas_ has quit IRC | 07:45 | |
*** tshtilma has quit IRC | 07:53 | |
*** afazekas_ has joined #openstack-ceilometer | 07:58 | |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: api: return 410 if only Gnocchi is enabled https://review.openstack.org/202925 | 08:01 |
openstackgerrit | Mehdi Abaakouk (sileht) proposed openstack/ceilometer: api: Redirect request to aodh if available https://review.openstack.org/202926 | 08:01 |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object initial patch https://review.openstack.org/178675 | 08:02 |
*** bdossant has joined #openstack-ceilometer | 08:02 | |
-openstackstatus- NOTICE: zuul jobs are currently stuck while problems with gearman are debugged | 08:13 | |
*** ChanServ changes topic to "zuul jobs are currently stuck while problems with gearman are debugged" | 08:13 | |
*** exploreshaifali has joined #openstack-ceilometer | 08:15 | |
*** safchain has joined #openstack-ceilometer | 08:16 | |
*** ildikov has joined #openstack-ceilometer | 08:29 | |
*** boris-42 has quit IRC | 08:32 | |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object initial patch https://review.openstack.org/178675 | 08:39 |
*** _nadya_ has joined #openstack-ceilometer | 08:41 | |
*** tshtilma has joined #openstack-ceilometer | 08:45 | |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object initial patch https://review.openstack.org/178675 | 08:47 |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object for Event model https://review.openstack.org/181338 | 08:47 |
*** ViswaV has joined #openstack-ceilometer | 08:54 | |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object for Event model https://review.openstack.org/181338 | 09:09 |
*** exploreshaifali has quit IRC | 09:10 | |
*** liusheng has quit IRC | 09:16 | |
*** liusheng has joined #openstack-ceilometer | 09:17 | |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object initial patch https://review.openstack.org/178675 | 09:19 |
openstackgerrit | Kamil Rykowski proposed openstack/ceilometer: Versioned object for Alarm and AlarmChange model https://review.openstack.org/180908 | 09:19 |
*** exploreshaifali has joined #openstack-ceilometer | 09:22 | |
_nadya_ | sileht: hi Mehdi! one quick question about archive policies. afair, in Gnocchi archive policy relates to metric. In the file you have added in "declarative resources for gnocchi" policy relates to resources. Are any changes gonna be done in Gnocchi data model? Or I missed anything? | 09:29 |
sileht | _nadya_, it's definied per resource block but it the archive policy of the metric | 09:30 |
sileht | _nadya_, I have put an example with 'identity' resource | 09:31 |
sileht | _nadya_, you can define two times the 'resource_type: identity' with different archive policy depending of the metrics into the blocks | 09:32 |
_nadya_ | sileht: hmm, ok, I see, but in your file "archive_policy: low" two times | 09:33 |
sileht | _nadya_, because I want to keep the default sane, with the example | 09:34 |
_nadya_ | sileht: I've understand what you meant, thanks! | 09:35 |
sileht | _nadya_, welcome | 09:35 |
liusheng | hi sileht jd__ the client side integration of aodh-api has also been tested OK with heat auto scaling test and rally scenario tests ;-) | 09:42 |
liusheng | https://review.openstack.org/#/c/202938/ | 09:42 |
sileht | liusheng, cool :) | 09:42 |
liusheng | sileht: ;) | 09:42 |
*** ViswaV has quit IRC | 09:44 | |
_nadya_ | liusheng: so it's safe to merge :)? | 09:46 |
liusheng | _nadya_: have tested basic scenario in my environment and integration tests about the functionality ;) | 09:48 |
_nadya_ | liusheng: ook, what does the env look like? Devstack with switched off ceilometer alarming, aodh is installed manually or with devstack plugin (have it worked already)? | 09:50 |
liusheng | _nadya_: I installed aodh manually with the master code and created the service and endpoint as the devstack plugin change proposed | 09:52 |
liusheng | _nadya_: my devstack is often installed failed for other reasons ;( | 09:53 |
_nadya_ | liusheng: ok, I see, thanks! Will test it today, I hope | 09:54 |
liusheng | _nadya_: thanks | 09:54 |
*** _nadya_ has quit IRC | 09:55 | |
*** liusheng has quit IRC | 10:02 | |
jd__ | liusheng: awesome | 10:02 |
*** liusheng has joined #openstack-ceilometer | 10:03 | |
*** tshtilma has quit IRC | 10:12 | |
*** tshtilma has joined #openstack-ceilometer | 10:15 | |
*** cdent has joined #openstack-ceilometer | 10:57 | |
*** promulo__ has joined #openstack-ceilometer | 11:06 | |
*** promulo_ has quit IRC | 11:06 | |
*** liusheng has quit IRC | 11:17 | |
*** liusheng has joined #openstack-ceilometer | 11:18 | |
*** jaosorior has joined #openstack-ceilometer | 11:20 | |
*** julim has joined #openstack-ceilometer | 11:30 | |
*** promulo__ has quit IRC | 11:31 | |
*** promulo_ has joined #openstack-ceilometer | 11:33 | |
*** deepthi has quit IRC | 11:35 | |
*** shardy_ has joined #openstack-ceilometer | 11:36 | |
*** shardy has quit IRC | 11:37 | |
*** promulo_ has quit IRC | 11:37 | |
*** promulo_ has joined #openstack-ceilometer | 11:40 | |
*** liusheng has quit IRC | 11:41 | |
*** shardy_ has quit IRC | 11:41 | |
*** shardy has joined #openstack-ceilometer | 11:42 | |
*** edmondsw has joined #openstack-ceilometer | 11:43 | |
*** shardy_ has joined #openstack-ceilometer | 11:48 | |
*** exploreshaifali_ has joined #openstack-ceilometer | 11:48 | |
*** exploreshaifali has quit IRC | 11:48 | |
*** shardy has quit IRC | 11:48 | |
*** david-lyle has joined #openstack-ceilometer | 11:52 | |
*** shardy_ has quit IRC | 11:53 | |
*** jruano has joined #openstack-ceilometer | 11:53 | |
*** shardy has joined #openstack-ceilometer | 11:54 | |
*** jaypipes has joined #openstack-ceilometer | 11:55 | |
*** jdandrea has joined #openstack-ceilometer | 12:17 | |
*** shardy_ has joined #openstack-ceilometer | 12:21 | |
*** shardy has quit IRC | 12:22 | |
openstackgerrit | Zhai, Edwin proposed openstack/ceilometer: Fix broken IPMI agent https://review.openstack.org/204573 | 12:23 |
*** exploreshaifali_ has quit IRC | 12:24 | |
*** shardy_ has quit IRC | 12:26 | |
*** shardy has joined #openstack-ceilometer | 12:27 | |
*** tongli has joined #openstack-ceilometer | 12:30 | |
*** shardy has quit IRC | 12:31 | |
*** shardy has joined #openstack-ceilometer | 12:32 | |
*** gordc has joined #openstack-ceilometer | 12:37 | |
*** prad has joined #openstack-ceilometer | 12:42 | |
*** renatoarmani has joined #openstack-ceilometer | 12:44 | |
*** thorst has joined #openstack-ceilometer | 12:50 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to event list https://review.openstack.org/196827 | 12:56 |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to resource list https://review.openstack.org/198427 | 12:58 |
*** lexholden has joined #openstack-ceilometer | 12:58 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to meter list https://review.openstack.org/198818 | 12:59 |
*** ddieterl_ has quit IRC | 12:59 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to complex query list https://review.openstack.org/199180 | 13:02 |
*** edmondsw has quit IRC | 13:02 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to complex query list https://review.openstack.org/199180 | 13:04 |
jd__ | cdent: sileht: I'm really starting to wonder if our DetachedInstanceError from http://logs.openstack.org/55/203755/1/check/gate-gnocchi-python27/02346f1/console.html is not a bug in SQLAlchemy | 13:05 |
cdent | they're known to happen | 13:06 |
cdent | should be we dig up zeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeeek | 13:06 |
*** yassine__ has joined #openstack-ceilometer | 13:06 | |
*** _nadya_ has joined #openstack-ceilometer | 13:07 | |
jd__ | cdent: yup :( I also tried to make Gnocchi use http://docs.openstack.org/developer/oslo.db/usage.html#session-handling but failed | 13:08 |
jd__ | I've opened a bug because it's either a doc issue or a bug :) | 13:08 |
jasonamyers | jd__ cdent I've not see a detached instance be a bug in sqlalchemy | 13:08 |
jd__ | none of this was what I expected to do today, obviously, hihi | 13:08 |
jasonamyers | it's normally some odd thing related to session handling | 13:08 |
jd__ | jasonamyers: me neither, but I'm really trying to understand what we do wrong and I can't see | 13:09 |
cdent | ah yes, we have jasonamyers in the house. he's got like sqla books and stuff | 13:09 |
jd__ | jasonamyers: if you have any hindsight… | 13:09 |
jasonamyers | whats the code look like where this is happening? | 13:09 |
jd__ | ah jasonamyers you happen to be a SQLa guru? | 13:09 |
jd__ | jasonamyers: it's pretty straightforward, let me show you | 13:09 |
jasonamyers | nope but I use it a lot | 13:09 |
jd__ | so step #1 we ask the indexer a list of metrics https://github.com/openstack/gnocchi/blob/master/gnocchi/storage/_carbonara.py#L220 | 13:10 |
jd__ | get_metrics(list of uuid) | 13:10 |
jd__ | here's get_metrics() in SQLAlchemy driver https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy.py#L129 | 13:11 |
jd__ | it just do a query with a joinedload() of the attribue such as archive_policy | 13:11 |
jd__ | and returns the list retrieved by the SELECT … JOIN … | 13:11 |
jd__ | we iterate on that and boum! | 13:11 |
jd__ | gnagnagna DetachedInstanceError | 13:11 |
jd__ | but we asked to joinedload('archive_policy') | 13:12 |
jasonamyers | okay so archive_policy is lazy loaded? | 13:12 |
jd__ | the model for Metric is at https://github.com/openstack/gnocchi/blob/master/gnocchi/indexer/sqlalchemy_base.py#L135 | 13:12 |
jd__ | jasonamyers: by default it is lazy loaded but in get_metrics() we ask for a joinedload() | 13:12 |
jd__ | so we want it to be there and loaded | 13:12 |
jd__ | jasonamyers: and the thing is that well the code works 99% of the time, it fails randomly | 13:12 |
jd__ | so I smell weak reference or something like that :( | 13:13 |
jasonamyers | hmm or some how the sessions is getting closed | 13:13 |
jasonamyers | what version of sqlalchemy? | 13:14 |
jasonamyers | hmm it's unversioned in requirements | 13:15 |
jasonamyers | so that's 1.0.7 | 13:15 |
gordc | you can just use session without any context? | 13:17 |
jasonamyers | yeah | 13:19 |
gordc | jasonamyers: how do you guarantee integrity/consistency/<whatever word i'm looking for> if something changes? | 13:20 |
jasonamyers | I don't think he's trying to do that here really | 13:21 |
jasonamyers | he's just querying up a list for aggregation | 13:22 |
jd__ | jasonamyers: well I guess the session is closed once get_metrics() returns | 13:22 |
jd__ | but that should not block us from using the object since it has been loaded by joinedload() | 13:22 |
jasonamyers | it could | 13:22 |
jd__ | *grmbl* | 13:23 |
jd__ | :( | 13:23 |
jasonamyers | because the related object could be lost | 13:23 |
jasonamyers | when the sessions drops | 13:23 |
* jd__ cries | 13:23 | |
jd__ | that's what I was kinda of afraid | 13:23 |
jasonamyers | and since you don't control it manually to close it | 13:23 |
jasonamyers | it doesn't know | 13:23 |
jasonamyers | so you can .expunge_all if you don't need to update the metrics | 13:23 |
jd__ | so you basically can't return SQLAlchemy objects outside sessions? | 13:23 |
jasonamyers | you can if you expunge them | 13:24 |
jd__ | what's expunge? | 13:24 |
* jd__ RTFM | 13:24 | |
jasonamyers | the issue is the relationships while joined still need to be done in a way that works indenpendant of the session | 13:24 |
jd__ | aaahhh | 13:24 |
jd__ | that sounds like a good idea, let me try that | 13:24 |
jasonamyers | word of caution here expunge... can lead to you doing bad things if you are trying to update things | 13:24 |
jd__ | yeah that's not the case here, it's just read-only | 13:25 |
jd__ | we never update things outside the driver itself so that's safe | 13:25 |
jd__ | expunge really looks like what I need | 13:25 |
* jd__ is now EXCITED | 13:26 | |
* gordc wants to suggest just avoiding orm. | 13:26 | |
* jd__ beats gordc | 13:26 | |
gordc | *shrugs* virtual beatings don't hurt. | 13:26 |
jd__ | gordc: my initial plan was to write only a psycopg2 driver | 13:26 |
jasonamyers | I prefer the SEL over ORM in SQLAlchemy but the ORM is pretty hot | 13:26 |
*** david-lyle has quit IRC | 13:27 | |
gordc | jd__: did you realise it was too fast? :) | 13:27 |
jasonamyers | he realized he shouldn't be writing raw sql | 13:28 |
jasonamyers | because handling connection pools is the worst | 13:28 |
jd__ | gordc: nah someone convinced me it was a good idea technically but a bad one politically and I noded | 13:28 |
jasonamyers | or done terribly in most cases | 13:28 |
gordc | jd__: sounds about right. politics always makes things better. | 13:29 |
gordc | what's SEL? all i know is orm and core | 13:29 |
jasonamyers | SEL is the language used in Core | 13:29 |
jasonamyers | SQL Expression Language | 13:30 |
jasonamyers | it's served | 13:30 |
gordc | oh. cool cool. yeah. it's much better. | 13:30 |
jasonamyers | ww | 13:30 |
gordc | you got served? | 13:30 |
jasonamyers | wrong window sorry | 13:30 |
gordc | lol | 13:30 |
jasonamyers | yeah in slack for work someone giphy'ed me | 13:31 |
jasonamyers | because you know I'm 35 almost 36 and he's 21 | 13:31 |
jasonamyers | so he only communicates in gif or emoji | 13:32 |
gordc | i had to google 'giphy' | 13:32 |
jasonamyers | I just leave up urban dictionary at this point and paste his comments in it | 13:33 |
gordc | urbandictionary is the best. | 13:34 |
gordc | cool CI is down everywhere. free day for everyone. | 13:35 |
jasonamyers | whelp cya later | 13:37 |
jd__ | sqlalchemy.orm.exc.UnmappedInstanceError: Class '__builtin__.NoneType' is not mapped | 13:37 |
jd__ | how does it dare talking to me like that | 13:38 |
jd__ | COMPUTER DO WHAT I SAY AND MAKE MAGIC | 13:38 |
jd__ | I map None to a table if I want :p | 13:39 |
gordc | some people just want to see the world burn. | 13:39 |
*** jwcroppe has quit IRC | 13:40 | |
jd__ | while the rest got lighters | 13:40 |
*** jaypipes has quit IRC | 13:40 | |
jasonamyers | haha | 13:41 |
*** cchang has joined #openstack-ceilometer | 13:41 | |
*** ddieterly has joined #openstack-ceilometer | 13:42 | |
*** liusheng_ has joined #openstack-ceilometer | 13:45 | |
gordc | ildikov: remove line83? | 13:45 |
*** ddieterly has quit IRC | 13:47 | |
*** edmondsw has joined #openstack-ceilometer | 13:48 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to event list https://review.openstack.org/196827 | 13:48 |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to resource list https://review.openstack.org/198427 | 13:48 |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to meter list https://review.openstack.org/198818 | 13:48 |
openstackgerrit | gordon chung proposed openstack/ceilometer: add mandatory limit value to complex query list https://review.openstack.org/199180 | 13:48 |
ildikov | gordc: yeap, I recognized that too | 13:50 |
ildikov | gordc: I just still don't get whether we want to allow query without limit or not :) | 13:51 |
ildikov | gordc: but I'm a bit slower today than as usual, so probably I miss something still | 13:52 |
gordc | ildikov: it's retricted via api/client | 13:52 |
gordc | ildikov: but if you have been hacking ceilometer code all along, or if you are testing. you can bypass it | 13:53 |
gordc | basically since the method is public, i didn't want to break whoever was using it already (i'm assuming no one but who knows) | 13:53 |
ildikov | gordc: but if someone is hacking in the code they can re-add it if needed | 13:53 |
gordc | re-add what. | 13:53 |
ildikov | I meant if we would make limit mandatory there | 13:54 |
ildikov | it's funny that you are the one always complaining about backward compat, etc. and now you don't modify the code to not break some hacking, if there is any out there :) | 13:55 |
gordc | :( i'm disappointed in myself. | 13:56 |
gordc | so you want to just drop optional param? the thing is we still need to add None check in case None gets passed in | 13:56 |
ildikov | bah, I forgot about the case when it's passed in, I guess it's not that part where we would need to deal with having one more if statement in the code or not | 14:00 |
ildikov | although I still think it would make the code more readable to drop the optional | 14:00 |
gordc | ildikov: can i do this in a followup? i'm pretty sure it's the existing behaviour. | 14:04 |
*** liusheng_ has quit IRC | 14:04 | |
gordc | i did a git blame on original code. you don't want me to tell you who checked it in. :) | 14:04 |
*** jwcroppe has joined #openstack-ceilometer | 14:05 | |
*** jwcroppe has quit IRC | 14:05 | |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: expunge objects before returning them https://review.openstack.org/204627 | 14:05 |
ildikov | gordc: you mean follow up having limit as optional or mandatory? | 14:06 |
*** jwcroppe has joined #openstack-ceilometer | 14:06 | |
gordc | having limit as mandatory param. | 14:06 |
ildikov | ok, we can follow up on that if you don't want to deal with the broken tests now :) | 14:06 |
gordc | i don't. :) | 14:07 |
ildikov | and well, the pymongo_base file is my fault, I haven't touched the functions though :) | 14:07 |
ildikov | I just removed some code duplication | 14:07 |
*** jaypipes has joined #openstack-ceilometer | 14:07 | |
ildikov | so don't blame me, even if git does ;) | 14:08 |
gordc | :) never refactor. it will end up blaming you. | 14:08 |
gordc | brb. meeting | 14:09 |
jasonamyers | ildikov: +1 | 14:09 |
ildikov | :) | 14:09 |
ildikov | jasonamyers: on removing duplication you mean? | 14:09 |
jasonamyers | yeah and not getting blamed | 14:09 |
-openstackstatus- NOTICE: CI is currently recovering from an outage overnight. It is safe to recheck results with NOT_REGISTERED errors. It may take some time for zuul to work through the backlog. | 14:13 | |
*** ChanServ changes topic to "CI is currently recovering from an outage overnight. It is safe to recheck results with NOT_REGISTERED errors. It may take some time for zuul to work through the backlog." | 14:14 | |
ildikov | jasonamyers: yeap, agreed | 14:15 |
*** boris-42 has joined #openstack-ceilometer | 14:16 | |
ildikov | although I don't see it as an issue here, so we are a nice project from this perspective I would say | 14:16 |
ildikov | gordc: BTW, do I see it correctly that for events the limit param is brand new? | 14:21 |
*** vishwanathj has joined #openstack-ceilometer | 14:24 | |
*** shardy_ has joined #openstack-ceilometer | 14:25 | |
*** shardy has quit IRC | 14:27 | |
*** rbak has joined #openstack-ceilometer | 14:27 | |
*** shardy_ has quit IRC | 14:30 | |
*** shardy has joined #openstack-ceilometer | 14:31 | |
*** ChanServ changes topic to "OpenStack Telemetry (Ceilometer) | https://wiki.openstack.org/wiki/Ceilometer" | 14:32 | |
-openstackstatus- NOTICE: CI is slowly recovering, please be patient while the backlog is worked through. | 14:32 | |
*** vishwana_ has joined #openstack-ceilometer | 14:36 | |
*** vishwanathj has quit IRC | 14:39 | |
*** lexholden has quit IRC | 14:44 | |
jasonamyers | jd__: cdent I would use expunge all as well | 14:57 |
jasonamyers | I'm not certain on the cascade as I think the joined load will get them all of the expunge all is used | 14:57 |
jd__ | jasonamyers: yeah that makes sense | 14:58 |
jd__ | ok I'll switch to expunge_all() everywhere | 14:58 |
*** david-lyle has joined #openstack-ceilometer | 15:09 | |
openstackgerrit | Merged openstack/aodh: Move alarm.service to service https://review.openstack.org/200593 | 15:10 |
openstackgerrit | Merged openstack/aodh: Move aodh.alarm.rpc to aodh.rpc https://review.openstack.org/200598 | 15:10 |
*** tshtilma has quit IRC | 15:13 | |
*** nijaba has quit IRC | 15:16 | |
*** nijaba has joined #openstack-ceilometer | 15:16 | |
*** nijaba has quit IRC | 15:16 | |
*** nijaba has joined #openstack-ceilometer | 15:16 | |
jd__ | jasonamyers: cdent: funny enough, I've changed get_metrics() to expunge_all() and now I can reproduce the statsd error 100% of the time :) | 15:17 |
cdent | huh | 15:17 |
jasonamyers | WOot! | 15:17 |
jasonamyers | consistancy | 15:17 |
cdent | that's gotta mean....something | 15:17 |
jd__ | so that maybe because expunge_all() does not cascade? | 15:17 |
jasonamyers | so expunge all must not satisfy the cascade | 15:17 |
cdent | but it is _all? | 15:17 |
jasonamyers | jd__: +1 | 15:17 |
jasonamyers | yeah, but that might not mean the relationships are maintained in expunge_all | 15:18 |
jasonamyers | I had assumed they would but obviously not | 15:18 |
jd__ | nop, restoring only this expunge_all() to expunge() does not change it… weird | 15:18 |
jd__ | so maybe there's another expunge vs expunge_all elsewhere playing? man this is a maze | 15:18 |
jasonamyers | relationships are hard | 15:19 |
jd__ | that's what she said | 15:20 |
jd__ | oh man I think I got it | 15:23 |
jd__ | facepalm | 15:23 |
cdent | zomg, is that the first twss I've ever seen in an openstack channel | 15:23 |
* cdent notes the date | 15:23 | |
jd__ | this is not related to expunge though that definitely helped me to understand | 15:23 |
* cdent waits with baited breath | 15:23 | |
jd__ | cdent: that's because you're still new | 15:23 |
jd__ | so the issue is only happening in test_statsd because well, we actually mock get_metrics()… | 15:24 |
cdent | Isn't that awesome? Been here more than a year, still new. OpenStack is one weird and messed up lace. | 15:24 |
cdent | ugh | 15:24 |
jd__ | and we mock it with a return [metric] where metric is a metric that has NO archive policy eagerly loaded | 15:24 |
jd__ | so well sometimes it fails because the session has gone obviously | 15:24 |
cdent | wow | 15:24 |
cdent | nice digging | 15:25 |
jd__ | newbie mistake, but I don't need to git blame to know I wrote that :> | 15:25 |
jd__ | now I feel smarter | 15:25 |
cdent | mocks are bad, mmmkay? | 15:26 |
jd__ | yeah :( | 15:26 |
jd__ | so expunge_all() is a good patch because it makes the issue pops up 100% time so I'll keep that patch | 15:26 |
jd__ | I just need to write a fix before it :) | 15:27 |
jasonamyers | jd__: awesome nicely done | 15:28 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: sqlalchemy: expunge objects before returning them https://review.openstack.org/204627 | 15:31 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: statsd: stop mocking indexer.get_metrics in tests https://review.openstack.org/204666 | 15:31 |
jd__ | yeah thanks for the hint jasonamyers! | 15:31 |
* jd__ high fives everyone | 15:31 | |
jasonamyers | happy it helped sorry it wasn't the answer | 15:32 |
openstackgerrit | Julien Danjou proposed openstack/gnocchi: Switch to facade https://review.openstack.org/204669 | 15:32 |
*** _nadya_ has quit IRC | 15:41 | |
*** _nadya_ has joined #openstack-ceilometer | 15:42 | |
*** chmouel has quit IRC | 15:44 | |
*** aignatov has quit IRC | 15:45 | |
cdent | I've just realized that my hardware keeps my office quite a bit warmer than the rest of the house | 15:45 |
*** _nadya_ has quit IRC | 15:46 | |
*** bdossant has quit IRC | 15:50 | |
*** aignatov has joined #openstack-ceilometer | 15:50 | |
*** chmouel has joined #openstack-ceilometer | 15:51 | |
*** promulo__ has joined #openstack-ceilometer | 15:56 | |
*** promulo_ has quit IRC | 15:56 | |
prad | is review.openstack really slow today? or is it just my network | 15:57 |
cdent | prad note being super slow for me | 16:04 |
cdent | I usually find it is slower than i want it to be pretty much all the time | 16:04 |
prad | it just times out for me .. Code Review - Error Server Unavailable | 16:05 |
prad | k i'll wait it out | 16:06 |
gordc | ildikov: all limit params are new (except for get_samples) | 16:11 |
ildikov | gordc: I just asked because in case of new ones we could have them as mandatory from the first moment | 16:12 |
gordc | ildikov: i could... or i could keep the method definition the same and leave it optional | 16:14 |
gordc | i think really the restriction should be placed at api. we don't really care what craziness you do with your hacks. | 16:15 |
ildikov | gordc: it seems odd to me that we would like to enforce the limit, but keep the param as optional | 16:15 |
ildikov | you mean if someone for his/her own risk would like to bypass the API and don't use limit, then it can be done this way easily? | 16:16 |
gordc | correct. | 16:16 |
*** safchain has quit IRC | 16:18 | |
gordc | ie. if you want to archive or do i data pull, i don't know if you need/want to do that directly from api... i don't know. maybe you do. | 16:18 |
*** changbl has quit IRC | 16:18 | |
openstackgerrit | Julien Danjou proposed openstack/aodh: notifier: move from alarm to top-level https://review.openstack.org/204107 | 16:23 |
*** ankita_wagh has joined #openstack-ceilometer | 16:26 | |
*** changbl has joined #openstack-ceilometer | 16:32 | |
gordc | edmondsw: regarding dikonoor's policy patch, is there an issue if we maintain admin requirement as default policy rule? | 16:33 |
gordc | asking because i have tempest test for nova events (non-audit) which expects admin. i don't want to merge it if i need to revert it later. https://review.openstack.org/#/c/155287/ | 16:34 |
ildikov | gordc: I wouldn't rely on internal functions for archives, etc. | 16:37 |
ildikov | I mean the API is what is user facing, therefore what is versioned and where backward compatibility etc. is really considered | 16:38 |
ildikov | all the other things can change any time | 16:38 |
*** _cjones_ has joined #openstack-ceilometer | 16:40 | |
*** ddaskal has joined #openstack-ceilometer | 16:40 | |
gordc | well we build the model at backend... so api doesn't actually guarantee anything more. | 16:44 |
gordc | ildikov: but as mentioned, having it optional or mandatory as a param doesn't change anything. except all the tests need to change. | 16:45 |
ildikov | sure, I just wanted to understand the motivation behind | 16:50 |
ildikov | sorry for stealing your time with this pretty minor topic | 16:50 |
gordc | ildikov: my motivation was i copied existing behaviour. :) | 16:54 |
ildikov | gordc: :) | 16:57 |
*** ankita_wagh has quit IRC | 16:57 | |
*** renatoar_ has joined #openstack-ceilometer | 17:01 | |
*** Ala has quit IRC | 17:01 | |
*** renatoar_ has quit IRC | 17:02 | |
ildikov | gordc: in the events patch for the sqlalchemy one, isn't a check needed there too whether the limit is none or not? | 17:02 |
ildikov | gordc: last question, I swear :) | 17:02 |
*** renatoarmani has quit IRC | 17:02 | |
gordc | ildikov: i'm pretty sure limit(None) is same as no limit. let me verify | 17:06 |
*** lexholden has joined #openstack-ceilometer | 17:06 | |
ildikov | gordc: if you have an env that's cool, I've just checked the other functions for that driver | 17:06 |
ildikov | gordc: but we can clean up those checks if it doesn't matter anyway | 17:07 |
*** _nadya_ has joined #openstack-ceilometer | 17:09 | |
cdent | gordc: when you get done with thinking about that and have a moment, ping me to talk grenade and devstack | 17:09 |
cdent | (but not yet, going for snack) | 17:09 |
*** thorst has quit IRC | 17:11 | |
*** shardy_ has joined #openstack-ceilometer | 17:11 | |
*** yprokule has quit IRC | 17:11 | |
gordc | ildikov: yeah, limit(None) is just ignored http://stackoverflow.com/questions/12836194/how-do-i-remove-a-limit-that-is-already-applied-to-a-query-object | 17:12 |
*** shardy has quit IRC | 17:13 | |
gordc | cdent: i can chat now, office is in the suburbs so i don't think you want to wait for me to get lunch. | 17:14 |
ildikov | gordc: cool, tnx, the world would stop without stackoverflow :) | 17:14 |
gordc | ildikov: sometimes i think about contributing... then i realise i don't care enough to create an account. | 17:15 |
ildikov | gordc: LOL :) | 17:15 |
*** shardy_ has quit IRC | 17:16 | |
*** shardy has joined #openstack-ceilometer | 17:17 | |
*** ankita_wagh has joined #openstack-ceilometer | 17:17 | |
cdent | gordc: so you know I'm sort of trying to make grenade and devstack plugins work throughout our stuff | 17:23 |
cdent | there are some hangups with trying to do _both_ with ceilometer because of the fact that ceilometer is "integrated" | 17:23 |
cdent | so one of the strategies that has been put forward is to get the grenade stuff working and then later get the devstack stuff working | 17:24 |
gordc | cdent: let me know when you done typing | 17:25 |
cdent | my concern with making any headway on this stuff is that once it starts to work there will be cascade of stuff to do: gate jobs to update, tempest checks to make sure things are happy, etc etc | 17:26 |
*** afazekas_ has quit IRC | 17:26 | |
cdent | the questions are a) have we got time for that b) something else but I've forgotten because I'm tired | 17:26 |
cdent | EOT | 17:26 |
gordc | this is just related to existing ceilometer repo? nothing to do with split right? | 17:27 |
cdent | there will be follow on stuff do related to the splitting | 17:27 |
cdent | (for example in tempest we may need to make aodh run or something like that) | 17:28 |
gordc | can we leave grenade/devstack as is and continue on with aodh, gnocchi stuff? | 17:28 |
gordc | making the appropriate changes required. | 17:29 |
cdent | I think so yes. The original plan was that if we could get all pluginified then the turnaround time on necessary changes would be lessened (since we would be doing the reviews) | 17:29 |
cdent | but given that I'm not able to make headway on getting reviews on the pluginification....shrug | 17:30 |
cdent | we've already hit situations where someone wants to make changes to devstack or grenade and needed to be redirected from the plugins to the non-plugin stuff with their changes because that stuff is not yet live | 17:31 |
cdent | I think I can still tick over on the plugin stuff in the background but given the pace we are keeping on the other stuff I think trying to sync up would cause delays. | 17:31 |
gordc | ah i see... yeah maybe it's something to defer and do it post freeze? | 17:32 |
gordc | how does grenade plugin work? is it just telling it look here instead of is there crazy dependencies. | 17:32 |
cdent | well | 17:32 |
cdent | the short answer is yes | 17:32 |
*** ildikov has quit IRC | 17:33 | |
gordc | i know tempest stuff has some crazy dependencies that you have to disable, do stuff, reenable. | 17:33 |
cdent | what I'd like to be able to do, I think, is see if I can get this merged: https://review.openstack.org/#/c/196441/9 | 17:34 |
gordc | seems like it won't improve ease of review for liberty since by the time we get it in, it'll be a short run to freeze | 17:35 |
cdent | that means that we'll have the grenade and devstack plugins in tree to play with experimentally | 17:35 |
cdent | but not for realsies | 17:35 |
cdent | the cost of that, though, is that we'll need to be on our guard for people trying to commit to the wrong place | 17:35 |
gordc | not for real means, it's in-tree but not being used? | 17:35 |
cdent | not being used in the gate | 17:36 |
cdent | there's currently an experimental grenade-plugin job: gate-grenade-dsvm-ceilometer: | 17:36 |
gordc | to enable it in gate is where all the dependencies come into play? | 17:36 |
cdent | to enable the plugins in gate we have to disable the non-plugins in lots of different places, that's the part that I don't think we have the cycles to fight with at the moment | 17:37 |
gordc | sure. i'd agree with you, it's not that important. | 17:39 |
gordc | so for the new files like meter.yaml and gnocchi_resources.yaml do we need to add them to both plugin and existing grenade? | 17:40 |
gordc | i don't see how we can do one and not the other | 17:40 |
*** thorst has joined #openstack-ceilometer | 17:40 | |
*** changbl has quit IRC | 17:41 | |
cdent | I think for the sake of sanity we just add them to existing devstack:lib/ceilometer | 17:41 |
cdent | and then do a sync up later | 17:42 |
cdent | we can, if people are feeling motivated do both, but I'm not sure if it is worth it? | 17:42 |
cdent | Obviously it is easier if we do both in sync | 17:42 |
gordc | cdent: makes sense to me. i think we just have the two files right now... or maybe just the declarative meters file | 17:43 |
gordc | i don't think we even need gnocchi resources yaml for grenade since it's new functionality. | 17:44 |
gordc | i'll let prad decide how much of a pain it was to sync both. | 17:44 |
gordc | let's try to keep in sync for now. since i don't really see grenade being changed much. | 17:45 |
gordc | the devstack plugin might be another story. | 17:45 |
cdent | most of what grenade does is call into devstack | 17:45 |
cdent | except for in the case of new files | 17:45 |
cdent | prad already had to make a chance to grenade for the declarative meters | 17:45 |
gordc | yeah, the only changes we've made to grenade were to add new files. which is pretty rare. | 17:46 |
gordc | i'm thinking to try to maintain both grenades for now and abandon if it becomes a pain... like we did with alarming... which became a pain after 1 patch. | 17:47 |
cdent | :) | 17:48 |
gordc | i'm thinking/hoping it's just going to be one patch. | 17:48 |
openstackgerrit | Chris Dent proposed openstack/ceilometer: [WIP] The start of a grenade plugin for ceilometer https://review.openstack.org/196441 | 17:49 |
cdent | (that's a rebase with a depends-on to see if I can get it to behave properly) | 17:49 |
cdent | dinner time | 17:49 |
gordc | cool lunch time. | 17:49 |
openstackgerrit | Merged openstack/gnocchi: Don't allow duplicate timestamps in carbonara series https://review.openstack.org/204177 | 17:50 |
*** changbl has joined #openstack-ceilometer | 17:52 | |
*** cchang has quit IRC | 17:55 | |
*** rjaiswal has joined #openstack-ceilometer | 18:00 | |
*** thorst has quit IRC | 18:02 | |
*** Kennan has joined #openstack-ceilometer | 18:04 | |
*** Kennan2 has quit IRC | 18:04 | |
*** jwcroppe_ has joined #openstack-ceilometer | 18:06 | |
*** changbl has quit IRC | 18:07 | |
*** pauldix has joined #openstack-ceilometer | 18:08 | |
*** jwcroppe_ has quit IRC | 18:10 | |
*** jwcroppe_ has joined #openstack-ceilometer | 18:10 | |
*** jwcroppe has quit IRC | 18:10 | |
*** changbl has joined #openstack-ceilometer | 18:10 | |
*** _nadya_ has quit IRC | 18:19 | |
*** _nadya_ has joined #openstack-ceilometer | 18:19 | |
*** changbl has quit IRC | 18:20 | |
*** cchang has joined #openstack-ceilometer | 18:21 | |
*** _nadya_ has quit IRC | 18:24 | |
*** thorst has joined #openstack-ceilometer | 18:24 | |
*** jwcroppe_ has quit IRC | 18:30 | |
*** changbl has joined #openstack-ceilometer | 18:33 | |
*** jwcroppe has joined #openstack-ceilometer | 18:34 | |
*** jwcroppe has quit IRC | 18:34 | |
*** jwcroppe has joined #openstack-ceilometer | 18:36 | |
openstackgerrit | Merged openstack/ceilometer: Pollsters now send notifications without doing transforms https://review.openstack.org/198865 | 18:43 |
cdent | gordc: you should go to the monasca mid cycle | 19:06 |
cdent | build bridges and stuff | 19:06 |
*** shardy_ has joined #openstack-ceilometer | 19:10 | |
prad | cdent, where is it hosted? | 19:10 |
cdent | Colorado | 19:10 |
cdent | there's been a posting to os-dev in the last little while | 19:10 |
prad | ah ok | 19:11 |
*** exploreshaifali has joined #openstack-ceilometer | 19:11 | |
*** shardy has quit IRC | 19:11 | |
cdent | They've got agenda items for a thing called ceilosca | 19:12 |
cdent | It would be awesome if we could all finally start working together more | 19:12 |
cdent | all this split out stuff we have planned out to make it more fun and easy to interoperate | 19:12 |
*** changbl has quit IRC | 19:12 | |
cdent | gordc ^ | 19:12 |
prad | there is a Ceilosca? lol | 19:12 |
*** changbl has joined #openstack-ceilometer | 19:12 | |
prad | cdent, ah they finally have a all-python monasca | 19:13 |
cdent | "all python" | 19:13 |
prad | :) | 19:14 |
cdent | I assume the quotes much be meaningful | 19:14 |
gordc | cdent: i saw it | 19:14 |
cdent | I'd go if someone was willing to foot the bill | 19:14 |
edmondsw | gordc, I think you're asking if it's ok to make the default policy for GETting events restricted to admins. Yes, I think so. | 19:15 |
gordc | edmondsw: yep, that was what i was asking. | 19:15 |
edmondsw | with the caveat that an admin should only be able to view events for their project, but I wouldn't expect that to be an issue for your tempest test | 19:15 |
gordc | i will reenable my patch then. | 19:15 |
*** shardy_ has quit IRC | 19:15 | |
*** shardy has joined #openstack-ceilometer | 19:16 | |
gordc | jasonamyers: ^ since you're the cisco connection, any idea if there's a reason for us to be part of it? | 19:18 |
*** jdandrea has left #openstack-ceilometer | 19:18 | |
*** cchang has quit IRC | 19:19 | |
*** cchang has joined #openstack-ceilometer | 19:19 | |
jasonamyers | umm can I decide to not comment? | 19:19 |
jasonamyers | ther eis not presently a ceilosca or an all-python monasca | 19:20 |
gordc | jasonamyers: no worries. i understand big company culture. :) | 19:21 |
gordc | i haven't heard anything about it specifically so i assume their 'integration' doesn't need us. | 19:22 |
jasonamyers | I'm just not part of that group so I have no idea what is going on except at a status update level | 19:22 |
gordc | jasonamyers: understood. same when i was at ibm. foot in everything. | 19:24 |
*** tshtilma has joined #openstack-ceilometer | 19:26 | |
cdent | totes time to start up, i hear openstack is startup friendly, read it in the rags | 19:27 |
*** cdent has left #openstack-ceilometer | 19:27 | |
*** cdent has joined #openstack-ceilometer | 19:27 | |
cdent | meh, ww | 19:27 |
gordc | lol | 19:28 |
jasonamyers | cdent: anything but that | 19:33 |
jasonamyers | it's time to move on to mesos | 19:34 |
cdent | no, that's already so last week | 19:34 |
cdent | I reckon it is time to unleash p2p onit™ based computing services | 19:35 |
cdent | http://im-onit.com | 19:35 |
gordc | cdent: is that your design? | 19:36 |
cdent | yeah, it's _very_ old | 19:36 |
gordc | are you hosting that website? | 19:37 |
cdent | but given advancement in processing technology I think it might finally be able to reach its true purpose | 19:37 |
cdent | yes | 19:37 |
gordc | what happens if i DDOS you? | 19:37 |
cdent | I won't get any email and since the only reason I ever do anything is because of email I won't do anything | 19:38 |
cdent | you might see that as a blessing or curse depending on how your feel about my contributions to the team | 19:38 |
gordc | do you earn a liveable wage from tshirts? i am thinking of a career change | 19:38 |
cdent | I don't host the tshirts | 19:38 |
cdent | just that front page | 19:38 |
cdent | and I think I've sold 4 | 19:38 |
*** tshtilma has quit IRC | 19:39 | |
gordc | i thought you get a cut?... or is it like a youtube view, you need millions. | 19:39 |
cdent | you get a cut after a certain threshold | 19:39 |
gordc | i'm guessing it's not 4. | 19:40 |
cdent | apparently not | 19:40 |
gordc | you should make an openstack one. an unlicensed openstack shirt. | 19:40 |
cdent | I suspect your contacts in china can help me with the onit manufacturing process | 19:40 |
gordc | 0p3nst@ck | 19:40 |
gordc | cdent: easily... but it'll get outsourced to another asian country. | 19:41 |
gordc | but that's not your concern... not liable. | 19:41 |
cdent | in fact I suspect your employers might be interested in helping me build the thing, It's not too dissimilar from devices I've had from them in the past | 19:42 |
gordc | you have huawei router? | 19:42 |
cdent | I used to have a dongle | 19:43 |
gordc | i've never seen anything huawei. hasn't reach Canada yet... or not consumer devices. | 19:45 |
cdent | they are fairly present in the UK | 19:46 |
*** pauldix has quit IRC | 19:46 | |
cdent | I guess in the US they are not present at all because ZOMG COMMIES | 19:46 |
gordc | we take that seriously here... a restaurant host told me i looked like Marx with my beard. he's no longer with us. | 19:47 |
gordc | i feel like our conversations don't translate well to logs. | 19:48 |
cdent | c'est la vie | 19:48 |
gordc | ouais | 19:49 |
cdent | bon | 19:49 |
gordc | pamplemousse. | 19:49 |
cdent | c'est vrai | 19:50 |
gordc | hahah | 19:50 |
cdent | that's it from me | 19:51 |
cdent | maybe the gate will have caught up by tomorrow | 19:52 |
cdent | gnight | 19:52 |
*** cdent has quit IRC | 19:52 | |
openstackgerrit | gordon chung proposed openstack/ceilometer: Revert "Revert "remove instance:<flavor> meter"" https://review.openstack.org/192503 | 19:57 |
*** ankita_w_ has joined #openstack-ceilometer | 19:59 | |
*** ankita_wagh has quit IRC | 19:59 | |
*** ankita_wagh has joined #openstack-ceilometer | 20:00 | |
*** ankita_w_ has quit IRC | 20:00 | |
*** pauldix has joined #openstack-ceilometer | 20:22 | |
*** pauldix has quit IRC | 20:26 | |
*** pauldix has joined #openstack-ceilometer | 20:28 | |
*** Putta has joined #openstack-ceilometer | 20:33 | |
openstackgerrit | Jason Myers proposed openstack/ceilometer: WIP Instance Caching https://review.openstack.org/204388 | 20:33 |
Putta | @gordc: I am trying to use notification-agent in HA mode, was wondering what changes do i need to make to work in HA mode apart from installing notification agent on multiple nodes (3 nodes) and setting up backend_url for coordination | 20:37 |
gordc | Putta: you'll also need to set workload_partitioning=True in [notification] section | 20:45 |
Putta | gordc: Do i need to set notification_workers = [>1] ?? | 20:46 |
gordc | no. that will just allow you to create multiple workers managed by a single service... by default each service has one worker. | 20:47 |
*** ankita_w_ has joined #openstack-ceilometer | 20:47 | |
gordc | if you want more than one agent on a node, you can either enable more workers or you can just launch another instance of service. they're pretty much equivalent, just difference is how you want to manage them. | 20:49 |
Putta | gordc, got it. thank you for the detailed info. One last question, how do i test it to make sure my setup is in fact working in HA mode for the notification agent | 20:51 |
*** ankita_wagh has quit IRC | 20:51 | |
gordc | Putta: faith. :) | 20:52 |
gordc | you can run "rabbitmqctl list_queues" | 20:52 |
gordc | you should see ceilometer-pipe-* queues corresponding to your defined pipeline.yaml | 20:53 |
gordc | i will check to see if there are any logs... i believe there are | 20:53 |
*** thorst has quit IRC | 20:54 | |
gordc | Putta: there should be a log "Joined partitioning group xyz" | 20:55 |
gordc | if you have debug enable you'll see log that says "Members of group: xyz" | 20:55 |
Putta | gordc: those messages are in central-agent log files | 20:56 |
gordc | yes. you'll see similar log in notification agent logs since we use same coordination mechanism. | 20:57 |
*** tongli has quit IRC | 20:57 | |
*** tongli has joined #openstack-ceilometer | 20:57 | |
gordc | central agent doesn't need workload_partitioning option.. it's enable implicitly when you launch multiple centrals agents with backend_url defined. | 20:57 |
*** thorst has joined #openstack-ceilometer | 20:58 | |
Putta | gordc: thank you so much. although i have faith, i will test it out :) | 21:01 |
gordc | Putta: good choice. let us know if you run into any issues. we can open bug if anything. | 21:01 |
Putta | gordc: sure, thanks again for the help. | 21:02 |
gordc | np | 21:04 |
openstackgerrit | gordon chung proposed openstack/ceilometer: clean up expected errors in test https://review.openstack.org/204762 | 21:08 |
*** pauldix has quit IRC | 21:09 | |
*** pauldix has joined #openstack-ceilometer | 21:10 | |
*** ddaskal has quit IRC | 21:13 | |
*** jruano has quit IRC | 21:15 | |
*** david-lyle has quit IRC | 21:17 | |
*** exploreshaifali has quit IRC | 21:18 | |
*** jruano has joined #openstack-ceilometer | 21:19 | |
*** yassine__ has quit IRC | 21:22 | |
*** pauldix has quit IRC | 21:23 | |
*** pauldix has joined #openstack-ceilometer | 21:25 | |
*** ddaskal has joined #openstack-ceilometer | 21:26 | |
*** pauldix has quit IRC | 21:30 | |
*** tongli has quit IRC | 21:37 | |
*** tongli has joined #openstack-ceilometer | 21:37 | |
*** exploreshaifali has joined #openstack-ceilometer | 21:44 | |
*** tongli has quit IRC | 21:46 | |
*** tongli has joined #openstack-ceilometer | 21:46 | |
*** jaypipes has quit IRC | 21:46 | |
*** tongli has quit IRC | 21:48 | |
*** edmondsw has quit IRC | 21:54 | |
*** Kennan has quit IRC | 22:02 | |
*** Kennan has joined #openstack-ceilometer | 22:03 | |
*** gordc has quit IRC | 22:06 | |
*** htruta_ has joined #openstack-ceilometer | 22:07 | |
*** thorst has quit IRC | 22:10 | |
*** Putta has quit IRC | 22:15 | |
*** cchang has quit IRC | 22:26 | |
*** ddaskal has quit IRC | 22:27 | |
*** exploreshaifali has quit IRC | 22:41 | |
*** prad has quit IRC | 22:53 | |
*** rbak has quit IRC | 22:53 | |
*** ddieterly has joined #openstack-ceilometer | 23:07 | |
*** shardy_ has joined #openstack-ceilometer | 23:12 | |
*** shardy has quit IRC | 23:14 | |
*** shardy_ has quit IRC | 23:17 | |
*** shardy has joined #openstack-ceilometer | 23:18 | |
*** jwcroppe has quit IRC | 23:19 | |
*** Putta has joined #openstack-ceilometer | 23:26 | |
*** Putta has quit IRC | 23:27 | |
*** ankita_wagh has joined #openstack-ceilometer | 23:38 | |
*** ankita_w_ has quit IRC | 23:41 | |
*** jaosorior has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!