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

Fixed apt cache update #6

Merged
merged 1 commit into from
Jan 9, 2025
Merged

Fixed apt cache update #6

merged 1 commit into from
Jan 9, 2025

Conversation

wsmirnow
Copy link
Contributor

@wsmirnow wsmirnow commented Dec 19, 2024

Apt cache update was triggered by a notify. A notify handler is running at the end of the Ansible block. As this role is used as dependency for other roles (like elan.opencast_elasticsearch), the trigger will run to late.

Example:

- name: Deploy Elasticsearch
  hosts: elasticsearch
  roles:
    - elan.opencast_elasticsearch <-- The packages are located in an Opencast repository
                                      that is unknown on a clean system. Without an apt update
                                      performed so far, the role will fail.
    <-- triggers would run here

Apt cache should be updated by this role and not by others.

Apt cache update was triggered by a notify. A notify handler is running at the end of the Ansible block. As this role is used as dependency for other roles (like opencast_elasticsearch), the trigger will run to late.
@wsmirnow
Copy link
Contributor Author

The tests are failing due to broken molecule tests. This is handled by #5.

Copy link
Collaborator

@lkiesow lkiesow left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me

@lkiesow lkiesow merged commit 5c6cf05 into main Jan 9, 2025
0 of 2 checks passed
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