Skip to content

Add geoserver yml#204

Merged
max-zilla merged 16 commits into
developfrom
add-geoserver-yml
Sep 10, 2021
Merged

Add geoserver yml#204
max-zilla merged 16 commits into
developfrom
add-geoserver-yml

Conversation

@tcnichol
Copy link
Copy Markdown
Contributor

@tcnichol tcnichol commented Apr 2, 2021

With this pull request, I have added a .yml file for the geoserver
next,

docker-compose -f docker-compose.yml -f docker-compose.geoserver.yml up -d

This will run clowder with geoserver and the extractors.

I am making this a work in progress and asking for feedback for the following reasons.

  1. I have not yet figured out how to use the proxy server.

Feel free to add any feedback.

@tcnichol tcnichol requested review from a team, bodom0015 and robkooper April 2, 2021 18:30
- PROXY_ON=${PROXY_ON}
- PROXY_HOST=${PROXY_HOST}
- PROXY_URL=${PROXY_URL}
- GEOSERVER_USERNAME=${GEOSERVER_USERNAME:-admin}
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you add these to the env.example and document them

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes, can do.

@tcnichol tcnichol changed the title Add geoserver yml {WORK IN PROGRESS} Add geoserver yml May 7, 2021
@tcnichol tcnichol changed the title Add geoserver yml Add geoserver yml {WORK IN PROGRESS} May 7, 2021
@lmarini lmarini marked this pull request as draft July 7, 2021 20:05
@tcnichol tcnichol changed the title Add geoserver yml {WORK IN PROGRESS} Add geoserver yml Aug 3, 2021
@tcnichol
Copy link
Copy Markdown
Contributor Author

tcnichol commented Aug 3, 2021

NOTE

This works with docker-compose, but I have not been able to figure out how to get the proxy to work.

My plan is to have this pull request, and then create a new issue for the proxy, and do that on a later pull request.

As things are now, they at least can be used with docker-compose.

@tcnichol tcnichol requested review from bingzhang and max-zilla August 6, 2021 16:06
@max-zilla
Copy link
Copy Markdown
Contributor

@tcnichol looks like env.example needs to be updated still, can you add those and then mark this not as draft? otherwise looks good.

Copy link
Copy Markdown
Contributor

@max-zilla max-zilla left a comment

Choose a reason for hiding this comment

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

update env.example

@tcnichol
Copy link
Copy Markdown
Contributor Author

I updated the env.example

Let me know if there are better values to put in than what I have there.

@max-zilla max-zilla marked this pull request as ready for review August 16, 2021 14:26
@max-zilla max-zilla merged commit c2e2069 into develop Sep 10, 2021
@max-zilla max-zilla deleted the add-geoserver-yml branch September 10, 2021 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants