9:30 a.m.
We should 'import gtk', not 'import pygtk', fixes bugs when the error
paths try to use gtk.messageDialog():
[root@ssdandy tuna]# tuna
Traceback (most recent call last):
File "/home/acme/git/tuna/tuna-cmd.py", line 654, in <module>
main()
File "/home/acme/git/tuna/tuna-cmd.py", line 648, in main
app = tuna_gui.main_gui(kthreads, uthreads, cpus_filtered)
File "/home/acme/git/tuna/tuna/tuna_gui.py", line 64, in __init__
self.profileview.init_default_file()
File "/home/acme/git/tuna/tuna/gui/profileview.py", line 141, in
init_default_file
not self.config.load(self.config.config['lastfile']):
File "/home/acme/git/tuna/tuna/config.py", line 48, in load
if(self.tuned2Tuna(profileName) < 0):
File "/home/acme/git/tuna/tuna/config.py", line 66, in tuned2Tuna
dialog = gtk.MessageDialog(None, 0, gtk.MESSAGE_ERROR,\
NameError: global name 'gtk' is not defined
[root@ssdandy tuna]#
Signed-off-by: Arnaldo Carvalho de Melo <acme(a)redhat.com>
---
tuna/config.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tuna/config.py b/tuna/config.py
index 77cc4d800c03..7e19a28f967f 100644
--- a/tuna/config.py
+++ b/tuna/config.py
@@ -1,5 +1,5 @@
import io, os, re, fnmatch
-import sys, pygtk
+import sys, gtk
import codecs, ConfigParser
from time import localtime, strftime
from subprocess import Popen, PIPE, STDOUT, call
--
1.8.3.1
9:46 a.m.
Assuming that this is untested though?
----- Original Message -----
We should 'import gtk', not 'import pygtk', fixes
bugs when the error
paths try to use gtk.messageDialog():
[root@ssdandy tuna]# tuna
Traceback (most recent call last):
File "/home/acme/git/tuna/tuna-cmd.py", line 654, in <module>
main()
File "/home/acme/git/tuna/tuna-cmd.py", line 648, in main
app = tuna_gui.main_gui(kthreads, uthreads, cpus_filtered)
File "/home/acme/git/tuna/tuna/tuna_gui.py", line 64, in __init__
self.profileview.init_default_file()
File "/home/acme/git/tuna/tuna/gui/profileview.py", line 141, in
init_default_file
not self.config.load(self.config.config['lastfile']):
File "/home/acme/git/tuna/tuna/config.py", line 48, in load
if(self.tuned2Tuna(profileName) < 0):
File "/home/acme/git/tuna/tuna/config.py", line 66, in tuned2Tuna
dialog = gtk.MessageDialog(None, 0, gtk.MESSAGE_ERROR,\
NameError: global name 'gtk' is not defined
[root@ssdandy tuna]#
Signed-off-by: Arnaldo Carvalho de Melo <acme(a)redhat.com>
---
tuna/config.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tuna/config.py b/tuna/config.py
index 77cc4d800c03..7e19a28f967f 100644
--- a/tuna/config.py
+++ b/tuna/config.py
@@ -1,5 +1,5 @@
import io, os, re, fnmatch
-import sys, pygtk
+import sys, gtk
import codecs, ConfigParser
from time import localtime, strftime
from subprocess import Popen, PIPE, STDOUT, call
--
1.8.3.1
10:14 a.m.
Em Wed, Apr 29, 2015 at 10:46:36AM -0400, John Kacur escreveu:
Assuming that this is untested though?
Tested, on a RHEL7 system, and that leads to another problem, where the
fact that there is nothing in /etc/tuna/ generates a messageDialog()
saying that the config file is "corrupted", which is not the case, it
should just start with default values, etc.
But that is for another patch. Trying to figure out how to do that, as
this was done by somobody else, using some ConfigParser() python classes
I am not familiar with.
To repro the problem, just run tuna, as root, on a system with nothing
on /etc/tuna/, it should produce the Dialog, and will freeze the screen,
with nothing on the tuna panes, till the user presses Ok.
- Arnaldo
----- Original Message -----
> We should 'import gtk', not 'import pygtk', fixes bugs when the
error
> paths try to use gtk.messageDialog():
>
> [root@ssdandy tuna]# tuna
> Traceback (most recent call last):
> File "/home/acme/git/tuna/tuna-cmd.py", line 654, in <module>
> main()
> File "/home/acme/git/tuna/tuna-cmd.py", line 648, in main
> app = tuna_gui.main_gui(kthreads, uthreads, cpus_filtered)
> File "/home/acme/git/tuna/tuna/tuna_gui.py", line 64, in __init__
> self.profileview.init_default_file()
> File "/home/acme/git/tuna/tuna/gui/profileview.py", line 141, in
> init_default_file
> not self.config.load(self.config.config['lastfile']):
> File "/home/acme/git/tuna/tuna/config.py", line 48, in load
> if(self.tuned2Tuna(profileName) < 0):
> File "/home/acme/git/tuna/tuna/config.py", line 66, in tuned2Tuna
> dialog = gtk.MessageDialog(None, 0, gtk.MESSAGE_ERROR,\
> NameError: global name 'gtk' is not defined
> [root@ssdandy tuna]#
>
> Signed-off-by: Arnaldo Carvalho de Melo <acme(a)redhat.com>
> ---
> tuna/config.py | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tuna/config.py b/tuna/config.py
> index 77cc4d800c03..7e19a28f967f 100644
> --- a/tuna/config.py
> +++ b/tuna/config.py
> @@ -1,5 +1,5 @@
> import io, os, re, fnmatch
> -import sys, pygtk
> +import sys, gtk
> import codecs, ConfigParser
> from time import localtime, strftime
> from subprocess import Popen, PIPE, STDOUT, call
> --
> 1.8.3.1
>
>
10:20 a.m.
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
On 04/29/2015 10:14 AM, Arnaldo Carvalho de Melo wrote:
Em Wed, Apr 29, 2015 at 10:46:36AM -0400, John Kacur escreveu:
> Assuming that this is untested though?
Tested, on a RHEL7 system, and that leads to another problem, where the
fact that there is nothing in /etc/tuna/ generates a messageDialog()
saying that the config file is "corrupted", which is not the case, it
should just start with default values, etc.
But that is for another patch. Trying to figure out how to do that, as
this was done by somobody else, using some ConfigParser() python classes
I am not familiar with.
To repro the problem, just run tuna, as root, on a system with nothing on
/etc/tuna/, it should produce the Dialog, and will freeze the screen,
with nothing on the tuna panes, till the user presses Ok.
- Arnaldo
I don't know why that configuration stuff was tacked onto tuna. It's the
reason tuna has to run as root now. I think it would be better if we just
had a button that launched a separate configuration app.
- --Guy
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2
iEYEARECAAYFAlVA9roACgkQ0Bme0QyNhPTwMwCeIUnDBPcSFKahaikpPt7haQHo
+rUAoJBnHMnGHeG5gsJL0w/IW4U02qBi
=ajnW
-----END PGP SIGNATURE-----
11:16 a.m.
Em Wed, Apr 29, 2015 at 10:20:26AM -0500, Guy Streeter escreveu:
On 04/29/2015 10:14 AM, Arnaldo Carvalho de Melo wrote:
> Em Wed, Apr 29, 2015 at 10:46:36AM -0400, John Kacur escreveu:
>> Assuming that this is untested though?
> Tested, on a RHEL7 system, and that leads to another problem,
where the
> fact that there is nothing in /etc/tuna/ generates a messageDialog()
> saying that the config file is "corrupted", which is not the case, it
> should just start with default values, etc.
> But that is for another patch. Trying to figure out how to do
that, as
> this was done by somobody else, using some ConfigParser() python classes
> I am not familiar with.
> To repro the problem, just run tuna, as root, on a system with
nothing on
> /etc/tuna/, it should produce the Dialog, and will freeze the screen,
> with nothing on the tuna panes, till the user presses Ok.
I don't know why that configuration stuff was tacked onto tuna.
It's the
reason tuna has to run as root now. I think it would be better if we just
had a button that launched a separate configuration app.
Was part of a integration with tuned, approved/requested by management
at Red Hat.
And yes, it is sad that the GUI now goes thru that sequence that ends up
asking for 'root' priviledges to be granted.
On the command line, tho, it works as before, subject to the permissions
enforced by the kernel to /proc, ethtool, schedutils, etc.
[acme@ssdandy ~]$ tuna -t docker -CP
thread ctxt_switches
pid SCHED_ rtpri affinity voluntary nonvoluntary cmd
1101 OTHER 0 0xff 634 140 docker
1538 OTHER 0 0xff 30 0 docker
1125 OTHER 0 0xff 7215 4 docker
1126 OTHER 0 0xff 3 3 docker
2152 OTHER 0 0xff 7241 1 docker
1258 OTHER 0 0xff 4 0 docker
1105 OTHER 0 0xff 30227 2 docker
1170 OTHER 0 0xff 14205 7 docker
[acme@ssdandy ~]$
I think this should be changed so that it works as before, i.e. as
non-root user one can use the GUI, and whatever operations that requires
root priviledges should be presented only if those priviledges are
available.
- Arnaldo
3284
days inactive
3284
days old
tuna-devel@lists.fedorahosted.org
4 comments
3 participants
participants (3)
-
Arnaldo Carvalho de Melo -
Guy Streeter -
John Kacur