needinfo canceled: [Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

Product: Fedora
Version: rawhide
Component: Package Review

Package Review  has canceled Package
Review 's request for James Hogarth
's needinfo:
Bug 1420275: Container Review Request : owncloud - self hosted file storage and
collaboration platform
https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #14 from Package Review 
---
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #12 from Josh Berkus  ---
OK.  Let me know when.  In the meantime, I'm giving it a practical test.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #11 from James Hogarth  ---
There's going to be one final version of this once owncloud-9.1.4-4.fc25 hits
stable.

This is going to enable the owncloud-cron.timer systemd unit and configure
background jobs to use that, rather than webcron, as a much better way of
handling that maintenance stuff required.

Once that is in place, and any final cleanup to bring it inline with all the
guidelines coming out of the VFAD, we should be good for the final review :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #10 from Josh Berkus  ---
Getting worked up to test fully.  I hope this works, because I'll be running it
for my own use ...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #9 from James Hogarth  ---
Following the output of the discussions on Friday about the description/help
labelling, systemd handling and volumes this dockerfile has been updated and
the README added alongside it

Dockerfile: https://fedorapeople.org/~jhogarth/owncloud_container/Dockerfile
README: https://fedorapeople.org/~jhogarth/owncloud_container/README

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #8 from James Hogarth  ---
After a check on #fedora-cloud these needed to be in the atomic-wg not the
cloud-sig one.



https://pagure.io/atomic-wg/issue/233
https://pagure.io/atomic-wg/issue/234
https://pagure.io/atomic-wg/issue/235

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #7 from James Hogarth  ---
The key questions I feel need to be answered in the guidelines to progress this
review are now in the Cloud SIG pagure instance for discussion.

https://pagure.io/cloud-sig/issue/1

https://pagure.io/cloud-sig/issue/2

https://pagure.io/cloud-sig/issue/3

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #6 from James Hogarth  ---
Small note this has been updated to owncloud 9.1.4 now that it has hit the
stable repos.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #5 from James Hogarth  ---
Just to clear up in the PHP bit ... these are the owncloud packages we support
on Fedora and that we support on the PHP shipped with Fedora - nothing special.

It's best not to get wrapped up in that as if there were issues we'd be dealing
with them in Fedora anyway (eg we will have PHP71 in F26 but that needs some
maintainer love to backport a patch from upstream, which I already do and
support, since they don't intend to support PHP71 until owncloud 10
themselves).

Ignore PHP versions here ... that's up to our talented package maintainers in
fedora ;)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #4 from Josh Berkus  ---
UPDATE:

Discussed structure of container on IRC.   Additional components *are*
separated.  This more points to the need for additional metadata in dockerfiles
for the build service.  Will be following up with suggestions.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #3 from Josh Berkus  ---
Ah, I misunderstood the test SQLite instance as the real database.

So the other obvious part is the file storage for WebDAV documents.  When I was
looking at contianerizing OwnCloud myself I identified a couple other
components; let me find the writeup and I'll post it here.

There's also Fedora's limitation to unsupported PHP, which is going to be a
problem here ...

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275



--- Comment #2 from James Hogarth  ---
This is just to sort of talk I'm hoping to engage with given the early
developments of the container review process.

In case you are not aware I am the owncloud maintainer in fedora/epel so am
intimately aware of what it involves.

This container is explicitly httpd+mod_php+mod_ssl as a minimal base, which is
the predominant deployment I've found.

This explicitly does not include a database (other than a test sqlite instance)
but directs the admin to have a database served out of another host/container
and that's an exercise for them.

This is just one component - though the one most people expect - and is similar
to the way it's packaged as a container upstream (not identical as it tries to
take advantage of Fedora features and infrastructure).

Rather than:

> Official Fedora containers need to be examples of good containerized 
> application design.

and

> This container should be sent back and broken into several services.

I'd appreciate some constructive feedback (which is the point of reviews after
all) on where you think a better split or an improved Dockerfile would worked
out - particularly given the lack of guidelines at present.

I care about these packages for some bizarre reason and I care about those who
consume my packages.

It was a crying shame when I had to retire owncloud from EPEL6 and point users
elsewhere, when the time soon comes to retire from EPEL7 I'd really like to be
able to point them to our containers to migrate to.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org

[Bug 1420275] Container Review Request : owncloud - self hosted file storage and collaboration platform

[bugzilla]

https://bugzilla.redhat.com/show_bug.cgi?id=1420275

Josh Berkus  changed:

   What|Removed |Added

 CC||jber...@redhat.com



--- Comment #1 from Josh Berkus  ---
In addition to the other issues mentioned, this is a terrible container. 
Owncloud is a multi-component platform, which should be broken into 3 or more
separate containers, not lumped together in one container like porting a VM. 
Official Fedora containers need to be examples of good containerized
application design.

This container should be sent back and broken into several services.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org