Wow! CLI GUI

classic Classic list List threaded Threaded
37 messages Options
12
Reply | Threaded
Open this post in threaded view
|

Wow! CLI GUI

jtgreene
Administrator
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
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan
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
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

jtgreene
Administrator
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
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>


--
Jason T. Greene
JBoss AS Lead / EAP Platform Architect
JBoss, a division of Red Hat
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Darran Lofthouse
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.



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
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>
>
>
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Stan Silvert
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
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Darran Lofthouse


On 01/10/2012 12:20 PM, [hidden email] 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.

Where do the username / password prompts or the SSL certificate prompts
currently show up?  Is this within the GUI or still on the 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
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan
In reply to this post by Stan Silvert
This has been merged. However, I've not been able to play with it. I added https://github.com/jbossas/jboss-as/commit/999beb90268d94e4e6547986b7277f166b1c2f69 to get around this problem

[~/sourcecontrol/jboss-clean-as7/git/jboss-as]
$./build/target/jboss-as-7.1.0.Final-SNAPSHOT/bin/jboss-admin.sh --connect --gui
java.lang.Error: Cannot load com.apple.laf.AquaLookAndFeel
        at javax.swing.UIManager.initializeDefaultLAF(UIManager.java:1334)
        at javax.swing.UIManager.initialize(UIManager.java:1422)
        at javax.swing.UIManager.maybeInitialize(UIManager.java:1410)
        at javax.swing.UIManager.getUI(UIManager.java:992)
        at javax.swing.text.JTextComponent.updateUI(JTextComponent.java:332)
        at javax.swing.text.JTextComponent.<init>(JTextComponent.java:306)
        at javax.swing.JTextField.<init>(JTextField.java:212)
        at javax.swing.JTextField.<init>(JTextField.java:153)
        at org.jboss.as.cli.gui.GuiMain.<clinit>(GuiMain.java:48)
        at org.jboss.as.cli.impl.CliLauncher.processGui(CliLauncher.java:210)
        at org.jboss.as.cli.impl.CliLauncher.main(CliLauncher.java:176)
        at org.jboss.as.cli.CommandLineMain.main(CommandLineMain.java:34)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
        at java.lang.reflect.Method.invoke(Method.java:597)
        at org.jboss.modules.Module.run(Module.java:248)
        at org.jboss.modules.Main.main(Main.java:313)
Caused by: java.lang.ClassNotFoundException: com.apple.laf.AquaLookAndFeel from [Module "org.jboss.as.cli:main" from local module loader @72e6f7d2 (roots: /Users/kabirkhan/sourcecontrol/jboss-clean-as7/git/jboss-as/build/target/jboss-as-7.1.0.Final-SNAPSHOT/modules)]
        at org.jboss.modules.ModuleClassLoader.findClass(ModuleClassLoader.java:190)
        at org.jboss.modules.ConcurrentClassLoader.performLoadClassUnchecked(ConcurrentClassLoader.java:468)
        at org.jboss.modules.ConcurrentClassLoader.performLoadClassChecked(ConcurrentClassLoader.java:456)
        at org.jboss.modules.ConcurrentClassLoader.performLoadClassChecked(ConcurrentClassLoader.java:423)
        at org.jboss.modules.ConcurrentClassLoader.performLoadClass(ConcurrentClassLoader.java:398)
        at org.jboss.modules.ConcurrentClassLoader.loadClass(ConcurrentClassLoader.java:120)
        at java.lang.Class.forName0(Native Method)
        at java.lang.Class.forName(Class.java:247)
        at javax.swing.SwingUtilities.loadSystemClass(SwingUtilities.java:1856)
        at javax.swing.UIManager.setLookAndFeel(UIManager.java:563)
        at javax.swing.UIManager.initializeDefaultLAF(UIManager.java:1329)
        ... 17 more

The next thing is that right-clicking does not seem to work on OS X so I don't see the menu

On 10 Jan 2012, at 12:20, [hidden email] 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
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Stan Silvert
In reply to this post by Darran Lofthouse
On 1/10/2012 7:43 AM, Darran Lofthouse wrote:

>
> On 01/10/2012 12:20 PM, [hidden email] 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.
> Where do the username / password prompts or the SSL certificate prompts
> currently show up?  Is this within the GUI or still on the command line?
Right now you have to connect from the command line when you start
jboss-admin.  You have to at least pass the --connect parameter.

>> 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
>>>>>> [hidden email]
>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Stan Silvert
In reply to this post by kkhan
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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>> [hidden email]
>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Darran Lofthouse
In reply to this post by Stan Silvert
Ok thanks, I will get an issue raised so I can follow up on the security
side, there are currently some command line based prompts that should
possibly be made gui compatible so users don't default to bad practices.

Regards,
Darran Lofthouse.


On 01/10/2012 01:14 PM, [hidden email] wrote:

> On 1/10/2012 7:43 AM, Darran Lofthouse wrote:
>>
>> On 01/10/2012 12:20 PM, [hidden email] 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.
>> Where do the username / password prompts or the SSL certificate prompts
>> currently show up?  Is this within the GUI or still on the command line?
> Right now you have to connect from the command line when you start
> jboss-admin.  You have to at least pass the --connect parameter.
>>> 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
>>>>>>> [hidden email]
>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Stan Silvert
In reply to this post by Stan Silvert
Kabir (or someone with OS X),

Can you give this a try?

https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8


On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>> [hidden email]
>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Alexey Loubyansky
In reply to this post by jtgreene
Fantastic really! And a complete surprise, to me at least. Stan didn't
ask (for) anything. Just did it :)

Thanks, Stan!

Alexey

On 01/09/2012 08:33 PM, 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)
>
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Brian Stansberry
+1. Great job, Stan!

On 1/10/12 8:26 AM, Alexey Loubyansky wrote:

> Fantastic really! And a complete surprise, to me at least. Stan didn't
> ask (for) anything. Just did it :)
>
> Thanks, Stan!
>
> Alexey
>
> On 01/09/2012 08:33 PM, 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)
>>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


--
Brian Stansberry
Principal Software Engineer
JBoss by Red Hat
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan
In reply to this post by Stan Silvert

On 10 Jan 2012, at 13:28, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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
Nope, does not work.
Someone else mentions you might have to check if the control key is pressed:
http://stackoverflow.com/questions/2972512/how-to-detect-right-click-event-for-mac-os
>
> 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, [hidden email] 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
>>>>>>> [hidden email]
>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan
In reply to this post by Stan Silvert
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, [hidden email] wrote:

> Kabir (or someone with OS X),
>
> Can you give this a try?
>
> https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8
>
>
> On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>>> [hidden email]
>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>> _______________________________________________
>>>>> jboss-as7-dev mailing list
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan
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.

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, [hidden email] wrote:
>
>> Kabir (or someone with OS X),
>>
>> Can you give this a try?
>>
>> https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8
>>
>>
>> On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>>>> [hidden email]
>>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>> _______________________________________________
>>>>>> jboss-as7-dev mailing list
>>>>>> [hidden email]
>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>> _______________________________________________
>>>>> jboss-as7-dev mailing list
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Stan Silvert
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, [hidden email] wrote:
>>
>>> Kabir (or someone with OS X),
>>>
>>> Can you give this a try?
>>>
>>> https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8
>>>
>>>
>>> On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>>>>> [hidden email]
>>>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>>> _______________________________________________
>>>>>>> jboss-as7-dev mailing list
>>>>>>> [hidden email]
>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>> _______________________________________________
>>>>>> jboss-as7-dev mailing list
>>>>>> [hidden email]
>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev

_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan

On 10 Jan 2012, at 18:29, [hidden email] wrote:

