Note: This bug is displayed in read-only format because the product is no longer active in Red Hat Bugzilla.

Bug 89882

Summary:	p2fs fixes
Product:	[Retired] Red Hat Enterprise CMS	Reporter:	Richard Li <richardl>
Component:	other	Assignee:	Justin Ross <jross>
Status:	CLOSED RAWHIDE	QA Contact:	Jon Orris <jorris>
Severity:	medium	Docs Contact:
Priority:	medium
Version:	nightly	CC:	lutter
Target Milestone:	---
Target Release:	---
Hardware:	All
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2003-06-02 21:39:52 UTC	Type:	---
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:
Bug Blocks:	88926

Description Richard Li 2003-04-29 12:18:31 UTC

We need to identify test cases and thoroughly test p2fs. We should also identify
any and all architectural changes we want to make.

Also, please take a look at 29182 to decide if it should be merged.

Comment 1 Richard Li 2003-04-29 12:23:08 UTC

Also please decide whether or not merging 29180 is appropriate.

Comment 2 Richard Li 2003-04-29 12:23:32 UTC

And 29056.

Comment 3 Richard Li 2003-05-29 13:36:30 UTC

David, can you comment on the three changelists above (and any others that may
be relevant?). Thanks.

Comment 4 David Lutterkort 2003-05-29 18:00:33 UTC

@29182: Having PFS responsible for deleting empty live bundles and folders is
really ugly. I had to put this back into WCMS since there is some pretty screwed
up, DP specific code that depends on it (the DP-specific move functionality).
For core CMS, it would be much cleaner if live bundles and folders were deleted
when the last instance/item in them is unpublished. PFS doesn't really care
either way.

@29180: This invariant should be added to core CMS; if live and draft
bundles/folders are not in the right position, horrible things will happen; in
particular, publishing new items in them will make them appear in the wrong
place on the live site.

@29056: AFAIK, SystemWideReader wasn't pulled into core; it is very DP specific.
It is used to log in as a special user which is then used to make the requests
back to the editorial server from PFS. Core CMS might need something there if
read access to live items on the editorial server is ever restricted via the
public viewers group, but the code in this changelist is very DP-specific and
most likely not useful for core CMS.

Comment 5 Richard Li 2003-06-02 21:39:52 UTC

Invariant is merged (and ported to ANSI standard SQL so that it runs in both PG
and Oracle).

Comment 6 Justin Ross 2003-06-05 16:42:08 UTC

Regarding the first paragraph of David's comment, change 31978 makes item
unpublish recursively unpublish parents that are empty folders or empty bundles.