mirror of
https://github.com/rroemhild/docker-test-openldap.git
synced 2026-04-25 15:15:53 +03:00
[GH-ISSUE #23] Fix listen address #12
Labels
No labels
pull-request
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/docker-test-openldap#12
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @wollanup on GitHub (Jan 4, 2021).
Original GitHub issue: https://github.com/rroemhild/docker-test-openldap/issues/23
Please merge https://github.com/rroemhild/docker-test-openldap/pull/21 ASAP.
CI stages based on your latest image are broken, and we can't rollback, as only
latestimage is provided on DockerHub.Thank you.
@liorhasson commented on GitHub (Jan 4, 2021):
Our CI also broken!!!! ASAP
Also can you please publish docker tags for every release, then we can use specific version!
@rroemhild
Thanks
@rroemhild commented on GitHub (Jan 4, 2021):
Sorry folks! I've reverted the port changes too. Builds will be tagged in the future.
@wollanup commented on GitHub (Jan 4, 2021):
Thank you @rroemhild
Image tags would be great !
It was not a bad idea to use a port > 1024, maybe in a 2.0 tag as it's a breaking change ?
@wollanup commented on GitHub (Jan 4, 2021):
Oh, I've just seen you change ports again a few seconds ago :D
So we keep unprivilegied ports
10XXX, which sounds logical.@rroemhild commented on GitHub (Jan 4, 2021):
I've made it worse :D
I add the tag
v1to the latest commit/build before I've done major changes.@wollanup commented on GitHub (Jan 4, 2021):
OK, your latest commit sounds goot to me, with the new unprivilegied ports config
10389, my CI stage pass.May I suggest you to publish docker image
2.0and1.0(as a fallback) on DockerHub for folks who can't update their code/config ?Maybe it's easier for some people to only add the target tag in a docker-compose file ?
@rroemhild commented on GitHub (Jan 4, 2021):
Great! Tags are now on DockerHub.