Skip to content

Commit

Permalink
fix(falcon_install): pass in the appropriate filter for architecture (#…
Browse files Browse the repository at this point in the history
…521)

* fix(sensor_download_info): remove offset from args

* fix(falcon_install): pass in the appropriate filter for architectures now

Finally we can take advantage of the new API endpoint that allows
querying by architecture!

* chore: add changelog fragments
  • Loading branch information
carlosmmatos authored May 21, 2024
1 parent 7d30714 commit bee4d05
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 4 deletions.
3 changes: 3 additions & 0 deletions changelogs/fragments/v2-sensor-installers.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
bugfixes:
- sensor_download_info - fix offset and use override for v2 endpoint (https://github.com/CrowdStrike/ansible_collection_falcon/pull/520)
- falcon_install - fix filter to take advantage of new architectures field (https://github.com/CrowdStrike/ansible_collection_falcon/pull/521)
1 change: 0 additions & 1 deletion plugins/modules/sensor_download_info.py
Original file line number Diff line number Diff line change
Expand Up @@ -151,7 +151,6 @@
DOWNLOAD_INFO_ARGS = {
"filter": {"type": "str", "required": False},
"limit": {"type": "int", "required": False, "default": 100},
"offset": {"type": "int", "required": False},
"sort": {"type": "str", "required": False},
}

Expand Down
6 changes: 3 additions & 3 deletions roles/falcon_install/vars/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,6 @@ falcon_cloud_urls:
us-gov-1: "api.laggar.gcw.crowdstrike.com"

falcon_os_arch_dict:
x86_64: "+os_version:!~'arm64'+os_version:!~'zLinux'"
aarch64: "+os_version:~'arm64'"
s390x: "+os_version:~'zLinux'"
x86_64: "+architectures:'x86_64'"
aarch64: "+architectures:'arm64'"
s390x: "+architectures:'s390x'"

0 comments on commit bee4d05

Please sign in to comment.