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

Initial support for RHEL8 #198

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Initial support for RHEL8 #198

wants to merge 2 commits into from

Conversation

cp-elastic
Copy link
Contributor

This PR fixes RHEL8's configurations to work with the new Podman workflow.

Related:
#196
#182

Copy link

@smalenfant smalenfant left a comment

Choose a reason for hiding this comment

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

This is not only for RHEL8, but please see that specific diff for systemd (podman-restart.service) that's required when a server reboots to restart the ECE services. I also added the "docker" command specific by the documentation (optional).

 - name: Podman daemon is enabled and systemd has read all changes
-  systemd:
+  ansible.builtin.systemd:
     name: "{{ item }}"
     enabled: yes
     daemon_reload: yes
   loop:
     - podman.service
-    - podman.socket
\ No newline at end of file
+    - podman.socket
+    - podman-restart.service
+
+- name: Create docker cmd redirect to podman
+  ansible.builtin.copy:
+    content: |
+      #!/bin/bash
+      podman-remote --url unix:///var/run/podman/podman.sock "$@"
+    dest: /usr/bin/docker
+    mode: 0755
\ No newline at end of file

@cp-elastic
Copy link
Contributor Author

cp-elastic commented Oct 25, 2024

@smalenfant Good catch on podman-restart.service, I will add that. I'm not sure I follow what the purpose of the copy task is for. The podman-docker package does something similar, but I chose to omit it to reduce complexity.

@smalenfant
Copy link

@smalenfant Good catch on podman-restart.service, I will add that. I'm not sure I follow what the purpose of the copy task is for. The podman-docker package does something similar, but I chose to omit it to reduce complexity.

I don't think we need that fake docker to podman link. This was only something documented by Elastic. Maybe required for diagnostics? I don't know.

RHEL/Rocky Step 14

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