-
Notifications
You must be signed in to change notification settings - Fork 37
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
Cherrypicks from18 #1065
Cherrypicks from18 #1065
Conversation
This is the same version that the main container currently uses. (cherry picked from commit d76d258)
After adding health checks to docker compose, this command is no longer used anywhere else. (cherry picked from commit 73732af)
Only install the test dependencies in in the image used by the CI. (cherry picked from commit 87e824a)
This setting only applies to gthread according to the documentation: https://docs.gunicorn.org/en/23.0.0/settings.html#threads (cherry picked from commit aa05e6c)
(cherry picked from commit 0d5ca2f)
(cherry picked from commit 5cbc246)
(cherry picked from commit a193373)
…er in pyproject.toml. (cherry picked from commit f341f3d)
(cherry picked from commit 9338e8f)
for more information, see https://pre-commit.ci
8e5029c
to
707c266
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop-1.9 #1065 +/- ##
============================================
Coverage 93.27% 93.28%
============================================
Files 53 53
Lines 7038 7039 +1
============================================
+ Hits 6565 6566 +1
Misses 473 473
|
dd5ff78
to
7071bde
Compare
I think #1046 is missing from the bottom of this PR. |
Those commits are still in the history - but maybe I've mangled them a bit trying to get all the changes working together nicely. This PR was for the PRs merged to |
06f0545
to
bf41dd9
Compare
I'm getting the ows user as 1001 (ubuntu being 1000). Looks like remap-user.sh only works if ows user is expected to be 1000? And I have no idea what is causing this difference between the ows and ows18 images. |
What do you think of this @pjonsson? I had to manually delete the ubuntu user to make it all work. I have no idea why the ubuntu user seems to exist here but not in ows18. |
I did a test run of the change in #1067 with PR #1068 and that passed. I've never done PRs into other PRs before, but I believe if you merge #1067, that will go into this branch and re-trigger the CI jobs for this PR, which should then pass. Here's what I did to construct #1067 when standing on the $ git diff master -- Dockerfile
diff --git a/Dockerfile b/Dockerfile
index bf69f07b..aa752873 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -69,6 +69,8 @@ RUN EXTRAS=$([ "$ENVIRONMENT" = "deployment" ] || echo ",test") && \
python3-pip))
# Configure user
+RUN userdel -r ubuntu
+RUN useradd -m -s /bin/bash ows
WORKDIR "/home/ubuntu"
ENV GDAL_DISABLE_READDIR_ON_OPEN="EMPTY_DIR" \ and then repeat that for a bunch of other files of interest. Then pick one file from the main branch that I wanted in its entirety, and checkout that from the main branch and make a commit with that file. Then rinse and repeat for all the other files I wanted in their entirety, but amend them to the commit:
After that, I had a look at the files where there should be some differences:
and then I just manually edited in the changes I wanted into the respective files, and amended that to the alignment commit. After completing the changes, I ran the word-diff again to see that the only changes that remained were the image names and other things that should be different. |
I will be offline for a couple of days now, sorry about that. I believe this should be fixed by my fixup, but if there's something more, I will check things on Thursday evening my time, which I guess is early Friday morning for you. |
I've been offline for a couple of days myself as it happens. Looks like your tweaks worked. I'll take one last read through the remaining changes this morning and merge. |
Forward ports of #1055, #1057, #1059, #1061 and #1064
📚 Documentation preview 📚: https://datacube-ows--1065.org.readthedocs.build/en/1065/