NOTICE: This channel is no longer actively logged.
[00:02:08] *** miclorb has joined #jboss-as7[00:04:31] *** jclingan has quit IRC[00:50:01] *** aslak has joined #jboss-as7[00:50:02] *** aslak has joined #jboss-as7[00:50:02] *** ChanServ sets mode: +v aslak[00:54:27] *** alesj has quit IRC[00:59:47] *** laneuis has joined #jboss-as7[01:11:42] *** miclorb has quit IRC[01:24:17] *** miclorb has joined #jboss-as7[02:09:37] *** echelog-2 has joined #jboss-as7[02:22:07] *** mbg has joined #jboss-as7[02:22:08] *** ChanServ sets mode: +v mbg[02:39:23] *** jwulf has joined #jboss-as7[02:47:54] *** mbg has quit IRC[02:53:24] *** Nihility has joined #jboss-as7[02:53:25] *** Nihility has joined #jboss-as7[02:53:25] *** ChanServ sets mode: +v Nihility[02:57:45] *** aslak has quit IRC[03:06:35] *** laneuis has left #jboss-as7[03:21:09] <Nihility> smarlow: did you do any kind of verification that this patch worked, you mentioned that it doesnt work with the order manager app?[03:42:59] *** jpearlin has quit IRC[03:45:29] *** miclorb has quit IRC[03:56:28] *** mmoyses has joined #jboss-as7[03:56:29] *** ChanServ sets mode: +v mmoyses[03:56:38] *** mmoyses has quit IRC[03:59:03] <smarlow> Nihility: I single stepped through the patch and it worked perfectly, as far as I could tell. We loaded the entities.[03:59:50] <smarlow> what I meant about the order manager app is that something else seemed to break in the app (independent of JPA)[03:59:51] <Nihility> smarlow: could you look into adding a simple test case that uses some annotated entities[04:00:22] <smarlow> Nihility: sure, I'll do that.[04:00:41] <Nihility> smarlow: ill check it out with the order app, maybe i can figure it out. its probably breaking at the sql level[04:01:15] <smarlow> Nihility: it appears to be broken, before that. The web ui doesn't show up in my browser anymore[04:02:12] <smarlow> unless I'm hitting the wrong url (http://localhost:8080/OrderManagerWeb)[04:04:16] <stuartdouglas> It looks like it is possible to add env-entries to application.xml now[04:04:43] <stuartdouglas> but I don't really see how that makes sense, as there is no application wide java:comp[04:07:58] *** miclorb has joined #jboss-as7[04:13:31] <Nihility> smarlow: we (acutally john) fixed the ear context path stuff[04:13:51] <Nihility> stuartdouglas: IIRC those env entires are in app/env[04:14:12] <Nihility> smarlow: so its now /Order IIRC see application.xml[04:14:13] <stuartdouglas> that would make sense[04:15:25] *** JimMa has joined #jboss-as7[04:15:26] <smarlow> ahh, will try that (context root is /Order in the app)[04:15:37] <smarlow> application.xml[04:15:53] *** miclorb has quit IRC[04:16:35] *** miclorb_ has joined #jboss-as7[04:19:55] * smarlow the sample app UI is working when I hit /Order... Will create a new Order entry now and see what JPA does...[04:22:31] <smarlow> Nihility: I get the expected SQL error because I'm not using the right database but otherwise looks good.[04:22:36] <smarlow> SQL Error: 90079, SQLState: 90079[04:22:54] <smarlow> Schema "EJB3" not found;[04:24:31] <smarlow> I'll squash my changes and request a merge so others can have the fix[04:25:52] <smarlow> Nihility: did you happen to pull the fix from my branch yet that I emailed you the link for? If not, I'll go ahead and squash a few changes together[04:29:32] *** miclorb_ has quit IRC[04:30:38] *** miclorb has joined #jboss-as7[04:42:46] *** miclor___ has joined #jboss-as7[04:43:11] *** miclorb has quit IRC[04:47:52] <Nihility> smarlow: no i wanted to make sure it was really ready first[04:48:05] <Nihility> smarlow: a couple things i am worried about are:[04:48:10] <Nihility> 1) leaking the index[04:48:25] <smarlow> I fixed 1)[04:49:02] *** miclor___ has quit IRC[04:49:28] <Nihility> 2) use of the TL (which seems to leak)[04:49:44] *** miclorb has joined #jboss-as7[04:49:48] <Nihility> also a third[04:50:08] <Nihility> 3) is there any possibility of concurrent access on this index?[04:51:30] <Nihility> cool[04:51:44] *** miclorb_ has joined #jboss-as7[04:51:57] *** miclorb has quit IRC[04:52:42] <smarlow> I already have the clearing of the TL in a finally but not triple protected (I'm cleaning three things up that shouldn't throw exceptions but i could put each of those in a try/finally as well.)[04:53:10] <smarlow> that could help with TL leakage[04:54:13] <smarlow> I'm not sure about concurrent access on this index. Its attached to the deployment. I'm not personally passing the deployment to other threads[04:54:51] <smarlow> and I'm clearing my reference to it very early (after the container entity managed factory is created)[04:55:25] *** miclorb_ has quit IRC[04:56:32] <smarlow> Nihility: I'm already loading the entity classes also, since Hibernate wants Classes returned to them. I also derive a list of packages from the Classes but that could be hacked around (could just parse the classname)[04:59:10] *** bstansberry has quit IRC[04:59:13] *** bstansberry_ has joined #jboss-as7[04:59:13] *** bstansberry_ has joined #jboss-as7[04:59:13] *** ChanServ sets mode: +v bstansberry_[05:00:27] <smarlow> Nihility: I'm adding triple protection against leaking the index :) finally {try { } finally { try {} finally {} } } :-)[05:02:23] <smarlow> so, other than the classloading that I felt guilty about doing, after using jandex so efficiently, we should be okay[05:02:59] <Nihility> oh you are clearing it[05:03:02] <Nihility> i didnt see that[05:04:46] <smarlow> yeah, its in a separate commit[05:05:11] <smarlow> https://github.com/scottmarlow/jboss-as/commit/6c72ca83c75e19bb91de4416049e198a17686df0[05:05:12] <jbossbot> git [jboss-as] 6c72ca8.. Scott Marlow JBAS-9141 close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory)[05:05:14] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[05:06:33] <smarlow> I also changed the JPA demo's persistence.xml to depend on our new HibernateAnnotationScanner[05:06:35] <smarlow> https://github.com/scottmarlow/jboss-as/commit/ae93841491b04cc629b15241603bca8583ec76e2[05:06:35] <jbossbot> git [jboss-as] ae93841.. Scott Marlow jpa demo change to depend on HibernateAnnotationScanner for reading entity classes[05:07:42] * smarlow still need to write some integration tests which I started but need to get back to[05:08:23] <smarlow> Nihility: I'll go ahead and squash these together now.[05:08:50] * smarlow starting to like git stash[05:10:49] <smarlow> Nihility: squashed commit is at https://github.com/scottmarlow/jboss-as/commit/c607a6aab0533b9c0647b3f252386e402eecd1ac[05:10:49] *** Nihility has quit IRC[05:10:50] <jbossbot> git [jboss-as] c607a6a.. Scott Marlow JBAS-9141 passing Hibernate annotation scanner in via hibernate.ejb.resource_scanner property. close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory). jpa demo change to depend on HibernateAnnotationScanner for reading entity classes[05:10:50] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[05:11:16] *** Nihility has joined #jboss-as7[05:11:16] *** Nihility has joined #jboss-as7[05:11:16] *** ChanServ sets mode: +v Nihility[05:11:23] <smarlow> Nihility: squashed commit is at https://github.com/scottmarlow/jboss-as/commit/c607a6aab0533b9c0647b3f252386e402eecd1ac[05:11:24] <jbossbot> git [jboss-as] c607a6a.. Scott Marlow JBAS-9141 passing Hibernate annotation scanner in via hibernate.ejb.resource_scanner property. close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory). jpa demo change to depend on HibernateAnnotationScanner for reading entity classes[05:11:25] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[05:11:57] *** yyang has joined #jboss-as7[05:33:47] *** jclingan has joined #jboss-as7[05:40:17] *** miclorb_ has joined #jboss-as7[05:59:48] *** smarlow has quit IRC[06:39:55] *** irooskov has quit IRC[06:47:26] *** maeste has joined #jboss-as7[06:47:26] *** ChanServ sets mode: +v maeste[08:01:30] *** jwulf has quit IRC[08:14:32] *** lgao has joined #jboss-as7[08:26:54] *** jclingan has quit IRC[08:34:04] *** jfclere has joined #jboss-as7[08:34:05] *** ChanServ sets mode: +v jfclere[08:48:53] *** stalep has joined #jboss-as7[08:55:34] *** jwulf has joined #jboss-as7[09:01:29] *** miclorb_ has quit IRC[09:03:42] *** tdiesler has joined #jboss-as7[09:03:43] *** ChanServ sets mode: +v tdiesler[09:13:25] *** wolfc has joined #jboss-as7[09:13:26] *** ChanServ sets mode: +v wolfc[09:14:26] *** adietisheim has joined #jboss-as7[09:17:24] *** bstansberry_ has quit IRC[09:17:55] *** jwulf has quit IRC[09:21:17] *** rmaucher has joined #jboss-as7[09:27:26] *** pilhuhn has joined #jboss-as7[09:27:27] *** pilhuhn has joined #jboss-as7[09:27:27] *** ChanServ sets mode: +v pilhuhn[09:27:38] *** opalka has joined #jboss-as7[09:27:38] *** opalka has joined #jboss-as7[09:27:38] *** ChanServ sets mode: +v opalka[09:27:46] <opalka> morning[09:29:50] *** cvasilak has joined #jboss-as7[09:30:30] *** anguenot has joined #jboss-as7[09:31:34] *** mgoldmann has joined #jboss-as7[09:31:34] *** ChanServ sets mode: +v mgoldmann[09:45:45] *** emuckenhuber has joined #jboss-as7[09:45:46] *** emuckenhuber has joined #jboss-as7[09:45:46] *** ChanServ sets mode: +v emuckenhuber[09:49:37] *** cvasilak has left #jboss-as7[09:49:46] *** Jaikiran has joined #jboss-as7[09:49:46] *** ChanServ sets mode: +v Jaikiran[09:51:42] *** cvasilak has joined #jboss-as7[09:52:35] *** cvasilak has left #jboss-as7[09:53:18] *** cvasilak has joined #jboss-as7[10:01:02] *** jwulf has joined #jboss-as7[10:05:53] *** torben has joined #jboss-as7[10:05:54] *** ChanServ sets mode: +v torben[10:06:40] *** torben has quit IRC[10:07:41] *** ALR has joined #jboss-as7[10:07:42] *** ChanServ sets mode: +v ALR[10:11:33] *** rotula has joined #jboss-as7[10:15:15] *** dimitris_ has joined #jboss-as7[10:15:16] *** dimitris_ has joined #jboss-as7[10:15:16] *** ChanServ sets mode: +v dimitris_[10:18:26] *** maeste has quit IRC[10:18:35] *** rotula has quit IRC[10:25:17] *** asoldano has joined #jboss-as7[10:25:18] *** ChanServ sets mode: +v asoldano[10:26:15] *** ALR has left #jboss-as7[10:31:25] *** maeste has joined #jboss-as7[10:31:26] *** ChanServ sets mode: +v maeste[10:33:39] *** maximilienw has joined #jboss-as7[10:34:13] <wolfc> Regression in testsuite http://hudson.qa.jboss.com/hudson/view/JBoss%20AS/job/JBoss-AS-7.0.x-testSuite-sun16/18/testReport/org.jboss.as.testsuite.integration.jpa.epcpropagation/EPCPropagationTestCase/testBMTPropagation/[10:34:20] *** emmanuel has joined #jboss-as7[10:34:21] *** ChanServ sets mode: +v emmanuel[10:34:23] <wolfc> I'm going after it[10:34:46] <wolfc> Actually new failures[11:04:41] *** sannegrinovero has joined #jboss-as7[11:04:42] *** sannegrinovero has quit IRC[11:04:42] *** sannegrinovero has joined #jboss-as7[11:04:42] *** ChanServ sets mode: +v sannegrinovero[11:12:01] *** anguenot has quit IRC[11:15:44] *** jcosta has joined #jboss-as7[11:15:44] *** ChanServ sets mode: +v jcosta[11:18:18] *** opalka has quit IRC[11:18:38] *** opalka has joined #jboss-as7[11:18:38] *** opalka has joined #jboss-as7[11:18:38] *** ChanServ sets mode: +v opalka[11:20:31] *** aslak has joined #jboss-as7[11:20:31] *** aslak has joined #jboss-as7[11:20:31] *** ChanServ sets mode: +v aslak[11:26:28] <Jaikiran> rmaucher: got a minute?[11:26:41] <rmaucher> ok[11:27:11] <Jaikiran> does the current AS7 upstream support injection in tag lib classes?[11:27:54] <rmaucher> no, the instance manager hasn't been ported[11:28:03] <Jaikiran> ok[11:28:06] <Jaikiran> thanks[11:29:07] <rmaucher> (OTOH, maybe it does, there's an instance manager already)[11:29:31] *** alesj has joined #jboss-as7[11:31:26] <Jaikiran> some tck tests show that a tag lib class (one which implements SimpleTag) isn't identified as a "component class" and hence isn't being processed[11:31:32] <Jaikiran> the servlets and filters are fine though[11:31:39] <Jaikiran> they get picked up as component classes[11:31:59] <rmaucher> ah maybe it filters our anything that isn't a filter or servlet[11:32:10] <rmaucher> it would be arbitraty I think[11:32:38] <Jaikiran> right, currently it filters out anything other than those pre-defined component types[11:33:31] <rmaucher> that's overzealous ;)[11:35:07] <Jaikiran> to some extent it does allow skipping of arbitrary classes which can cause issues (for ex: @Resource on some spring bean)[11:35:36] <rmaucher> ok[11:37:16] <rmaucher> so I think there's an opportunity to compile the list of tag classnames, listeners, etc[11:38:06] <Jaikiran> yep[11:38:50] <Jaikiran> is the instance manager responsible for handling JSF managed beans too?[11:39:15] <Jaikiran> those too can be some arbitrary classes[11:47:22] *** anguenot has joined #jboss-as7[11:47:39] *** maximilienw has left #jboss-as7[11:48:02] <rmaucher> all instances are supposed to go through that[11:48:09] *** jhalliday has joined #jboss-as7[11:48:34] <rmaucher> so maybe the filter in strategy that is used at the moment is not a good idea[11:48:53] <rmaucher> I can get a list of tag classes, I cannot know about JSF stuff[12:07:58] *** jclingan has joined #jboss-as7[12:09:34] *** jclingan1 has joined #jboss-as7[12:12:13] *** jclingan has quit IRC[12:13:38] *** emmanuel has quit IRC[12:33:11] *** hbraun has joined #jboss-as7[12:33:20] <hbraun> emuckenhuber: ping[12:36:02] <rmaucher> ok, so ... pull request: https://github.com/rmaucher/jboss-as/commit/76ce2238f67c97479b8584176166e976c4a05417[12:36:03] <jbossbot> git [jboss-as] 76ce223.. Rémy Maucherat Add tag classes to the list of injectable classes.[12:39:33] <emuckenhuber> hbraun: pong[12:39:34] *** jclingan1 has quit IRC[12:39:41] <hbraun> guten morgen[12:39:49] <emuckenhuber> hehe, guten morgen[12:39:51] <hbraun> oder mahlzeit besser gesagt[12:40:03] <hbraun> did you see my latest emails?[12:40:15] <emuckenhuber> not yet, just came back from lunch[12:40:26] <hbraun> wrt a) system prop on server groups and b) update-resource-operations[12:40:40] <hbraun> [localhost:9999 /] /server-group=development-group:read-children-types[12:40:40] <hbraun> {[12:40:40] <hbraun> "outcome" => "success",[12:40:40] <hbraun> "result" => [[12:40:40] <hbraun> "jvm",[12:40:41] <hbraun> "deployment"[12:40:41] <hbraun> ],[12:40:41] <hbraun> "compensating-operation" => undefined[12:40:41] <hbraun> }[12:40:44] <emuckenhuber> let me check[12:40:49] <hbraun> looks like there are no system porperties on server groups anymore[12:43:19] <emuckenhuber> hmm, ok about a) system-properties - they are there, however registered as attributes[12:43:33] <emuckenhuber> therefore they don't show up in the read-children-types[12:44:03] <emuckenhuber> which i agree is a bit weird - they might should be a "subModel" as well[12:44:14] <hbraun> that#s what I was expecting[12:44:27] <hbraun> how are they treated on other elements?[12:44:32] <emuckenhuber> the same[12:44:49] <emuckenhuber> afaik they even have special operation names[12:45:00] <emuckenhuber> "add-system-property"[12:45:09] <emuckenhuber> and remove-system-property[12:45:25] <hbraun> they show up here:[12:45:26] <hbraun> [localhost:9999 /] /host=local/server-config=server-one:read-resource[12:45:26] <hbraun> {[12:45:26] <hbraun> "outcome" => "success",[12:45:27] <hbraun> "result" => {[12:45:27] <hbraun> "path" => undefined,[12:45:27] <hbraun> "system-property" => undefined,[12:45:27] <hbraun> "interface" => undefined,[12:45:28] <hbraun> "jvm" => {"default" => undefined},[12:45:28] <hbraun> "socket-binding-group" => undefined,[12:45:29] <hbraun> "socket-binding-port-offset" => undefined,[12:45:29] <hbraun> "name" => "server-one",[12:45:30] <hbraun> "group" => "main-server-group",[12:45:30] <hbraun> "auto-start" => true[12:46:08] <hbraun> does this mean they are implemented as subresources on server-config resources?[12:46:47] <emuckenhuber> not really, it's just that we initialize the model value[12:47:10] <hbraun> but regarding the op names: I cannot see special op's for system props[12:48:07] <emuckenhuber> ah ok - for server-groups they don't seem to be registered :-/[12:48:15] <emuckenhuber> and the resource description is missing as well[12:48:26] <hbraun> ok, that explains it[12:49:09] <hbraun> can you give me an example where they are registered?[12:49:14] *** JimMa has quit IRC[12:50:31] <emuckenhuber> hmm on the domain and host root[12:50:37] <emuckenhuber> they should be registered[12:51:10] <emuckenhuber> on server-group and server-config they are missing[12:51:11] <hbraun> i see[12:53:05] <hbraun> i've created an issue: https://issues.jboss.org/browse/JBAS-9160[12:53:06] <jbossbot> jira [JBAS-9160] Provide <add/remove>-system-property operations on server-config and server-group [Open (Unresolved) Feature Request, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9160[12:53:35] <emuckenhuber> ok, thanks - i'm going to add them[12:53:45] <hbraun> what about " b) update-resource-operations"[12:53:47] <hbraun> ?[12:54:12] <hbraun> there is add/remove resource, but how do I update an existing one?[12:54:22] <hbraun> i.e. a server-group?[12:54:34] <hbraun> changing the JVM property at some later point[12:54:43] <emuckenhuber> right - so we don't really have a generic update operation[12:55:05] <hbraun> but it makes sense, do you agree?[12:55:26] <emuckenhuber> well for now we allow write-attribute handlers[12:55:39] <emuckenhuber> so you can write single attributes[12:55:49] <emuckenhuber> although i guess there are quite some attribute handlers missing[12:56:03] <hbraun> the problem in my case is that I don't know what has changed[12:56:21] <hbraun> at least I don't track every single property and I don't like to[12:57:05] <hbraun> does it makes sense to raise an issue?[12:57:27] <emuckenhuber> yes, we should talk with Brian and see if we can add a generic "update" operations[12:57:41] <hbraun> I can create a feature request[12:57:42] <emuckenhuber> which leverages the attribute handlers and compares the values[12:58:11] <hbraun> right, in that sense it can be treated as a composite write attribute operation[12:58:15] <emuckenhuber> at least i remember this was a big issue with the AS5 console and ProfileService we couldn't really solve[12:58:32] <hbraun> what kind of issue[12:58:34] <hbraun> ?[12:59:37] *** darranl has joined #jboss-as7[12:59:37] *** darranl has joined #jboss-as7[12:59:37] *** ChanServ sets mode: +v darranl[12:59:38] <emuckenhuber> that from the the console was just sending the complete update and figuring out what really changed on the server-side did not work out either[12:59:55] <hbraun> why would you need to figure it out?[13:00:11] <hbraun> performance optimization, yes maybe[13:00:19] <hbraun> but otherwise?[13:00:42] <hbraun> or do you have actions that are executed when attributes change?[13:00:50] <emuckenhuber> more fine grained updates and also if "null" means "null" or not changed :)[13:01:01] <emuckenhuber> some issues like that, but can't really remember the edge cases[13:01:27] *** hbraun has left #jboss-as7[13:01:58] *** hbraun has joined #jboss-as7[13:02:02] <emuckenhuber> yes, "attribute" changes could also affect runtime state[13:02:08] <hbraun> here we go: JBAS-9161[13:02:09] <jbossbot> jira [JBAS-9161] Provide a "update-resource-operation" [Open (Unresolved) Feature Request, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9161[13:02:50] <hbraun> I can see the problem, but I believe pushing this decision to each client implementation doesn't work either[13:04:16] <hbraun> I do assign these to you for now. Please delegate them accordingly[13:04:48] <emuckenhuber> yes, the single read/write-attribute is nice for the CLI and unit tests - however i doubt it's that for i.e. a web console[13:05:14] <emuckenhuber> ok, i'll chat with Brian and see what he thinks re: update-resource-op[13:06:02] <hbraun> right, in the CLI you work with a different conceptual model. Here it makes sense to change one thing at a time[13:06:25] <hbraun> but in the console you work on a larger picture: i.e. a full entity[13:06:31] <hbraun> (server-group)[13:06:59] <hbraun> in this case enither the user nor the technology will track single attribute modifications[13:07:33] <hbraun> thanks[13:23:11] *** pmuir has joined #jboss-as7[13:23:11] *** pmuir has joined #jboss-as7[13:23:11] *** ChanServ sets mode: +v pmuir[13:28:01] *** epbernard has joined #jboss-as7[13:28:01] *** epbernard is now known as emmanuel[13:28:02] *** ChanServ sets mode: +v emmanuel[13:37:34] <pilhuhn> Wasn't there the idea of batching of stuff (composite ops) to write multiple attributes at once?[13:37:54] <pilhuhn> But I agree, for write-attribute I'd like to update more than one attribute at a time[13:38:03] <hbraun> yes, but that more addressing operations across resources[13:38:34] <hbraun> IMO it#s very unnatural to write single attributes[13:38:40] <hbraun> at least in the consoel[13:38:46] <hbraun> CLI is a differnt story[13:39:11] <pilhuhn> so instead of {operation=write-attribute, address=(..), name=x,value=y} this may need to change to {ops=..,addr=,[{name=,value=}*}[13:39:44] <pilhuhn> hbraun same here[13:40:07] <pilhuhn> hbraun is the console something to download separately -or is that in master?[13:40:28] <hbraun> it's an external binary that#s pulled as a dependency[13:40:37] <hbraun> http://localhost:9990/console/index.html[13:40:45] <hbraun> that#s where you get it[13:40:57] <hbraun> but currently there isn't much[13:41:25] <hbraun> beta2 will include some management ops like start/stop server and such[13:41:32] <hbraun> create groups, etc[13:41:35] *** yyang has quit IRC[13:41:42] <hbraun> but mostly domain management[13:41:51] <hbraun> didn't look into standalone yet[13:41:58] <pilhuhn> Ah, I was missing the /index.html part - that's why it did not show up - thanks[13:42:09] <hbraun> yes, I'll fix that too[13:42:13] <hbraun> it's annyoing[13:42:14] <hbraun> ;)[13:43:17] <hbraun> https://issues.jboss.org/browse/JBAS-9162[13:43:19] <jbossbot> jira [JBAS-9162] Serve console from "/console" (w/o index.html) [Open (Unresolved) Feature Request, Major, Heiko Braun] https://issues.jboss.org/browse/JBAS-9162[13:45:40] <hbraun> emmanuel: ping[13:45:53] <hbraun> emmanuel: sorry, wrong person[13:45:59] <hbraun> emuckenhuber: ping[13:46:03] <emmanuel> hbraun: arrrrrrrrggggggggggg[13:46:11] <hbraun> emmanuel: ping[13:46:15] <hbraun> emmanuel: sorry[13:46:21] <hbraun> ;)[13:46:34] <emuckenhuber> hbraun: pong[13:46:45] *** aamonten has joined #jboss-as7[13:46:49] <hbraun> emuckenhuber: I am looking for the op to add new server configurations[13:46:55] <emuckenhuber> i guess i should think of using a different nick ;)[13:47:21] <hbraun> [localhost:9999 /] /host=local:read-operation-names[13:47:22] <hbraun> {[13:47:22] <hbraun> "outcome" => "success",[13:47:23] <hbraun> "result" => [[13:47:23] <hbraun> "add-namespace",[13:47:23] <hbraun> "add-schema-location",[13:47:24] <hbraun> "add-system-property",[13:47:24] <hbraun> "read-attribute",[13:47:24] <hbraun> "read-children-names",[13:47:25] <hbraun> "read-children-resources",[13:47:25] <hbraun> "read-children-types",[13:47:26] <hbraun> "read-operation-description",[13:47:26] <hbraun> "read-operation-names",[13:47:26] <hbraun> "read-resource",[13:47:30] *** asoldano is now known as asoldano_lunch[13:48:15] <hbraun> emuckenhuber: I would expect something like :add-server-config, :remove-server-config[13:49:05] <emuckenhuber> hbraun: actually in most cases all the "add stuff ops" are like: /host=local/server-config=new-server-config:add(....)[13:49:09] <emmanuel> emuckenhuber: how about theotherone ;)[13:49:51] <hbraun> emmanuel: "therightone" ?[13:50:15] <hbraun> emuckenhuber: i don't get it[13:50:27] <emuckenhuber> hehe, yeah i guess it should be enough to use "emanuel" - at least that should fix the irc autocomplete :)[13:50:55] <emuckenhuber> hbraun: basically adding a new submodel the "name" is part of the address[13:51:21] <hbraun> emuckenhuber: you mean calling add() on the host element?[13:51:39] <hbraun> ah, i see what you mean[13:51:57] <maeste> hbraun: ping[13:52:17] <hbraun> emuckenhuber: similar to this one: final ModelNode group = new ModelNode();[13:52:17] <hbraun> group.get(OP).set(ModelDescriptionConstants.ADD);[13:52:17] <hbraun> group.get(ADDRESS).add(ModelDescriptionConstants.SERVER_GROUP, record.getGroupName());[13:52:17] <hbraun> group.get("profile").set(record.getProfileName());[13:52:27] <hbraun> maeste: pong[13:52:55] <emuckenhuber> right - so that's the most common case... as for system-properties i don't really know why we did not use the same addressing scheme[13:53:11] <maeste> hbraun: let say that I'm able to read attribute and add/remove datasources from jboss-admin.sh what should I have to do to enable it in web console?[13:53:30] <hbraun> maeste: magic?[13:53:49] <hbraun> maeste: it needs to be implemented[13:53:58] <hbraun> maeste: probably by me[13:54:03] <maeste> hbraun: oki[13:54:07] <hbraun> maeste: ;)[13:54:26] <maeste> hbraun: :)[13:54:38] <hbraun> maeste: that's the reason why I said: provide a reasonable set of management ops first, then I'll look into it[13:55:07] <hbraun> maeste: basically anything you can do from the CLI can then be supported (and implemented) in the console[13:55:09] *** dimitris_ has quit IRC[13:55:18] <maeste> hbraun: oki I'll write you an email today or tomorrow with a resonable list :)[13:55:46] <hbraun> maeste: well, send me an update when it's part of AS7 upstream[13:56:21] <maeste> hbraun: oki, at middle of the week so[13:56:31] <hbraun> maeste: great[14:13:47] *** mmoyses has joined #jboss-as7[14:13:47] *** ChanServ sets mode: +v mmoyses[14:15:42] *** jpederse has joined #jboss-as7[14:26:22] *** dimitris_ has joined #jboss-as7[14:26:23] *** dimitris_ has joined #jboss-as7[14:26:23] *** ChanServ sets mode: +v dimitris_[14:31:37] *** cvasilak has quit IRC[14:32:35] *** cvasilak has joined #jboss-as7[14:33:48] *** tdiesler has quit IRC[14:45:38] *** asoldano_lunch is now known as asoldano[14:46:44] *** aloubyansky has joined #jboss-as7[14:46:44] *** ChanServ sets mode: +v aloubyansky[14:51:07] *** mmoyses has quit IRC[14:51:13] *** mmoyses has joined #jboss-as7[14:51:13] *** ChanServ sets mode: +v mmoyses[14:55:05] *** jpederse has quit IRC[14:55:05] *** jpederse has joined #jboss-as7[15:03:02] *** torben has joined #jboss-as7[15:03:02] *** torben has joined #jboss-as7[15:03:02] *** ChanServ sets mode: +v torben[15:06:19] *** lgao has quit IRC[15:07:16] *** mmoyses has quit IRC[15:09:26] *** fnasser has joined #jboss-as7[15:13:12] *** bstansberry has joined #jboss-as7[15:13:13] *** ChanServ sets mode: +v bstansberry[15:18:31] *** jpederse_ has joined #jboss-as7[15:18:31] *** jpederse has quit IRC[15:20:37] *** ccrouch has joined #jboss-as7[15:20:37] *** ChanServ sets mode: +v ccrouch[15:21:43] *** jpederse_ has quit IRC[15:25:56] *** frainone has joined #jboss-as7[15:25:56] *** ChanServ sets mode: +v frainone[15:26:27] *** smarlow has joined #jboss-as7[15:26:28] *** ChanServ sets mode: +v smarlow[15:33:46] *** pferraro has joined #jboss-as7[15:33:46] *** ChanServ sets mode: +v pferraro[15:38:26] * smarlow going to squash once more and request a merge...[15:43:39] *** mmoyses has joined #jboss-as7[15:43:39] *** ChanServ sets mode: +v mmoyses[15:44:04] *** jpederse has joined #jboss-as7[15:44:13] <smarlow> dmlloyd, Nihility, bstansberry: Could someone please merge https://github.com/scottmarlow/jboss-as/commit/74f67c4673f20080317219326df40013b69a09ef (this should help us get further with Andy's deployment)[15:44:14] <jbossbot> git [jboss-as] 74f67c4.. Scott Marlow JBAS-9141 passing Hibernate annotation scanner in via hibernate.ejb.resource_scanner property. close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory). jpa demo change to depend on HibernateAnnotationScanner for reading entity classes. JBAS-9141 make sure the TL gets[15:44:15] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[15:44:16] *** jpederse has quit IRC[15:44:16] *** jpederse has joined #jboss-as7[15:47:01] <pilhuhn> emuckenhuber is it on purpose, that socket-bindings in domain mode have no port-offset defined, while in standalone mode they do?[15:47:11] <pilhuhn> e.g. http://localhost:9990/domain-api/socket-binding-group/standard-sockets?json.pretty=true&recursive[15:48:18] <emuckenhuber> pilhuhn: yeah i think so, since in the domain mode you would rather set port offsets on the server-group, host or server-config level[15:49:21] <pilhuhn> emuckenhuber I could imagine that users want to define the groups "top level" and then just use them further down (s-g, host)[15:54:32] <wolfc> smarlow: ping[15:54:43] *** mmoyses has quit IRC[15:55:02] <smarlow> wolfc: Hi[15:55:31] <wolfc> smarlow, I'm looking at http://hudson.qa.jboss.com/hudson/view/JBoss%20AS/job/JBoss-AS-7.0.x-testSuite-sun16/18/testReport/org.jboss.as.testsuite.integration.jpa.epcpropagation/EPCPropagationTestCase/testBMTPropagation/[15:55:48] <smarlow> wolfc: thanks![15:55:53] <wolfc> https://github.com/wolfc/jboss-as/commits/jpa[15:55:58] <wolfc> smarlow, has that test ever passed?[15:55:59] *** asaldhan has joined #jboss-as7[15:55:59] *** ChanServ sets mode: +v asaldhan[15:56:15] <smarlow> wolfc: no, not yet.[15:57:02] <asaldhan> bstansberry: hey I have a pull request from yesterday. Hope one of you pull it today.[15:57:27] <bstansberry> ok[15:57:36] <smarlow> wolfc: I can drop the @Local but am curious of its too early to try and get that test working?[15:58:14] <wolfc> smarlow, I'm passed that[15:58:39] <asaldhan> smarlow: integration test EPCPropagation test was failing yesterday for me.[15:58:49] <wolfc> smarlow, right now I think XPC prop isn't working because SFSBContextHandleImpl doesn't implement equals and hashCode. While it is a key in a map.[15:59:34] <wolfc> smarlow, I'm still fiddling bits and pieces[15:59:44] *** mmoyses has joined #jboss-as7[15:59:45] *** ChanServ sets mode: +v mmoyses[16:00:40] <smarlow> wolfc: easy enough to add equals + hashcode. Should I do that or is it part of your changes already?[16:00:55] <wolfc> smarlow, it's a click in Intellij :-)[16:01:02] <smarlow> :-)[16:01:20] *** jfd has joined #jboss-as7[16:01:20] *** jfd has quit IRC[16:01:20] *** jfd has joined #jboss-as7[16:01:20] *** ChanServ sets mode: +v jfd[16:07:05] <pilhuhn> who is the drummer ?[16:11:23] <smarlow> wolfc, Nihility: added equals/hashcode via https://github.com/scottmarlow/jboss-as/commit/6f4d4ffd0d264018500a57e337079f9226d4bb29[16:11:24] <jbossbot> git [jboss-as] 6f4d4ff.. Scott Marlow add equals and hashCode to SFSBContextHandleImpl[16:13:28] <asoldano> dimitris_, Nihility , dmlloyd : hi, just a reminder... I'd need the jboss-modules release, further jbossws releases to as7 are blocked by that[16:13:56] <dmlloyd> yeah working on it[16:14:04] <asoldano> dmlloyd, thanks :-)[16:14:50] *** pgier has joined #jboss-as7[16:14:50] *** ChanServ sets mode: +v pgier[16:20:09] <jhalliday> dmlloyd: ping[16:20:34] <dmlloyd> what's up jhalliday[16:21:20] <jhalliday> dmlloyd: since moving to the tools generated i18n classes, the method name in the logs is for the wrong stack frame. is there a param to override it when the logger is created?[16:21:48] <dmlloyd> ah. that'd be a bug in the generated class.[16:22:08] <jhalliday> oh good, someone else's problem then :-)[16:22:26] <dmlloyd> yeah it should be sending in the FQCN of the logger implementation class and it must not be.[16:31:56] <bstansberry> asaldhan: you want https://github.com/anilsaldhana/jboss-as/commit/ec0639cb7500efc271d852ead82bfeb6334a1ac4 as well?[16:31:58] <jbossbot> git [jboss-as] ec0639c.. Anil Saldhana JBAS-9153: add test for BASIC auth[16:31:59] <jbossbot> jira [JBAS-9153] Integration test for web security BASIC authentication [Open (Unresolved) Task, Major, Anil Saldhana] https://issues.jboss.org/browse/JBAS-9153[16:32:12] <wolfc> smarlow, https://github.com/wolfc/jboss-as/commits/jpa it now says the 7.6.3.1 error[16:32:13] <asaldhan> bstansberry: just one commit[16:32:30] <asaldhan> bstansberry: right[16:32:37] <asaldhan> bstansberry: it went to two commits?[16:33:00] <bstansberry> asaldhan: you have two in https://github.com/anilsaldhana/jboss-as/commits/[16:33:19] <asaldhan> bstansberry: ok those two[16:36:55] <dmlloyd> jhalliday, LOGTOOL-12.[16:36:57] <jbossbot> jira [LOGTOOL-12] Generated classes should be using a different logger method [Open (Unresolved) Bug, Major, Unassigned] https://issues.jboss.org/browse/LOGTOOL-12[16:38:09] <jhalliday> dmlloyd: thanks[16:40:09] <wolfc> dmlloyd, Nihility, bstansberry, I got for example issue JBAS-9159[16:40:10] <jbossbot> jira [JBAS-9159] Setting beanName to an EJB that does not expose the interface leads to hang [Open (Unresolved) Bug, Critical, Unassigned] https://issues.jboss.org/browse/JBAS-9159[16:40:20] <wolfc> the deployment does not complete and the client is stuck[16:40:26] *** pferraro has quit IRC[16:40:33] <dmlloyd> I really can't help you wolfc[16:40:55] <dmlloyd> saying "it sucks" repeatedly doesn't help get the problem fixed any faster[16:41:18] <wolfc> that's not my point[16:41:27] <dmlloyd> okay, then you have some new info?[16:41:54] <wolfc> I want to know how we can get to something so we see a response of some sort[16:42:29] <Nihility> when the deployment listener returns a list of broken deps[16:42:34] <dmlloyd> we need a listener which can know definitively when everything that can be started, has been started[16:42:36] *** pferraro has joined #jboss-as7[16:42:36] *** ChanServ sets mode: +v pferraro[16:42:38] <Nihility> the deployment will have a status to return[16:42:40] <dmlloyd> which is exactly what I'm working on now[16:42:45] <bstansberry> what is "the client" -- demo? arquillian? FS scanner?[16:43:26] <wolfc> okay cool[16:43:39] <bstansberry> I believe there are JIRAs for dealing with the client behavior for each of those[16:44:35] <Nihility> the "hang" is the deployment api synchronously waiting for an outcome for something that will never complete[16:44:42] <bstansberry> s/believe/know[16:44:47] <wolfc> right, "hang" is an overstatement[16:44:58] * dmlloyd makes a noose[16:45:20] *** pferraro has quit IRC[16:45:42] <Nihility> dmlloyd is spending most of his time fixing JBAS-9010 for about the last week[16:45:44] <jbossbot> jira [JBAS-9010] Provide more comprehensive error messages for missing dependencies [Open (Unresolved) Task, Major, David Lloyd] https://issues.jboss.org/browse/JBAS-9010[16:45:55] *** opalka has quit IRC[16:45:57] <Nihility> but its a complex update[16:46:12] *** aloubyansky has quit IRC[16:46:34] <wolfc> I think we definitely need it. We need AS to be predictable and responsive at all times.[16:47:04] <dmlloyd> of course we need it, that's why I'm doing it :)[16:48:25] <Nihility> i just made it a blocker for beta3, just so its clear that it WILL be fixed[16:48:41] *** jamezp has joined #jboss-as7[16:48:47] <wolfc> excellent[16:50:26] <wolfc> dmlloyd, http://fpaste.org/p6eL/ can I modify jboss-invocation to throw the original exception in MethodInvokingInterceptor?[16:50:40] <dmlloyd> hey jamezp... was wondering if you might have a look at LOGTOOL-12 if you want a change of pace[16:50:41] <jbossbot> jira [LOGTOOL-12] Generated classes should be using a different logger method [Open (Unresolved) Bug, Major, Unassigned] https://issues.jboss.org/browse/LOGTOOL-12[16:51:16] <dmlloyd> wolfc: ah, yeah that's a bug. And also that's why I hate "throws Exception" :)[16:51:23] <wolfc> hehe[16:51:50] <jamezp> dmlloyd: Ah, will do.[16:51:51] <dmlloyd> wolfc: use org.jboss.invocation.Interceptors#rethrow to make life marginally easier[16:51:54] <dmlloyd> jamezp: thanks.[16:51:58] * Nihility goes to change all of his throws Exception to throws Throable[16:52:02] <Nihility> hehehehe[16:52:08] * dmlloyd throws Nihility[16:52:09] <dmlloyd> :)[16:52:18] <dmlloyd> DefenstrationException[16:52:22] <hbraun> Nihility: Exception is useless[16:52:29] <wolfc> will do[16:52:36] <Nihility> throws stuff;[16:52:41] <hbraun> Nihility: if it fails it should blow up[16:52:49] <hbraun> Nihility: the way of the samurai[16:52:50] <wolfc> System.exit(1) :-)[16:52:56] <Nihility> yes awesome[16:53:13] <jamezp> Nihility dmlloyd: If either of you are taking pull requests today :-) https://github.com/jamezp/jboss-as/commit/9cc85ba7cefbaa6d6e07ef4ad617e916163e9789[16:53:14] <jbossbot> git [jboss-as] 9cc85ba.. James Perkins JBAS-8991 Added woodstox modules for JAXP processing. This allowed older JDK's to launch the server.[16:53:15] <jbossbot> jira [JBAS-8991] Cannot start beta1 server on RHEL6 with OpenJDK 1.6 [Open (Unresolved) Bug, Critical, James Perkins] https://issues.jboss.org/browse/JBAS-8991[16:53:17] <hbraun> Nihility: "Exception is so 80s"[16:53:20] <Nihility> System.exit(message.hashCode()) // keep them guessing[16:53:33] *** cvasilak has quit IRC[16:53:43] <smarlow> wolfc: looks good, thanks. I need to make another pass through on general XPC processing to fill in some gaps but it looks like you caught some of them already.[16:55:13] <smarlow> wolfc: how do we submit your changes for merging? Am I supposed to pull from you and submit them or do you do that directly?[16:55:40] <baileyje> Good Morning/Afternoon/Evening everybody[16:55:43] <smarlow> either way is fine with me, just not sure what the process is[16:56:16] <wolfc> smarlow, cherry-pick what you want onto your branch and continue fixing until the test is passing. If I'm going to submit something for upstream it is disablement of the test.[16:56:55] <smarlow> wolfc: great, thanks for the help![16:59:07] <baileyje> Nihility: Did you still want something done with the default we-context name for WARs in EARs?[17:00:03] <Nihility> baileyje: sure[17:00:22] <Nihility> smarlow: im looking at your patch now[17:02:06] <smarlow> Nihility: great, thanks! the renaming of the Hibernate integration sources, seemed to read a little oddly in the patch (but maybe your more used to the git patches than me).[17:02:44] *** torben is now known as torben|call[17:03:27] <jbossbot> git [jboss-as] push master ec0639c.. Anil Saldhana JBAS-9153: add test for BASIC auth[17:03:28] <jbossbot> jira [JBAS-9153] Integration test for web security BASIC authentication [Open (Unresolved) Task, Major, Anil Saldhana] https://issues.jboss.org/browse/JBAS-9153[17:03:28] <jbossbot> git [jboss-as] push master ac96e5d.. Anil Saldhana JBAS-9153: add test for BASIC auth[17:03:28] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/07d3ffc...ac96e5d[17:04:47] <rmaucher> can I get a pull of https://github.com/rmaucher/jboss-as/commit/76ce2238f67c97479b8584176166e976c4a05417 as well ? thanks[17:04:49] <jbossbot> git [jboss-as] 76ce223.. Rémy Maucherat Add tag classes to the list of injectable classes.[17:06:54] <asaldhan> bstansberry: thanks.[17:07:00] *** frainone is now known as frainone_lunch[17:09:33] *** hbraun has quit IRC[17:27:15] *** dimitris_ has quit IRC[17:27:51] <wolfc> dmlloyd, https://github.com/wolfc/jboss-invocation/commit/644f10e8a345b391e3e577885ad51cab5290fe91[17:27:52] <jbossbot> git [jboss-invocation] 644f10e.. Carlo de Wolf Fix exception handling[17:28:52] <dmlloyd> great, thanks.[17:29:13] <jbossbot> git [jboss-invocation] push master 644f10e.. Carlo de Wolf Fix exception handling[17:29:13] <jbossbot> git [jboss-invocation] push master URL: http://github.com/jbossas/jboss-invocation/compare/ad955a0...644f10e[17:29:58] <wolfc> thanks[17:32:14] <baileyje> Nihility: Do we want.... "/SomeEar/SomeWar" for the context root or "/SomeEar.ear/SomeWar"[17:32:39] <baileyje> My fear is a number of customers use a similar if not the same EAR and WAR name sometimes.[17:32:48] <maeste> dmlloyd: ping[17:32:57] <dmlloyd> what's up maeste[17:33:03] <baileyje> and "/TestApp/TestApp" is a weird root[17:33:29] <baileyje> Nihility: of course to those people I say they need to define the root[17:34:03] <maeste> dmlloyd: I need to inject in one of our service a com.arjuna.ats.jbossatx.jta.RecoveryManagerService[17:34:24] <maeste> dmlloyd: but it is created and started as part of ArjunaTransactionManagerService[17:35:12] <maeste> dmlloyd: even if the value is another one. Ant chance to get that RecoveryManager? Or should this MSC service splitted?[17:35:33] <maeste> s/Ant/Any[17:35:34] *** hbraun has joined #jboss-as7[17:35:48] <hbraun> emuckenhuber: ping[17:35:53] <hbraun> emuckenhuber: about JBAS-9130[17:35:56] <jbossbot> jira [JBAS-9130] Create a unified server-config resource that includes the actual instance operations [Open (Unresolved) Feature Request, Major, Emanuel Muckenhuber] https://issues.jboss.org/browse/JBAS-9130[17:36:00] <emuckenhuber> hbraun: yes[17:36:17] <hbraun> this has an impact on the current console implementation[17:36:41] <dmlloyd> maeste: it should be split out. But if there's a way you can get out of the ArjunaTransactionManagerService then that'd be an okay temporary workaround, then you can file an issue for Beta3 to fix it.[17:36:50] <hbraun> emuckenhuber: so I would prefer if it doesn't become a last minute change[17:37:22] <hbraun> emuckenhuber: IMO we can oush it to beta3 even[17:37:23] <emuckenhuber> hbraun: working on it - i can just keep the start/stop operations on the server-config until you're done with the change[17:37:31] <maeste> dmlloyd: unlucky there isn't AFAIK[17:37:43] <maeste> dmlloyd: there is no pointer between the 2[17:37:46] <hbraun> emuckenhuber: well, if you are done by tomorrow I don't see a problem[17:38:08] <emuckenhuber> ok[17:38:17] <hbraun> emuckenhuber: I basically just need some time to patch the console[17:38:20] <dmlloyd> maeste: okay then it has to be split out first I guess[17:38:31] <hbraun> emuckenhuber: but no pressure from my side[17:38:34] <maeste> dmlloyd: in fact it's a blocker issue for our next pull request[17:38:42] *** asoldano is now known as asoldano_away[17:39:00] <hbraun> emuckenhuber: i don't necessarily need it in beta2[17:39:03] <maeste> dmlloyd: I'll create a Jira, or do you prefer to do it yourself[17:39:34] <dmlloyd> go ahead, maeste[17:40:18] <hbraun> emuckenhuber: if I understand it correctlly, anything "instance" related will go under the "server" eresource?[17:40:59] <hbraun> emuckenhuber: or is it the other way around? "server" will go and "server-config" be the address[17:41:29] <hbraun> emuckenhuber: I mean compared to what we have now, what is going to change?[17:41:51] <emuckenhuber> hbraun: so instance related operations will go in "server" - but i don't really see a good way how to represent the server configuration and runtime state together[17:42:24] <hbraun> emuckenhuber: i don't think it makes much sense[17:42:42] <hbraun> emuckenhuber: we have server configurations and server instances[17:43:11] <hbraun> emuckenhuber: speaking of which, why can there only be one server instance per config?[17:44:44] <hbraun> emuckenhuber: I get back to you tomorrow[17:44:52] <hbraun> emuckenhuber: got to run[17:44:54] <emuckenhuber> hbraun: well one reason is that you most likely get conflicting ports...[17:44:56] <emuckenhuber> hbraun: ok[17:45:05] *** hbraun has quit IRC[17:47:47] *** torben|call is now known as torben[17:48:07] <maeste> dmlloyd: JBAS-9168[17:48:08] <jbossbot> jira [JBAS-9168] ArjunaTransactionManagerService should be splitted [Open (Unresolved) Task, Major, David Lloyd] https://issues.jboss.org/browse/JBAS-9168[17:48:28] <dmlloyd> don't assign it to me, I can't do it right now :)[17:48:30] *** asaldhan has left #jboss-as7[17:48:46] <maeste> dmlloyd: oki. Leave it unassigned so?[17:48:48] *** jfclere has quit IRC[17:48:53] <dmlloyd> already done[17:49:02] <maeste> k[17:50:53] *** emuckenhuber has quit IRC[17:57:36] *** torben has quit IRC[18:06:16] *** jcosta has quit IRC[18:06:50] <baileyje> Nihility: https://github.com/baileyje/jboss-as/commit/9ef9bcbd3f2b5b9fa84518baae8c90dd445c88cc[18:06:50] <jbossbot> git [jboss-as] 9ef9bcb.. John E. Bailey [JBAS-9143] - Update default module name when no context root is provided by EAR descriptors[18:06:51] <jbossbot> jira [JBAS-9143] Make sure the web context is propagated from the application.xml to the web deployments [Open (Unresolved) Feature Request, Major, John Bailey] https://issues.jboss.org/browse/JBAS-9143[18:07:38] <rmaucher> should be fine :)[18:08:06] <baileyje> rmaucher: Was that to me? If not can you also look at that change? :)[18:09:26] <jbossbot> git [jboss-as] push master e4248d3.. Heiko Braun Fix for JBAS-9162[18:09:26] <jbossbot> jira [JBAS-9162] Serve console from "/console" (w/o index.html) [Open (Unresolved) Feature Request, Major, Heiko Braun] https://issues.jboss.org/browse/JBAS-9162[18:09:27] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/ac96e5d...e4248d3[18:10:15] <rmaucher> baileyje, yes, it's for this commit[18:10:37] <baileyje> thanks for looking at it..[18:11:22] <rmaucher> I was wondering if it would override something set in jboss-web.xml, but it shouldn't so it's good[18:13:02] <baileyje> Yeah. It allows application.xml or jboss-web.xml to have the first and second crack[18:17:47] <pilhuhn> bstansberry http://localhost:9990/domain-api/host/localhost?json.pretty=true hangs for me -- while .../host/local/ works[18:18:05] <pilhuhn> actually all invalid hostnames make the API call hang here. Not for other paths[18:18:17] <pilhuhn> Looks like an issue with name resolution[18:18:44] <bstansberry> can you file a JIRA?[18:18:54] <pilhuhn> yep[18:18:57] <bstansberry> thanks[18:19:17] <bstansberry> that call will go into logic that tries to route it to a remote host controller[18:19:19] <pilhuhn> bstansberry is it on purpose that /server-group/* entries do not list the servers in each group?[18:19:34] *** mmoyses is now known as mmoyses_[18:20:08] <bstansberry> yes and no[18:20:19] <bstansberry> yes, it forces the domain controller to maintain information about all running servers[18:20:23] <bstansberry> well, it could[18:20:50] <bstansberry> it should be possible to work around that[18:21:41] <pilhuhn> only the list of server names - I think the "old api" provided this[18:22:41] <bstansberry> we might be able to do that. I can't promise when though[18:23:10] <bstansberry> some stuff emuckenhuber's doing should help[18:23:49] <pilhuhn> JBAS-9175 filed[18:23:50] <jbossbot> jira [JBAS-9175] Call to /host/<invalid> makes the API call hang [Open (Unresolved) Bug, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9175[18:23:58] <bstansberry> thanks![18:24:21] <pilhuhn> I'll file a FR for the other thingy[18:25:53] <pilhuhn> JBAS-9176[18:25:54] <jbossbot> jira [JBAS-9176] List servers in a servergroup below /server-group/<groupname> [Open (Unresolved) Enhancement, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9176[18:26:01] <pilhuhn> thanks bstansberry - l8er[18:26:09] <bstansberry> bye![18:26:13] *** pilhuhn is now known as pil-dinner[18:30:06] *** frainone_lunch is now known as frainone[18:32:36] *** alesj has quit IRC[18:34:22] <smarlow> I'm trying to debug a testsuite integration test but my breakpoints aren't getting triggered for some reason[18:35:08] *** emuckenhuber has joined #jboss-as7[18:35:08] *** emuckenhuber has joined #jboss-as7[18:35:08] *** ChanServ sets mode: +v emuckenhuber[18:42:52] *** fnasser has quit IRC[18:44:49] *** mmoyses_ is now known as mmoyses[18:49:20] *** fnasser has joined #jboss-as7[18:49:21] *** jclingan has joined #jboss-as7[19:16:36] *** mmoyses has quit IRC[19:16:54] *** jwulf has quit IRC[19:22:50] *** mmoyses has joined #jboss-as7[19:22:51] *** ChanServ sets mode: +v mmoyses[19:26:09] *** anguenot has quit IRC[19:27:47] <Jaikiran> dmlloyd: could you please pull this branch upstream https://github.com/jaikiran/jboss-as/commits/ejb3-dd[19:28:08] <Jaikiran> it consists of a major chunk of ejb-jar.xml processing and merging of annotation + DD for EJBs[19:28:27] <Jaikiran> it's still WIP but a major chunk of it is workin[19:28:29] <Jaikiran> working[19:28:35] <Jaikiran> the existing tests pass[19:28:54] <Jaikiran> i need to add more tests and work on the remaining part of interceptor/resource injections via ejb-jar.xml[19:28:59] <Jaikiran> i'll be doing that tomorrow[19:29:34] *** darranl has quit IRC[19:29:50] <dmlloyd> ok will get to it soon[19:33:47] <Jaikiran> thanks[19:37:43] *** pferraro has joined #jboss-as7[19:37:43] *** ChanServ sets mode: +v pferraro[19:40:22] <baileyje> dmlloyd, Nihility: can I get someone to review and push https://github.com/baileyje/jboss-as/commit/81da06bed566820ccd70f47b84d662ac2be1ed45[19:40:23] <jbossbot> git [jboss-as] 81da06b.. John E. Bailey [JBAS-9167] - Fix EAR level processors that fail when no modules are present[19:40:24] <jbossbot> jira [JBAS-9167] EjbJarDeploymentProcessor will throw a NullPointer if there are no modules defined in an application.xml [Open (Unresolved) Bug, Major, John Bailey] https://issues.jboss.org/browse/JBAS-9167[19:40:54] <Nihility> baileyje: yeah ill add yours together[19:41:01] <baileyje> Nihility: Great. Thanks[19:49:19] <jbossbot> git [jboss-as] push master 82d729c.. Jason T. Greene Fix JAXB[19:49:19] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/e4248d3...82d729c[19:50:21] <smarlow> Nihility: I have more JPA changes (including some fixes that I cherry picked from wolfc). should I squash these changes with the commit that you were going to review? Or submit them separately?[19:50:44] <Nihility> if its two different problems[19:50:48] <Nihility> then two commits is better[19:51:56] <smarlow> okay, I'll keep them separate then. I've been squashing everything down to one commit.[19:52:03] <jamezp> Nihility: If you're taking requests :-) or if there is a preferred way instead of IRC, let me know. https://github.com/jamezp/jboss-as/commit/9cc85ba7cefbaa6d6e07ef4ad617e916163e9789[19:52:03] <jbossbot> git [jboss-as] 9cc85ba.. James Perkins JBAS-8991 Added woodstox modules for JAXP processing. This allowed older JDK's to launch the server.[19:52:04] <jbossbot> jira [JBAS-8991] Cannot start beta1 server on RHEL6 with OpenJDK 1.6 [Open (Unresolved) Bug, Critical, James Perkins] https://issues.jboss.org/browse/JBAS-8991[19:52:30] *** adietisheim has quit IRC[19:53:17] *** rmaucher has quit IRC[19:55:29] *** Jaikiran has quit IRC[19:57:49] *** darranl has joined #jboss-as7[19:57:50] *** darranl has joined #jboss-as7[19:57:50] *** ChanServ sets mode: +v darranl[19:59:46] <Nihility> grr[19:59:51] <Nihility> what happened to our start time?[19:59:57] *** bmachado has joined #jboss-as7[19:59:58] *** ChanServ sets mode: +v bmachado[20:00:25] <bmachado> ping frainone[20:00:46] <frainone> bmachado: pong[20:00:49] <bmachado> frainone, pm[20:02:49] <jbossbot> git [jboss-as] push master 877f075.. John E. Bailey [JBAS-9143] - Update default module name when no context root is provided by EAR descriptors[20:02:51] <jbossbot> jira [JBAS-9143] Make sure the web context is propagated from the application.xml to the web deployments [Open (Unresolved) Feature Request, Major, John Bailey] https://issues.jboss.org/browse/JBAS-9143[20:02:51] <jbossbot> git [jboss-as] push master a5c9cac.. John E. Bailey [JBAS-9167] - Fix EAR level processors that fail when no modules are present[20:02:52] <jbossbot> jira [JBAS-9167] EjbJarDeploymentProcessor will throw a NullPointer if there are no modules defined in an application.xml [Open (Unresolved) Bug, Major, John Bailey] https://issues.jboss.org/browse/JBAS-9167[20:02:52] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/82d729c...a5c9cac[20:03:51] <Nihility> jamezp: why did you delete xalan from the jaxp provider override?[20:04:33] <baileyje> Nihility: thanks[20:04:36] <jamezp> Nihility: I thought, probably misunderstood, we wanted to replace it with woodstox.[20:05:24] <Nihility> jamezp: we are replacing the jdk stax impl, xalan is just the XSLT stuff (unrelated), so it should be additive[20:05:56] <Nihility> jamezp: we are slowly replacing all jaxp impls :)[20:06:24] <jamezp> Nihility: Ah, my mistake. I'll get that fixed. I just need to add the woodstox stuff to the JAXP module as a dependency correct?[20:06:51] <jamezp> Nihility: I guess I'm just trying to make it go faster by breaking things :-D[20:06:57] <Nihility> jamezp: correct this is just a list of everything we override[20:07:01] <Nihility> haha[20:08:16] <jamezp> Perfect, that's simple enough. I'll just get it fixed here soon. Thanks for the info.[20:09:15] <Nihility> somehow we went from 1800ms on my box to around 2400-2500[20:09:21] <Nihility> looks like alot of that is the h2 driver[20:09:51] <Nihility> but disabling that only gets me to 2100[20:10:03] <Nihility> WHO TOOK MY 300ms !!! :)[20:14:07] <wolfc> you could write a git bisect script...[20:14:28] <Nihility> yeah ill do that at some point this week[20:14:53] <Nihility> i was planning on doing it manually though[20:15:00] <Nihility> since times are "fuzzy"[20:16:06] *** opalka has joined #jboss-as7[20:16:06] *** opalka has joined #jboss-as7[20:16:06] *** ChanServ sets mode: +v opalka[20:16:59] <wolfc> isn't there a tool somewhere that analyzes the performance of java code?[20:17:10] <wolfc> it shouldn't be that hard to detect loops and such[20:17:33] <wolfc> and I mean the theoretical performance, not the actual[20:18:55] *** jclingan has quit IRC[20:20:57] *** jfd has quit IRC[20:24:44] *** jfd has joined #jboss-as7[20:24:45] *** jfd has quit IRC[20:24:45] *** jfd has joined #jboss-as7[20:24:45] *** ChanServ sets mode: +v jfd[20:25:45] <jbossbot> git [jboss-as] push master 17e45dd.. Emanuel Muckenhuber [JBAS-9160] add system property operations to server-config and server-group[20:25:46] <jbossbot> jira [JBAS-9160] Provide <add/remove>-system-property operations on server-config and server-group [Open (Unresolved) Feature Request, Major, Emanuel Muckenhuber] https://issues.jboss.org/browse/JBAS-9160[20:25:46] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/a5c9cac...17e45dd[20:25:58] <bstansberry> emuckenhuber: ^^^[20:26:17] <emuckenhuber> thanks![20:27:09] <bstansberry> emuckenhuber: you want me to put https://github.com/bstansberry/jboss-as/commit/6531608837f1e2a5dfd37bc67e6ea5949ba0b83c in upstream?[20:27:10] <jbossbot> git [jboss-as] 6531608.. bstansberry at jboss dot com HostControllerImpl doesn't need to be a ModelController[20:27:18] *** bmachado is now known as bmachado_afk[20:27:21] <bstansberry> now it's on a topic branch[20:27:47] <emuckenhuber> bstansberry: yes, that would make sense[20:28:03] <bstansberry> I suppose the whole topic branch could go in, but I'll hold off on that for a bit[20:28:43] <Nihility> jamezp: just ping me when you are ready and ill merge it[20:29:08] <jamezp> Sure thing. I'm launching IDEA now.[20:33:09] *** bmachado_afk is now known as bmachado[20:33:33] * dmlloyd really wishes that IDEA-59300 would get fixed[20:33:33] <jbossbot> youtrack [IDEA-59300] Code style optimization - wildcards [Submitted, 1, cdr] http://youtrack.jetbrains.net/rest/issue/IDEA-59300[20:36:16] <jbossbot> git [jboss-as] push master 0975737.. bstansberry at jboss dot com HostControllerImpl doesn't need to be a ModelController[20:36:16] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/17e45dd...0975737[20:43:47] *** stalep has quit IRC[20:44:29] *** anguenot has joined #jboss-as7[20:50:38] <jbossbot> git [jboss-as] push master 6a72ca9.. bstansberry at jboss dot com [JBAS-9070] Support for auto-deployment by the deployment scanner[20:50:39] <jbossbot> jira [JBAS-9070] Support for auto-deployment by the deployment scanner [Open (Unresolved) Task, Major, Brian Stansberry] https://issues.jboss.org/browse/JBAS-9070[20:50:39] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/0975737...6a72ca9[20:52:47] <jamezp> Nihility: I think I did this right, the commit I mean https://github.com/jamezp/jboss-as/commit/d95de94a164c26acff5e4177e2b6090bbdc33219[20:52:48] <jbossbot> git [jboss-as] d95de94.. James Perkins JBAS-8991 Added woodstox modules for JAXP processing. This allowed older JDK's to launch the server.[20:52:49] <jbossbot> jira [JBAS-8991] Cannot start beta1 server on RHEL6 with OpenJDK 1.6 [Open (Unresolved) Bug, Critical, James Perkins] https://issues.jboss.org/browse/JBAS-8991[20:54:52] *** stalep has joined #jboss-as7[20:54:54] <smarlow> Nihility: I have some additional JPA changes, including the simple test to ensure that Hibernate can discover the entity classes.[20:55:19] *** mgoldmann has quit IRC[20:56:31] *** stalep has quit IRC[20:58:41] *** stalep has joined #jboss-as7[21:01:04] *** stalep has quit IRC[21:01:20] <Nihility> hmm something strange going on in SLSB land[21:02:01] *** maeste has quit IRC[21:02:28] *** anguenot has quit IRC[21:03:04] *** stalep has joined #jboss-as7[21:08:36] <Nihility> interesting that component deps get processed so lazily[21:08:40] <Nihility> in some cases[21:08:41] *** alesj has joined #jboss-as7[21:15:19] *** emuckenhuber has quit IRC[21:16:08] *** darranl has quit IRC[21:18:54] *** maeste has joined #jboss-as7[21:19:00] *** ChanServ sets mode: +v maeste[21:25:02] <jbossbot> git [jboss-as] push smarlow 0c787f9.. Scott Marlow JBAS-9141 passing Hibernate annotation scanner in via hibernate.ejb.resource_scanner property. close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory). jpa demo change to depend on HibernateAnnotationScanner for reading entity classes. JBAS-9141 m[21:25:03] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[21:25:04] <jbossbot> git [jboss-as] push smarlow 18c1b28.. Scott Marlow add equals and hashCode to SFSBContextHandleImpl[21:25:04] <jbossbot> git [jboss-as] push smarlow d11935e.. Carlo de Wolf EJB 3.1 FR 4.9.2.1 there is no component inheritance, so local business view must be explicit[21:25:04] <jbossbot> git [jboss-as] push smarlow 15c1b32.. Carlo de Wolf Use the correct business interface (see also JBAS-9159)[21:25:04] <jbossbot> jira [JBAS-9159] Setting beanName to an EJB that does not expose the interface leads to hang [Open (Unresolved) Bug, Critical, Unassigned] https://issues.jboss.org/browse/JBAS-9159[21:25:05] <jbossbot> git [jboss-as] push smarlow 753176a.. Carlo de Wolf Fixed the lookups[21:25:05] <jbossbot> git [jboss-as] push smarlow dabe638.. Carlo de Wolf Fix NPE[21:25:05] <jbossbot> git [jboss-as] push smarlow 9d83b7a.. Carlo de Wolf Always have EntityManagerMetadata[21:25:05] <jbossbot> git [jboss-as] push smarlow 183ec6d.. Scott Marlow reformat epcpropagation[21:25:06] <jbossbot> git [jboss-as] push smarlow ccafaff.. Scott Marlow JBAS-8962 missing null check[21:25:06] <jbossbot> jira [JBAS-8962] add XPC support [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-8962[21:25:07] <jbossbot> git [jboss-as] push smarlow URL: http://github.com/jbossas/jboss-as/compare/0000000...ccafaff[21:25:53] <smarlow> Nihility: thank you![21:26:55] <Nihility> strange[21:26:58] <Nihility> that didnt go through[21:27:11] <Nihility> but jbossbot got a notification on it[21:28:04] <jbossbot> git [jboss-as] push master 7e8b812.. Scott Marlow JBAS-9141 passing Hibernate annotation scanner in via hibernate.ejb.resource_scanner property. close reference Annotation Index and temp classloader after calling provider to createContainerEntityManagerFactory (both are only valid during call to createContainerEntityManagerFactory). jpa demo change to depend on HibernateAnnotationScanner for reading entity classes. JBAS-9141 ma[21:28:04] <jbossbot> jira [JBAS-9141] handle JPA 8.2.1.6.1 Annotated Classes in the Root of the Persistence Unit [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-9141[21:28:05] <jbossbot> git [jboss-as] push master eaa91f9.. Scott Marlow add equals and hashCode to SFSBContextHandleImpl[21:28:05] <jbossbot> git [jboss-as] push master 7a7e132.. Carlo de Wolf EJB 3.1 FR 4.9.2.1 there is no component inheritance, so local business view must be explicit[21:28:05] <jbossbot> git [jboss-as] push master 0406841.. Carlo de Wolf Use the correct business interface (see also JBAS-9159)[21:28:06] <jbossbot> jira [JBAS-9159] Setting beanName to an EJB that does not expose the interface leads to hang [Open (Unresolved) Bug, Critical, Unassigned] https://issues.jboss.org/browse/JBAS-9159[21:28:06] <jbossbot> git [jboss-as] push master fea99f7.. Carlo de Wolf Fixed the lookups[21:28:06] <jbossbot> git [jboss-as] push master 53064fe.. Carlo de Wolf Fix NPE[21:28:06] <jbossbot> git [jboss-as] push master 7374e32.. Carlo de Wolf Always have EntityManagerMetadata[21:28:06] <jbossbot> git [jboss-as] push master 28fa5ec.. Scott Marlow reformat epcpropagation[21:28:07] <jbossbot> git [jboss-as] push master 3640b21.. Scott Marlow JBAS-8962 missing null check[21:28:07] <jbossbot> jira [JBAS-8962] add XPC support [Coding In Progress (Unresolved) Sub-task, Major, Scott Marlow] https://issues.jboss.org/browse/JBAS-8962[21:28:08] <jbossbot> git [jboss-as] push master URL: http://github.com/jbossas/jboss-as/compare/6a72ca9...3640b21[21:28:50] <jbossbot> git [jboss-as] push smarlow URL: http://github.com/jbossas/jboss-as/compare/ccafaff...0000000[21:29:09] <Nihility> i had a syntax error[21:29:24] <Nihility> git push branch:(space)master[21:29:32] <Nihility> oops other way around[21:29:39] *** emuckenhuber has joined #jboss-as7[21:29:39] *** emuckenhuber has joined #jboss-as7[21:29:39] *** ChanServ sets mode: +v emuckenhuber[21:29:41] <Nihility> git push branch(space):master[21:29:53] <Nihility> it took that as push branch AND :master[21:33:02] *** asoldano_away is now known as asoldano[21:34:23] *** stalep has quit IRC[21:35:08] *** stalep has joined #jboss-as7[21:41:16] *** jhalliday has quit IRC[21:42:01] *** sannegrinovero has quit IRC[21:46:00] *** jhalliday has joined #jboss-as7[21:51:23] <dmlloyd> heh now you have to delete the smarlow branch you created Nihility :)[21:51:36] <Nihility> already done[21:51:48] <dmlloyd> it's still not as bad as the time I accidentally rewrote all of history[21:52:01] <dmlloyd> good thing git makes it easy to fix :)[21:52:16] <Nihility> yeah it amounted to just extra noise[21:52:27] <Nihility> and a strange branch that was a clone of master[21:52:27] <Nihility> haha[21:55:54] <smarlow> I thought that I could attach intellij to the running testsuite/integration tests and set break points before (by running with -Dmaven.surefire.debug="..."). Does that work for anyone else? Maybe I need to try a different version of the jdk...[21:58:20] *** miclorb_ has joined #jboss-as7[21:58:30] *** bstansberry is now known as bstans_afk[22:00:48] *** bmachado has quit IRC[22:05:28] <baileyje> dmlloyd: Do we need JBAS-9168 for Beta2?[22:05:29] <jbossbot> jira [JBAS-9168] ArjunaTransactionManagerService should be splitted [Open (Unresolved) Task, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9168[22:06:27] <dmlloyd> we need it for maeste's next patch sequence[22:06:37] <dmlloyd> whether we need that for beta2, I'm not sure[22:06:42] <baileyje> Ok. You want me to work on it now?[22:06:46] <dmlloyd> maeste: do you have a JIRA for the work you're doing?[22:07:07] <dmlloyd> baileyje: yeah go ahead, it shouldn't take you too long I would think[22:07:19] <baileyje> Yeah. That is what I was thinking.[22:07:53] <jpederse> dmlloyd: we can disable that part for Beta2[22:08:02] <jpederse> dmlloyd: so not critical[22:09:02] *** frainone is now known as frainone_afk[22:12:40] <dmlloyd> does someone have an example of a messed up deployment handy[22:12:45] <dmlloyd> just testing the error reporting[22:12:48] *** pil-dinner has quit IRC[22:14:14] <dmlloyd> hm looks like I forgot the part where the listener tells the operation that it's complete[22:16:55] *** pmuir has quit IRC[22:23:24] <baileyje> between git and the admin CLI, my brain now assumes everything has command completion and is frustrated to find out they don't.[22:23:38] <maeste> baileyje: I've added a comment to the JBAS-9168 about another service missing. It is a bit more critical then the original one. IOW both are needed, but while we can disable feature of the first one, it's not possible to do the same with the second one[22:23:40] <jbossbot> jira [JBAS-9168] ArjunaTransactionManagerService should be splitted [Open (Unresolved) Task, Major, Unassigned] https://issues.jboss.org/browse/JBAS-9168[22:24:21] <baileyje> maeste: I can add that too. I am testing the first now[22:24:36] <maeste> baileyje: cool[22:30:26] *** emmanuel has quit IRC[22:38:21] <dmlloyd> http://github.com/dmlloyd/jboss-as/commit/68bb357 is as far as I can get for now, until a few more changes to MSC are done.[22:38:22] <jbossbot> git [jboss-as] 68bb357.. David M. Lloyd [JBAS-9010] Proper deployment state tracking (WIP)[22:38:24] <jbossbot> jira [JBAS-9010] Provide more comprehensive error messages for missing dependencies [Open (Unresolved) Task, Blocker, David Lloyd] https://issues.jboss.org/browse/JBAS-9010[22:38:50] <dmlloyd> has anyone looked at smarlow and jaikiran's patches yet?[22:39:16] <wolfc> smarlow's stuff is in[22:40:43] <wolfc> Jaikiran and I are still working on getting a good ejb3 branch that doesn't regress anybody[22:41:55] *** aamonten has quit IRC[22:43:13] <baileyje> dmlloyd hmmm. It seems like the transaction manager service never should have started.[22:45:32] <wolfc> and we are blue again: http://hudson.qa.jboss.com/hudson/view/JBoss%20AS/job/JBoss-AS-7.0.x-testSuite-sun16/[22:46:06] <dmlloyd> baileyje: heh.[22:46:29] * wolfc is of to bed[22:46:46] <baileyje> It has an optional dep on ORB, that is not being fulfilled, and it is not asking for an optional value[22:47:04] <baileyje> and it isn't able to create the RecoveryManger with the current classloader[22:47:18] *** wolfc has quit IRC[22:49:20] *** emuckenhuber has quit IRC[22:51:58] <dmlloyd> well, it was one of the very first subsystems :)[22:52:08] <dmlloyd> it seemed to kinda work at the time![22:55:54] *** jclingan has joined #jboss-as7[23:01:17] *** asoldano has quit IRC[23:04:07] *** emuckenhuber has joined #jboss-as7[23:04:08] *** emuckenhuber has joined #jboss-as7[23:04:08] *** ChanServ sets mode: +v emuckenhuber[23:04:19] <jpederse> bstans_afk, pferraro: domain-communication ?[23:04:43] <baileyje> dmlloyd: I am wrong it was working. Just has some weird static setup stuff I needed to shift around.[23:05:22] <Nihility> i have seen wierd errors[23:05:24] <Nihility> with the TM[23:05:26] <pferraro> jpederse: except that domain != cluster[23:05:43] <jpederse> pferraro: yeah[23:09:17] *** mmoyses has quit IRC[23:09:58] <baileyje> maeste: Where does UserTransactionRegistry come from?[23:10:29] <jpederse> baileyje: the org.jboss.tm package - 2 sec[23:10:36] <baileyje> maeste: I guess I mean, where should it be created?[23:10:44] <baileyje> I see where the class exists.[23:11:46] <jpederse> baileyje: it is a top level service[23:12:03] <baileyje> but currently there isn't one, correct?[23:12:07] <jpederse> correct[23:12:28] <baileyje> Ok. I will just have the txn subsystem add a service for it[23:12:41] <smarlow> pferraro: ha-partition, cluster[23:13:00] <jpederse> baileyje: this is our embedded setup - http://viewvc.jboss.org/cgi-bin/viewvc.cgi/jbossas/projects/jboss-jca/trunk/embedded/src/main/resources/transaction.xml?revision=111023&view=markup - if that helps[23:13:34] <pferraro> smarlow: this is just for the subsystem that's providing the ChannelFactory service[23:15:38] <baileyje> jpederse: Ok. So do we also want the DefaultUserTransactionprovider?[23:16:23] <jpederse> baileyje: no[23:16:36] <baileyje> ok. I will stick to just the registry for now[23:17:12] <jpederse> baileyje: well, that is more a question for jhalliday - I think we should just add the needed services now - and revisit once he has 4.15 ready[23:17:42] <Nihility> ah yes[23:17:44] <Nihility> note to self[23:17:52] <Nihility> need to implement ORB subsystem[23:18:47] *** irooskov has joined #jboss-as7[23:19:41] <Nihility> pferraro: what things do we want to access the group communication channels[23:19:59] <Nihility> pferraro: obviously the clustering subsystems, but what else[23:20:39] <Nihility> pferraro: if for example you dont want any non-clustering code touching your channel, maybe it should just be a part of the clustering subsystem?[23:21:09] <Nihility> pferraro: on the other hand perhaps we want the same mechanism to be useful to users that directly use jgroups[23:21:21] *** frainone_afk has quit IRC[23:22:10] <Nihility> btw i kind of like "group-communication"[23:23:49] <pferraro> Nihility: the channels themselves are effectively private (i.e. specific to a service) - its the channel factory mechanism that is public - and is the subsystem whose name is in question[23:24:08] <Nihility> ah ok[23:24:43] <Nihility> one other interesting thing is[23:24:52] <dmlloyd> does it really need a subsystem? is there really some global config to be done?[23:25:40] <Nihility> if we are worried about it being called jgroups, but yet it has jgroups all over the stack configuration, and jgroups is the api people use have we really gained much[23:26:05] *** jpederse has quit IRC[23:26:19] <dmlloyd> just saying we need to cut down the number of no-config subsystems because in many cases there's really no reason for them[23:26:20] <pferraro> Nihilty: the subsystem is meant to be a registry of configs available to those who want to create a channel[23:27:23] <pferraro> Nihility: and bstans_afk wanted to use a more generic subsystem name than "jgroups" (what I originally proposed)[23:27:39] *** bstans_afk is now known as bstansberry[23:27:47] <bstansberry> hi pferrraro[23:27:54] <pferraro> bstansberry: hey[23:28:00] <Nihility> it certainly makes the config look nicer to someone who doesnt know what jgroups is[23:28:08] * bstansberry scans back through the log[23:28:35] <Nihility> pferraro: are those configs more like templates, or do we actually expect users to alter them[23:29:38] <pferraro> Nihility: for 95% of users yes - though there needs to be a mechanism to alter them (or define them outright)[23:31:37] <bstansberry> it would be nice if users don't have to see an can't alter the standard configs[23:32:04] *** jfd has quit IRC[23:32:15] <bstansberry> but, they need to be able to declare their own configs[23:32:20] *** rawbdor has joined #jboss-as7[23:32:20] *** ChanServ sets mode: +v rawbdor[23:32:39] <pferraro> Nihility: in the initial revision, I'm just defining the configs explicitly - I've created a subsequent jira to provide higher-level configuration - that reference pre-defined stacks[23:33:08] <bstansberry> pferraro -- looking at the patch, none of the stack information from the parser is getting stored in the model -- it just goes straight into the runtime[23:34:35] <pferraro> bstansberry: I see what you mean[23:34:55] <bstansberry> so it will be lost when the config persists following any update anywhere[23:35:36] <bstansberry> should each stack be a resource?[23:36:05] <bstansberry> otherwise you'll have a single attribute in the subsystems "stacks" which will be the mother of all complex attributes[23:36:56] <baileyje> dmlloyd, maeste: https://github.com/baileyje/jboss-as/commit/569c85793cc25b1d6b2d1738f9ca2238b35f6a93[23:36:57] <jbossbot> git [jboss-as] 569c857.. John E. Bailey [JBAS-9168] - Add additional TXN services for RecoveryManager and UserTransactionRegistry[23:36:58] <jbossbot> jira [JBAS-9168] ArjunaTransactionManagerService should be splitted [Open (Unresolved) Task, Major, John Bailey] https://issues.jboss.org/browse/JBAS-9168[23:37:30] *** alesj has quit IRC[23:37:59] <pferraro> bstansberry: yeah - I guess each stack should be a resource, with add/remove operations[23:39:14] *** jwulf has joined #jboss-as7[23:44:13] <bstansberry> on the "jgroups" subsystem name, I don't feel terribly strongly about it[23:44:34] <bstansberry> having suffered for years under the HAPartition abstraction of JGroups which never had a different impl[23:46:03] <bstansberry> I'm more concerned about a slippery slope effect where suddenly everybody uses their impl[23:46:04] *** jclingan has quit IRC[23:46:09] <pferraro> bstansberry: given that the only service of this subsystem is to provide a jgroups channel, "jgroups" seemed like an appropriate subsystem name[23:46:44] <bstansberry> true; we're not handing out some abstraction[23:46:45] <maeste> baileyje: cool! Thanks John[23:46:54] <bstansberry> of a channel[23:47:20] <maeste> baileyje: head to bed now. I'll use it tomorrow morning[23:48:37] <opalka> dmlloyd, bstansberry please review & cherry pick https://github.com/ropalka/jboss-as/commit/a3e54ef7b1df3134049283ef2b97f1ebfa8480b8[23:48:38] <jbossbot> git [jboss-as] a3e54ef.. Richard Opalka refactoring + fixing bug in params processing method (if all 'export', 'services' & 'optional' were specified on manifest dependency, the last one in the param list have been improperly evaluated)[23:48:49] <bstansberry> same applies to infinispan. it doesn't produce a JSR-107 cache or anything[23:49:28] *** maeste has quit IRC[23:50:08] <bstansberry> in a way it's better to call it the infinispan subsystem. if we'd done this in 2006 and called it "distributed-caching" the subsystem would be harder to transition from JBC to infinispan[23:50:50] <pferraro> bstansberry: agreed[23:53:52] <bstansberry> opalka: will push in a sec[23:59:24] <dmlloyd> org.jboss.as.test.embedded.demos.ws.WSTestCase seems to be failing on me