Attempt to define a binary RPM for a full server package

This is an attempt to provide an end-to-end app RPM.

@michael.thomas, the only addition I was confident enough to make was to have a binary RPM that installs /etc/gwdatafind-server.ini. @duncan.meacher's docker config uses gunicorn and supervisor, what are your recommendations for infrastructure for an RPM equivalent (mod_wsgi)?

changed milestone to %GWDataFind Server 1.1.0

@duncanmmacleod The setup that I'm currently testing is using apache as a proxy to the gunicorn service, bypassing wsgi. I'm not sure if this is the best approach, though.

A sample apache config should be included, but can be put into the %doc line of the %files section. Perhaps two should be included, one for running as a proxy to the gunicorn service, and one for running as a wsgi.

@michael.thomas, I've uplodated 'v2' which includes example .conf files into /usr/share/gwdatafind-server, if that's appropriate. I haven't tested these.

@duncan.meacher, this MR now separates your original httpd.conf file into a gunicorn http file with no gwdatafind references, and a separate gwdatafind-server.conf file - can you please test the Dockerfile and related changes?

added 3 commits

• 1fdd7f6a...968ccb72 - 2 commits from branch computing/gwdatafind:master
• 0368ff69 - attempt to define a binary RPM for a full server package

Compare with previous version

added 1 commit

• cfe45353 - attempt to define a binary RPM for a full server package

Compare with previous version

added 18 commits

• cfe45353...96841028 - 17 commits from branch computing/gwdatafind:master
• 1eb282ee - attempt to define a binary RPM for a full server package

Compare with previous version

added 1 commit

• 7df088fb - tests: patch isfile to check get_config_path error

Compare with previous version

marked this merge request as ready

Ok, this is good enough for a first cut, I know that @duncan.meacher has extra changes to make on top of this, so I propose we get this merged now and move on with out lives.

@michael.thomas, if you have more suggestions, please open new tickets.

requested review from @duncan.meacher

assigned to @duncan.meacher

added packaging label

approved this merge request

mentioned in commit 4854c4f7

merged

mentioned in merge request !37 (merged)