Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Review server upgrade and password provider pages #576

Merged
merged 13 commits into from Dec 12, 2013

Conversation

sbesson
Copy link
Member

@sbesson sbesson commented Dec 6, 2013

This PR addresses two critical tickets for 5.0.0-beta2: https://trac.openmicroscopy.org.uk/ome/ticket/11787 and http://trac.openmicroscopy.org.uk/ome/ticket/11770

The server upgrade page on the sysadmin section should now:

  • include a top-level warning about the new password provider with a link to the relevant configuration section
  • show the exact DB upgrade commands when upgrading from either 5.0.0-beta1 or 4.4.9
  • add a note about the need to reset the JVM memory settings after upgrade
    The 5.0.0-beta1 page about the no-upgrade from 4.4 is removed and the server upgrade page is added to the what's new page.

In addition to this documentation PR, a matrix job has been set up OMERO-upgrade-develop which tests most aspects of the upgrade as listed on the server upgrade page:

  • fire up a previous server
  • download the new binaries, update the DB
  • perform the optimisation steps
  • restarts the upgrade server with the upgrade server
  • test the dump restore and restart the old server
    Some obvious next steps for this upgrade job are:
  • fill the DB with some basic components (user, group, images, annotations…)
  • test script upgrades

--rebased-to #994 (salting only)

@bpindelski
Copy link

The omero/developers/Server/PasswordProvider.txt:Things to keep in mind paragraph elements have either no dot or two dots at the end of the sentence.

@bpindelski
Copy link

omero/sysadmins/index.txt has OMERO.fs: file upload, Advanced configuration and LDAP authentication twice - once in Advanced topics and once in Advanced Server Installation.

@@ -37,6 +34,16 @@ below. Please refer to each section for additional details.
:local:
:depth: 1

.. warning::
with 5.0.0-beta2, the default JDBC password provider has been modified to

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sentence could start with a capital letter?

@bpindelski
Copy link

Your OMERO configuration is stored using config.xml in the etc/grid `` - that's a nasty tick-ing around etc/grid in `sysadmins/server-upgrade.html`


(1 row)

From a 5.0.0-beta1 server

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two spaces

@bpindelski
Copy link

Some comments left. OK to merge after they have been looked at. Thanks 👍

@hflynn
Copy link
Member

hflynn commented Dec 10, 2013

Looks like this is good to merge now. @joshmoore are you happy this is clear? I'll merge this evening if no further comments.

@hflynn
Copy link
Member

hflynn commented Dec 10, 2013

Actually, did you want to add the password thing to the whatsnew file too? It would make sense to have it on this PR. I'll leave off merging for tonight.

@hflynn
Copy link
Member

hflynn commented Dec 12, 2013

Thanks. Merging now.

hflynn added a commit that referenced this pull request Dec 12, 2013
Review server upgrade and password provider pages
@hflynn hflynn merged commit 4b9fd5e into ome:develop Dec 12, 2013
@sbesson sbesson deleted the 11787_server_upgrade branch December 15, 2013 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants