Merge lp://staging/~raghavendra-prabhu/percona-xtradb-cluster/bug-1131102 into lp://staging/percona-xtradb-cluster/percona-xtradb-cluster-5.5

Proposed by Raghavendra D Prabhu
Status: Merged
Approved by: Vadim Tkachenko
Approved revision: no longer in the source branch.
Merged at revision: 387
Proposed branch: lp://staging/~raghavendra-prabhu/percona-xtradb-cluster/bug-1131102
Merge into: lp://staging/percona-xtradb-cluster/percona-xtradb-cluster-5.5
Diff against target: 220 lines (+198/-0)
4 files modified
policy/apparmor/usr.sbin.mysqld (+116/-0)
policy/apparmor/usr.sbin.mysqld.local (+2/-0)
policy/selinux/percona-xtradb-cluster.fc (+7/-0)
policy/selinux/percona-xtradb-cluster.te (+73/-0)
To merge this branch: bzr merge lp://staging/~raghavendra-prabhu/percona-xtradb-cluster/bug-1131102
Reviewer Review Type Date Requested Status
Alexey Kopytov (community) Needs Information
Vadim Tkachenko Approve
Review via email: mp+152455@code.staging.launchpad.net

Description of the change

Selinux and Apparmor policies for PXC.

To post a comment you must log in.
Revision history for this message
Vadim Tkachenko (vadim-tk) :
review: Approve
Revision history for this message
Alexey Kopytov (akopytov) wrote :

Shouldn't this be MPed for Percona Server first, and then merged to PXC naturally?

Also, it looks like this fix is missing the packaging part, i.e. it just adds a file, but it will not be used and installed by packages?

review: Needs Information
Revision history for this message
Raghavendra D Prabhu (raghavendra-prabhu) wrote :

Yes, these are just the files. The packaging part needs to be done, preferrably by Ignacio or BAlexey. However, I can also do this. Packaging initially can be only distributing the '.pp'/apparmor profile file(s) as part of package as done by the upstream

I have also submitted MP for both PXC and PS.

Revision history for this message
Alexey Kopytov (akopytov) wrote :

Thanks for clarifications. Can you also create a separate packaging bug?

Revision history for this message
Raghavendra D Prabhu (raghavendra-prabhu) wrote :

Ack. Will report this as a separate bug.

Revision history for this message
Raghavendra D Prabhu (raghavendra-prabhu) wrote :

Created lp:1159765 for that.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches