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

set DOCKER_HOST only if needed #2149

Merged
merged 1 commit into from
Jan 11, 2024
Merged

Conversation

fruch
Copy link
Contributor

@fruch fruch commented Jan 10, 2024

we have places where we use a specific DOCKER_HOST in order to be able to support case of running the monitoring stack inside a running podman instance via socket, those places changing the DOCKER_HOST are breaking this usage, and were preventing the monitoring stack from starting

we have places where we use a specific DOCKER_HOST in order
to be able to support case of running the monitoring stack inside
a running podman instance via socket, those places
changing the DOCKER_HOST are breaking this usage, and were
preventing the monitoring stack from starting
@fruch fruch requested a review from amnonh January 10, 2024 07:40
@amnonh
Copy link
Collaborator

amnonh commented Jan 10, 2024

a PR should fix an issue, can you open one?

@fruch
Copy link
Contributor Author

fruch commented Jan 10, 2024

It's in the context of this issue

scylladb/scylla-cluster-tests#7080

fruch added a commit to fruch/scylla-cluster-tests that referenced this pull request Jan 10, 2024
* podman now defaults to short-name-mode="enforcing", and the monitor stack
  doesn't able to pull images when working via socket,
  it doesn't have tty to offer which repository to alias it,
  setting short-name-mode="permissive" on ~/.config/containers/registries.conf
  alleviate this issue

* scylla-monitoring was using DOCKER_HOST inside it's start scripts, hence breaking
  our ability to use it to point to podman socket, a fix was subbmitted
  but until then we patch those out of the scripts so older collected monitoring
  data can be used

Ref: scylladb/scylla-monitoring#2149
Fixes: scylladb#7080
fruch added a commit to fruch/scylla-cluster-tests that referenced this pull request Jan 11, 2024
* podman now defaults to short-name-mode="enforcing", and the monitor stack
  doesn't able to pull images when working via socket,
  it doesn't have tty to offer which repository to alias it,
  setting short-name-mode="permissive" on ~/.config/containers/registries.conf
  alleviate this issue

* scylla-monitoring was using DOCKER_HOST inside it's start scripts, hence breaking
  our ability to use it to point to podman socket, a fix was subbmitted
  but until then we patch those out of the scripts so older collected monitoring
  data can be used

Ref: scylladb/scylla-monitoring#2149
Fixes: scylladb#7080
@amnonh amnonh merged commit 5c0068b into scylladb:master Jan 11, 2024
@amnonh
Copy link
Collaborator

amnonh commented Jan 11, 2024

@fruch I've backported to branch 4.6, this means for QA it is already available

@fruch
Copy link
Contributor Author

fruch commented Jan 11, 2024

@fruch I've backported to branch 4.6, this means for QA it is already available

thanks mean while, I'm patching it at SCT end, for people to be able to retrieve monitor data that didn't yet had this fix (we are caching the monitoring stack with the data)

fruch added a commit to scylladb/scylla-cluster-tests that referenced this pull request Jan 11, 2024
* podman now defaults to short-name-mode="enforcing", and the monitor stack
  doesn't able to pull images when working via socket,
  it doesn't have tty to offer which repository to alias it,
  setting short-name-mode="permissive" on ~/.config/containers/registries.conf
  alleviate this issue

* scylla-monitoring was using DOCKER_HOST inside it's start scripts, hence breaking
  our ability to use it to point to podman socket, a fix was subbmitted
  but until then we patch those out of the scripts so older collected monitoring
  data can be used

Ref: scylladb/scylla-monitoring#2149
Fixes: #7080
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.

2 participants