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

install role: make sure pkg_mgr fact is available for localhost installs #146

Merged
merged 4 commits into from
Jun 15, 2023

Conversation

felixfontein
Copy link
Collaborator

Fixes #145.

@codecov
Copy link

codecov bot commented Jun 1, 2023

Codecov Report

Merging #146 (78fe92a) into main (b7bfe2a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #146   +/-   ##
=======================================
  Coverage   65.92%   65.92%           
=======================================
  Files          12       12           
  Lines         989      989           
  Branches      223      223           
=======================================
  Hits          652      652           
  Misses        256      256           
  Partials       81       81           
Flag Coverage Δ
integration 64.82% <ø> (ø)
sanity 22.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@felixfontein felixfontein marked this pull request as draft June 2, 2023 06:07
@felixfontein
Copy link
Collaborator Author

After some tries to reproduce this, and looking up the ansible-core code, I don't think this will actually fix the probem. See #145 (comment) for details.

@felixfontein felixfontein marked this pull request as ready for review June 2, 2023 11:29
@felixfontein
Copy link
Collaborator Author

It would be nice to know why #145 actually happens, but this seems to fix it (#145 (comment)).

@felixfontein felixfontein merged commit 2f12a19 into ansible-collections:main Jun 15, 2023
@felixfontein felixfontein deleted the play branch June 15, 2023 06:37
doonga referenced this pull request in doonga/greyrock-ops Jun 16, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[community.sops](https://togithub.com/ansible-collections/community.sops)
| galaxy-collection | patch | `1.6.1` -> `1.6.2` |

---

### Release Notes

<details>
<summary>ansible-collections/community.sops</summary>

###
[`v1.6.2`](https://togithub.com/ansible-collections/community.sops/blob/HEAD/CHANGELOG.rst#v162)

[Compare
Source](https://togithub.com/ansible-collections/community.sops/compare/1.6.1...1.6.2)

\======

## Release Summary

Maintenance release.

## Bugfixes

- install role - make sure that the `pkg_mgr` fact is definitely
available when installing on `localhost`. This can improve error
messages in some cases
([https://github.com/ansible-collections/community.sops/issues/145](https://togithub.com/ansible-collections/community.sops/issues/145),
[https://github.com/ansible-collections/community.sops/pull/146](https://togithub.com/ansible-collections/community.sops/pull/146)).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTguMCIsInVwZGF0ZWRJblZlciI6IjM1LjExOC4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: greyrock-bot <1583719+greyrock-bot[bot]@users.noreply.github.com>
coolguy1771 referenced this pull request in coolguy1771/home-ops Jun 20, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
|
[community.sops](https://togithub.com/ansible-collections/community.sops)
| galaxy-collection | patch | `1.6.1` -> `1.6.2` |

---

### ⚠ Dependency Lookup Warnings ⚠

Warnings were logged while processing this repo. Please check the
Dependency Dashboard for more information.

---

### Release Notes

<details>
<summary>ansible-collections/community.sops (community.sops)</summary>

###
[`v1.6.2`](https://togithub.com/ansible-collections/community.sops/blob/HEAD/CHANGELOG.rst#v162)

[Compare
Source](https://togithub.com/ansible-collections/community.sops/compare/1.6.1...1.6.2)

\======

## Release Summary

Maintenance release.

## Bugfixes

- install role - make sure that the `pkg_mgr` fact is definitely
available when installing on `localhost`. This can improve error
messages in some cases
([https://github.com/ansible-collections/community.sops/issues/145](https://togithub.com/ansible-collections/community.sops/issues/145),
[https://github.com/ansible-collections/community.sops/pull/146](https://togithub.com/ansible-collections/community.sops/pull/146)).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you
are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Renovate
Bot](https://togithub.com/renovatebot/renovate).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNS4xMTguMCIsInVwZGF0ZWRJblZlciI6IjM1LjEzMy4xIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9-->

Co-authored-by: lumiere-bot <98047013+lumiere-bot[bot]@users.noreply.github.com>
nrdufour added a commit to nrdufour/home-ops that referenced this pull request Jul 27, 2023
This PR contains the following updates:

| Package | Type | Update | Change |
|---|---|---|---|
| [community.sops](https://github.com/ansible-collections/community.sops) | galaxy-collection | patch | `1.6.1` -> `1.6.4` |

---

### Release Notes

<details>
<summary>ansible-collections/community.sops (community.sops)</summary>

### [`v1.6.4`](https://github.com/ansible-collections/community.sops/blob/HEAD/CHANGELOG.rst#v164)

[Compare Source](ansible-collections/community.sops@1.6.3...1.6.4)

\======

## Release Summary

Maintenance/bugfix release for the move of sops to the new `getsops GitHub organization <https://github.com/getsops>`\__.

## Bugfixes

-   install role - fix `sops_github_latest_detection=latest-release`, which broke due to sops moving to another GitHub organization (ansible-collections/community.sops#151).

### [`v1.6.3`](https://github.com/ansible-collections/community.sops/blob/HEAD/CHANGELOG.rst#v163)

[Compare Source](ansible-collections/community.sops@1.6.2...1.6.3)

\======

## Release Summary

Maintenance release with updated documentation.

From this version on, community.sops is using the new `Ansible semantic markup <https://docs.ansible.com/ansible/devel/dev_guide/developing_modules_documenting.html#semantic-markup-within-module-documentation>`\__
in its documentation. If you look at documentation with the ansible-doc CLI tool
from ansible-core before 2.15, please note that it does not render the markup
correctly. You should be still able to read it in most cases, but you need
ansible-core 2.15 or later to see it as it is intended. Alternatively you can
look at `the devel docsite <https://docs.ansible.com/ansible/devel/collections/community/sops/>`\__
for the rendered HTML version of the documentation of the latest release.

## Known Issues

-   Ansible markup will show up in raw form on ansible-doc text output for ansible-core before 2.15. If you have trouble deciphering the documentation markup, please upgrade to ansible-core 2.15 (or newer), or read the HTML documentation on https://docs.ansible.com/ansible/devel/collections/community/sops/.

### [`v1.6.2`](https://github.com/ansible-collections/community.sops/blob/HEAD/CHANGELOG.rst#v162)

[Compare Source](ansible-collections/community.sops@1.6.1...1.6.2)

\======

## Release Summary

Maintenance release.

## Bugfixes

-   install role - make sure that the `pkg_mgr` fact is definitely available when installing on `localhost`. This can improve error messages in some cases (ansible-collections/community.sops#145, ansible-collections/community.sops#146).

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update again.

---

 - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box

---

This PR has been generated by [Renovate Bot](https://github.com/renovatebot/renovate).
<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi4yMy4yIiwidXBkYXRlZEluVmVyIjoiMzYuMjMuMiIsInRhcmdldEJyYW5jaCI6Im1haW4ifQ==-->

Co-authored-by: Nicolas Dufour <nrdufour@gmail.com>
Reviewed-on: https://git.home/nrdufour/home-ops/pulls/7
Co-authored-by: Renovate <renovate@ptinem.io>
Co-committed-by: Renovate <renovate@ptinem.io>
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.

convenience playbooks fail to missing facts
1 participant