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

[video reserves] Try to add php to mods-enabled #5456

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

maxkadel
Copy link
Contributor

@maxkadel maxkadel commented Oct 23, 2024

Does not result in php8.3 being included in /etc/apache2/mods-enabled

@maxkadel maxkadel changed the title videoreserves Try to add php to mods-enabled [video reserves] Try to add php to mods-enabled Oct 23, 2024
@kayiwa kayiwa force-pushed the videoreserves_incident branch from db7cb22 to d7e453c Compare October 25, 2024 12:59
The apache2_module is redundant
@kayiwa
Copy link
Member

kayiwa commented Oct 25, 2024

I added the following debug tasks:

- name: Video_reserves | Get list of enabled Apache modules
  ansible.builtin.command: apache2ctl -M
  register: apache2_modules
  changed_when: false

- name: Video_reserves | install apache modules
  community.general.apache2_module:
    name: "{{ item }}"
    identifier: "{{ item }}"
    state: present
  when: >-
        (apache2_modules.stdout is defined and
        'php{{ php_version }}' not in apache2_modules.stdout.splitlines())
  notify: restart apache
  loop:
    - php{{ php_version }}

and I am pretty sure we do not need to enable this module.

TASK [video_reserves : Video_reserves | install apache php modules] ************
ok: [instance] => (item=libapache2-mod-php8.3)

TASK [video_reserves : Video_reserves | Get list of enabled Apache modules] ****
ok: [instance]

TASK [video_reserves : Debug apache2_modules output] ***************************
ok: [instance] => {
    "msg": [
        "Loaded Modules:",
        " core_module (static)",
        " so_module (static)",
        " watchdog_module (static)",
        " http_module (static)",
        " log_config_module (static)",
        " logio_module (static)",
        " version_module (static)",
        " unixd_module (static)",
        " access_compat_module (shared)",
        " alias_module (shared)",
        " auth_basic_module (shared)",
        " authn_core_module (shared)",
        " authn_file_module (shared)",
        " authz_core_module (shared)",
        " authz_host_module (shared)",
        " authz_user_module (shared)",
        " autoindex_module (shared)",
        " deflate_module (shared)",
        " dir_module (shared)",
        " env_module (shared)",
        " expires_module (shared)",
        " filter_module (shared)",
        " headers_module (shared)",
        " mime_module (shared)",
        " mpm_prefork_module (shared)",
        " negotiation_module (shared)",
        " php_module (shared)",
        " reqtimeout_module (shared)",
        " rewrite_module (shared)",
        " setenvif_module (shared)",
        " status_module (shared)",
        " vhost_alias_module (shared)"
    ]
}

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