[PR #127] [MERGED] enh: groupList: use cache to speedup calls #259

New issue

Closed

opened 2026-05-07 00:20:02 +02:00 by BreizhHardware · 0 comments

BreizhHardware commented

2026-05-07 00:20:02 +02:00

Owner

📋 Pull Request Information

Original PR: https://github.com/ovh/the-bastion/pull/127
Author: @speed47
Created: 1/19/2021
Status: ✅ Merged
Merged: 1/20/2021
Merged by: @speed47

Base: master ← Head: cache

📝 Commits (3)

171b791 enh: groupList: use cache to speedup calls
b56eb40 fix: debian9: create_file_if_not_exists couldn't chgrp by filehanddle
d5a664c chore: tests: also update totalerrors while tests are running

📊 Changes

4 files changed (+71 additions, -23 deletions)

View changed files

📝 bin/plugin/open/groupList (+5 -5)
📝 lib/perl/OVH/Bastion.pm (+15 -6)
📝 lib/perl/OVH/Bastion/allowkeeper.inc (+42 -10)
📝 tests/functional/launch_tests_on_instance.sh (+9 -2)

📄 Description

On bastions with thousands of group, the speedup is ~x10

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/ovh/the-bastion/pull/127 **Author:** [@speed47](https://github.com/speed47) **Created:** 1/19/2021 **Status:** ✅ Merged **Merged:** 1/20/2021 **Merged by:** [@speed47](https://github.com/speed47) **Base:** `master` ← **Head:** `cache` --- ### 📝 Commits (3) - [`171b791`](https://github.com/ovh/the-bastion/commit/171b791c0f2e89f39016e1f06d36b2af5ef9c102) enh: groupList: use cache to speedup calls - [`b56eb40`](https://github.com/ovh/the-bastion/commit/b56eb40ea5564929e98eacb47e3155706947a174) fix: debian9: create_file_if_not_exists couldn't chgrp by filehanddle - [`d5a664c`](https://github.com/ovh/the-bastion/commit/d5a664c1145f75b4a47606101045610f342647ba) chore: tests: also update totalerrors while tests are running ### 📊 Changes **4 files changed** (+71 additions, -23 deletions) <details> <summary>View changed files</summary> 📝 `bin/plugin/open/groupList` (+5 -5) 📝 `lib/perl/OVH/Bastion.pm` (+15 -6) 📝 `lib/perl/OVH/Bastion/allowkeeper.inc` (+42 -10) 📝 `tests/functional/launch_tests_on_instance.sh` (+9 -2) </details> ### 📄 Description On bastions with thousands of group, the speedup is ~x10 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>