geerlingguy / ansible-role-nodejs

Ansible Role - Node.js
https://galaxy.ansible.com/geerlingguy/nodejs/
MIT License
410 stars 252 forks source link

Need architectures config in ansible.builtin.deb822_repository #161

Closed gilbert-hsu closed 1 week ago

gilbert-hsu commented 6 months ago

after upgrade to 7.0.0, find notice with apt update

N: Skipping acquire of configured file 'main/binary-i386/Packages' as repository 'https://deb.nodesource.com/node_18.x nodistro InRelease' doesn't support architecture 'i386'

with various architectures, an architecture map would be good ?

such like

architecture_map:
  amd64: amd64
  x86_64: amd64
  armv7l: arm
  aarch64: arm64
  64-bit: amd64

and introduce architecture_map[ansible_architecture] as architectures config in ansible.builtin.deb822_repository

ansible.builtin.deb822_repository:
  name: nodesource_{{ nodejs_version }}
  uris: "https://deb.nodesource.com/node_{{ nodejs_version }}"
  types: deb
  suites: nodistro
  architectures: "{{ architecture_map[ansible_architecture] }}"
  components: main
  signed_by: "{{ node_signed_key.dest }}"
  state: present
github-actions[bot] commented 2 months ago

This issue has been marked 'stale' due to lack of recent activity. If there is no further activity, the issue will be closed in another 30 days. Thank you for your contribution!

Please read this blog post to see the reasons why I mark issues as stale.

github-actions[bot] commented 1 week ago

This issue has been closed due to inactivity. If you feel this is in error, please reopen the issue or file a new issue with the relevant details.