geoserver icon indicating copy to clipboard operation
geoserver copied to clipboard

Published 20 hours ago • verified publisher icon hishamkaram

AMP changes

Open wichert opened this issue 2 years ago • 4 comments

I'm afraid this PR is a bit messy. At this point I am mostly hoping to get an indication of interest for the changes in here. If there is interest we can work on cleaning things up and creating a a set of smaller PRs for individual changes.

This PR is a collection of changes that have been made by the AMP team in Woven Planet to help us configure geoserver clusters. The changes include:

  • support for additional endpoints
  • a couple of bugfixes
  • some code cleanup (gitignore, use stdlib http status, use of go modules
  • simpify the docker container

wichert avatar Feb 21 '22 13:02 wichert

@wichert thank you for your contribution, i will take a look on your Pull Request

hishamkaram avatar Feb 22 '22 05:02 hishamkaram

thank you for your contribution, what are your plans for the logger @wichert ?

eknowles avatar Mar 02 '22 20:03 eknowles

thank you for your contribution, what are your plans for the logger @wichert ?

Are you referring to the changes in 84727c1ca7dbdb1acfe799053110ab88dae59f87 ? I suspect that was done to decrease verbosity and move logging control to the caller.

wichert avatar Mar 07 '22 12:03 wichert

@wichert

  • could you update your branch and check if the unit test passes? (I had to revert this PR #15 because the unit test is failing)
  • I would like to know which versions of geoserver this PR is compatible with?

hishamkaram avatar Apr 17 '22 04:04 hishamkaram