Red Hat Bugzilla – Bug 802381
Port Printers panel to FirewallD1 API
Last modified: 2013-07-04 17:23:14 EDT
Description of problem:
Firewalld has a new API since version 2.0. control-center needs to check whether there is this new version of firewalld available or the old one and run appropriate functions to enable required ports for printers detection.
Version-Release number of selected component (if applicable):
Steps to Reproduce:
1. service firewalld start
2. try to add new printer with control-center
The dialog says that FirewallD is not running.
The dialog doesn't inform you about FirewallD and it just enables required services for given timeout (5 minutes). It also has to enable used service permanently after new printer was added.
You have to get active zones with function getActiveZones and enable required services for all of them. You have to also disable them after you don't need them (don't wait for the timeout, just disable it when you don't need them - the timeout is just a fail-safe solution), so a list of zones on which you enabled those services is needed (some of those zones could have the service enabled before, so don't disable the service for them).
We need the same for gnome-settings-daemon.
> We need the same for gnome-settings-daemon.
This is not true.
I won't be able to finish this this week.
Created attachment 569376 [details]
This need to change methods' names (and their path to .zone*) and incorporate processing of zones (getActiveZones - output is dictionary of zones where you have list of interfaces for each active zone. Just get names of zones. Don't care about interfaces.).
enableService -> addService
disableService -> removeService
As an initial response, I'd be unlikely to vote for this as a Beta blocker with the current description: printing functionality is not considered to block Beta (or, for that matter, Final).
Note that the feature and release validation processes are explicitly considered to be separate: a feature being incomplete does not, in and of itself, constitute a Beta blocker.
Fedora Bugzappers volunteer triage team
I've prepared a patch for this. It is attached to https://bugzilla.gnome.org/show_bug.cgi?id=672682.
control-center shows the message about missing firewalld even if firewalld-2.* is installed and running without this patch.
Marek, we should get this landed soon, I guess
I've committed this to Fedora 17 repository but not filled an update for it. Do you think that it should go to updates now or can it wait until next week when the gnome-control-center-3.4.1 is planned?
*** Bug 812824 has been marked as a duplicate of this bug. ***
Discussed at 2012-04-20 blocker review meeting - http://meetbot.fedoraproject.org/fedora-bugzappers/2012-04-20/fedora-bugzappers.2012-04-20-17.01.log.txt . Rejected as a blocker as it doesn't hit the criteria (we don't cover printing) and can be fixed with an update.
This message is a reminder that Fedora 17 is nearing its end of life.
Approximately 4 (four) weeks from now Fedora will stop maintaining
and issuing updates for Fedora 17. It is Fedora's policy to close all
bug reports from releases that are no longer maintained. At that time
this bug will be closed as WONTFIX if it remains open with a Fedora
'version' of '17'.
Package Maintainer: If you wish for this bug to remain open because you
plan to fix it in a currently maintained version, simply change the 'version'
to a later Fedora version prior to Fedora 17's end of life.
Bug Reporter: Thank you for reporting this issue and we are sorry that
we may not be able to fix it before Fedora 17 is end of life. If you
would still like to see this bug fixed and are able to reproduce it
against a later version of Fedora, you are encouraged change the
'version' to a later Fedora version prior to Fedora 17's end of life.
Although we aim to fix as many bugs as possible during every release's
lifetime, sometimes those efforts are overtaken by events. Often a
more recent Fedora release includes newer upstream software that fixes
bugs or makes them obsolete.
The current stable version of control-center for F17 is 3.4.3-1:
so I'm pretty sure we can consider this fixed.