> 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.
I got around this in the jmx facade. Connect to the running process with jconsole, and go to the jboss.as:management=server mbean (appears as simply 'server' in the list. This corresponds to the root resource. In the operations it has an addXXX operation for each child type. e.g.

addExtension -> /extension=*:add
addSubsystemEjb3 -> /subsystem=ejb3:add
addSystemProperty -> /system-property=*:add
addSubsystemJaxrs -> /subsystem=jaxrs: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.

I'm not 100% sure what you mean here so I might be misunderstanding what you're saying. The add handlers are different for different places e.g.:

/system-property=*:read-operation-description(name=add)
/interface=*:read-operation-description(name=add)
/subsystem=jmx:read-operation-description(name=add)
/subsystem=ee:read-operation-description(name=add)

all return different descriptions. (Some have an add operation for '*' meaning all children take the same parameters, others one for each child like the subsystems)

>
>
> 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, [hidden email] wrote:
>>>
>>>> Kabir (or someone with OS X),
>>>>
>>>> Can you give this a try?
>>>>
>>>> https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8
>>>>
>>>>
>>>> On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>>>>>> [hidden email]
>>>>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>>>> _______________________________________________
>>>>>>>> jboss-as7-dev mailing list
>>>>>>>> [hidden email]
>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>>> _______________________________________________
>>>>>>> jboss-as7-dev mailing list
>>>>>>> [hidden email]
>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>> _______________________________________________
>>>>> jboss-as7-dev mailing list
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>
>>> _______________________________________________
>>> jboss-as7-dev mailing list
>>> [hidden email]
>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>
> _______________________________________________
> jboss-as7-dev mailing list
> [hidden email]
> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

kkhan

On 10 Jan 2012, at 20:51, Kabir Khan wrote:

>
> On 10 Jan 2012, at 18:29, [hidden email] wrote:
>
>> 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.
> I got around this in the jmx facade. Connect to the running process with jconsole, and go to the jboss.as:management=server mbean (appears as simply 'server' in the list. This corresponds to the root resource. In the operations it has an addXXX operation for each child type. e.g.
>
> addExtension -> /extension=*:add
> addSubsystemEjb3 -> /subsystem=ejb3:add
> addSystemProperty -> /system-property=*:add
> addSubsystemJaxrs -> /subsystem=jaxrs:add
Or to use your logging subsystem example, go to jboss.as:subsystem=logging (appears as just 'logging' in jconsole), and you will see addSizeRotatingFileHandler, addPeriodicRotatingFieldHandler, addLogger etc.

>
>
>> 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.
>
> I'm not 100% sure what you mean here so I might be misunderstanding what you're saying. The add handlers are different for different places e.g.:
>
> /system-property=*:read-operation-description(name=add)
> /interface=*:read-operation-description(name=add)
> /subsystem=jmx:read-operation-description(name=add)
> /subsystem=ee:read-operation-description(name=add)
>
> all return different descriptions. (Some have an add operation for '*' meaning all children take the same parameters, others one for each child like the subsystems)
>
>>
>>
>> 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, [hidden email] wrote:
>>>>
>>>>> Kabir (or someone with OS X),
>>>>>
>>>>> Can you give this a try?
>>>>>
>>>>> https://github.com/ssilvert/jboss-as/commit/9f90081c86c54278ec3be444f9fb1906051653e8
>>>>>
>>>>>
>>>>> On 1/10/2012 8:28 AM, [hidden email] 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/999beb90268d94e4e6547986b7277f166b1c2f69 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, [hidden email] 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
>>>>>>>>>>>> [hidden email]
>>>>>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>>>>> _______________________________________________
>>>>>>>>> jboss-as7-dev mailing list
>>>>>>>>> [hidden email]
>>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>>>> _______________________________________________
>>>>>>>> jboss-as7-dev mailing list
>>>>>>>> [hidden email]
>>>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>>> _______________________________________________
>>>>>> jboss-as7-dev mailing list
>>>>>> [hidden email]
>>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>> _______________________________________________
>>>>> jboss-as7-dev mailing list
>>>>> [hidden email]
>>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>>>
>>>> _______________________________________________
>>>> jboss-as7-dev mailing list
>>>> [hidden email]
>>>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>>
>> _______________________________________________
>> jboss-as7-dev mailing list
>> [hidden email]
>> https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
>


_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
Reply | Threaded
Open this post in threaded view
|

Re: Wow! CLI GUI

Alexey Loubyansky
In reply to this post by Stan Silvert
On 01/10/2012 07:29 PM, [hidden email] wrote:

> 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.

But it is supposed to. This is why it is there in the first place
(otherwise it's completely useless), if it's not working as it should,
it's a bug (a jira issue should be raised against that component).

> 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.

And not only the 'add' operation. All of the operations, attributes and
nodes must be properly described in the model.

> Anyone else have comments/suggestions?

Management model description is what the tools should rely on. Lets fix
bugs, not find ways to workaround them.

Alexey
_______________________________________________
jboss-as7-dev mailing list
[hidden email]
https://lists.jboss.org/mailman/listinfo/jboss-as7-dev
12