On 1/10/2012 11:30 AM, Kabir Khan wrote:
That worked for me so I pushed that change. It looks great!
The one thing which is a bit strange is the ability to add a child resource. e.g. the add
operation should really by on the parent of the resource being targeted. For example, the
ee extension has the add operation in its list, which gives the op:
"/extension=org.jboss.as.ee/:add(module=sasasa)". Which is a bit pointless since
it already exists, and there is no way to add a new extension. I had similar issues when
writing the management jmx facade, see ChildAddOperationFinder and its callers.
Yea,
I agree that it's a bit strange. But I'm not sure if this is a
problem that can be solved by the tool. What I have seen is that often
you need to give an address that doesn't exist yet, and append :add to
it. For instance, to add a new logger you say
/subsystem=logging/logger=foo/:add
But if you look at subsystem=logging you see that there are many kinds
of entities that might need to be created (loggers, handlers, etc.).
That makes it hard for the tool to guess what you are trying to add.
Beyond that, there is no way for the tool to know what params are
required for a new entity. :read-operation-description(name=add)
doesn't usually help in this case.
IMO, each management handler should fully implement the add operation,
specify the expected params, and never rely on default behavior. But
that would be a lot of work to go back and fix them all.
Anyone else have comments/suggestions?
On 10 Jan 2012, at 16:16, Kabir Khan wrote:
> I've been out so I didn't see this until my previous reply, I'll try it
now
>
> On 10 Jan 2012, at 14:11, ssilvert(a)redhat.com wrote:
>
>> Kabir (or someone with OS X),
>>
>> Can you give this a try?
>>
>>
https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1...
>>
>>
>> On 1/10/2012 8:28 AM, ssilvert(a)redhat.com wrote:
>>> On 1/10/2012 8:06 AM, Kabir Khan wrote:
>>>> This has been merged. However, I've not been able to play with it. I
added
https://github.com/jbossas/jboss-as/commit/999beb90268d94e4e6547986b7277f...
to get around this problem
>>> Thanks. I don't have an OS X box, so I don't have any way to test.
>>>> The next thing is that right-clicking does not seem to work on OS X so I
don't see the menu
>>> Looks like this is a well known issue with OS X and the Swing
>>> isPopupTrigger() method. What I'm reading says that you need to hold
>>> down the control key while pressing button one.
>>>
http://stackoverflow.com/questions/5736872/java-popup-trigger-in-linux
>>>
>>> I'll see if there is a workaround. I'll probably also add a regular
>>> menu at the top so we don't have to rely on right-click.
>>>> On 10 Jan 2012, at 12:20, ssilvert(a)redhat.com wrote:
>>>>
>>>>> On 1/10/2012 4:47 AM, Darran Lofthouse wrote:
>>>>>> Do we want to verify the various security features already in the
CLI
>>>>>> are supported before it is pulled or follow up once it is in?
>>>>>>
>>>>>> Regards,
>>>>>> Darran Lofthouse.
>>>>> The GUI is built on top of the CLI. So it's running the same
>>>>> authentication/authorization. The only difference is that it brings
up
>>>>> a GUI instead of a command line.
>>>>>
>>>>> BTW, I have to give props to Alexey on this. His CLI code is
extremely
>>>>> well organized and easy to extend.
>>>>>> On 01/09/2012 09:42 PM, Jason T. Greene wrote:
>>>>>>> You have to merge his pull to get it
>>>>>>>
>>>>>>> On 1/9/12 3:41 PM, Kabir Khan wrote:
>>>>>>>> Does it need to be enabled somehow?
>>>>>>>>
>>>>>>>> [~/sourcecontrol/jboss-as7/git/jboss-as]
>>>>>>>>
$./build/target/jboss-as-7.1.0.Final-SNAPSHOT/bin/jboss-admin.sh --gui --connect
>>>>>>>> '--gui' is not a valid operation name.
>>>>>>>>
>>>>>>>> [~/sourcecontrol/jboss-as7/git/jboss-as]
>>>>>>>>
$./build/target/jboss-as-7.1.0.Final-SNAPSHOT/bin/jboss-admin.sh --connect --gui
>>>>>>>> '--gui' is not a valid operation name.
>>>>>>>>
>>>>>>>> On 9 Jan 2012, at 19:33, Jason T. Greene wrote:
>>>>>>>>
>>>>>>>>> Just saw Stan did this:
>>>>>>>>>
http://community.jboss.org/docs/DOC-17457
>>>>>>>>>
>>>>>>>>> I haven't played with it yet, but it looks
awesome!
>>>>>>>>>
>>>>>>>>> IMO this is exactly what we need for an
"advanced" mode in the console
>>>>>>>>> (post 7.1 obviously)
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Jason T. Greene
>>>>>>>>> JBoss AS Lead / EAP Platform Architect
>>>>>>>>> JBoss, a division of Red Hat
>>>>>>>>> _______________________________________________
>>>>>>>>> jboss-as7-dev mailing list
>>>>>>>>> jboss-as7-dev(a)lists.jboss.org
>>>>>>>>>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>> _______________________________________________
>>>>>> jboss-as7-dev mailing list
>>>>>> jboss-as7-dev(a)lists.jboss.org
>>>>>>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>> _______________________________________________
>>>>> jboss-as7-dev mailing list
>>>>> jboss-as7-dev(a)lists.jboss.org
>>>>>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> jboss-as7-dev(a)lists.jboss.org
>>>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>> _______________________________________________
>> jboss-as7-dev mailing list
>> jboss-as7-dev(a)lists.jboss.org
>>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> jboss-as7-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev