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

[GEOS-10618] DOCS: Official GeoServer Docker Image (GSIP-192) #6107

Merged
merged 4 commits into from
Aug 19, 2022

Conversation

davidblasby
Copy link
Contributor

@davidblasby davidblasby commented Aug 11, 2022

GEOS-10618

This is some quick instructions for using the new Geoserver Docker Image.

Checklist

For core and extension modules:

  • [n/a] New unit tests have been added covering the changes.
  • Documentation has been updated (if change is visible to end users).
  • [n/a] The REST API docs have been updated (when changing configuration objects or the REST controllers).
  • There is an issue in the GeoServer Jira (except for changes that do not affect administrators or end users in any way).
  • [will squash] Commit message(s) must be in the form [GEOS-XYZWV] Title of the Jira ticket.
  • [n/a] Bug fixes and small new features are presented as a single commit.
  • Each commit has a single objective (if there are multiple commits, each has a separate JIRA ticket describing its goal).

Copy link
Member

@jodygarnett jodygarnett left a comment

Choose a reason for hiding this comment

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

Thanks David; looks good; sorry for annoying sphinx feedback.

Docker Container
================

Geoserver is also packaged as a Docker Container. For more details, see the `Geoserver Docker Container Project <https://github.com/geoserver/docker>`_.
Copy link
Member

Choose a reason for hiding this comment

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

Use two _ to avoid making a html target (sphinx is annoying).


This will run the container with the data directory shipped with the container (which is typically an empty directory).

#. Make sure you have `Docker <https://www.docker.com/>`_ installed.
Copy link
Member

Choose a reason for hiding this comment

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

As above


.. code-block::

docker pull docker.osgeo.org/geoserver:2.21.1
Copy link
Member

Choose a reason for hiding this comment

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

Would like to figure out a way for these commands to be updated with the version number; not sure we can do so as part of a code block.

How does the following look:

#. Download the container

   > docker pull docker.osgeo.org/geoserver:|release|


.. code-block::

docker run -it -p8080:8080 docker.osgeo.org/geoserver:2.21.1
Copy link
Member

Choose a reason for hiding this comment

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

As above let's try and get the version numbers correct with no ongoing effort.

doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
Copy link
Member

@jodygarnett jodygarnett left a comment

Choose a reason for hiding this comment

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

Some minor suggestions where I was unsure of formatting.

doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Show resolved Hide resolved
doc/en/user/source/installation/docker.rst Outdated Show resolved Hide resolved
@jodygarnett
Copy link
Member

Copy link
Member

@jodygarnett jodygarnett left a comment

Choose a reason for hiding this comment

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

Thanks for updating; looks good to me

@jodygarnett jodygarnett added the backport 2.21.x Instructs the bot to create a 2.21.x backport PR on merge label Aug 19, 2022
@jodygarnett jodygarnett merged commit edf788f into geoserver:main Aug 19, 2022
@jodygarnett jodygarnett deleted the _dockerdoc branch August 19, 2022 12:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 2.21.x Instructs the bot to create a 2.21.x backport PR on merge
Projects
None yet
2 participants