Skip to content

Created execute.sh to execute the same command inside multiple contai…#747

Merged
MickLesk merged 1 commit intocommunity-scripts:mainfrom
jeroenzwart:tools-pve-execute
Aug 6, 2025
Merged

Created execute.sh to execute the same command inside multiple contai…#747
MickLesk merged 1 commit intocommunity-scripts:mainfrom
jeroenzwart:tools-pve-execute

Conversation

@jeroenzwart
Copy link
Contributor

✍️ Description

Added new script based on other script in tool section. With this script you can execute the same command at once inside multiple containers. Select containers to skip, enter the command and execute it inside the containers with a output of the command.

🔗 Related PR / Issue

Link: None

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No breaking changes – Existing functionality remains intact.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔍 Code & Security Review (X in brackets)

  • Follows Code_Audit.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials

📋 Additional Information (optional)

I don't program daily in shell. So feel free to adjust the code what could be better ;)

@jeroenzwart jeroenzwart requested a review from a team as a code owner July 24, 2025 22:06
@MickLesk MickLesk merged commit b3c41fd into community-scripts:main Aug 6, 2025
1 check passed
@jeroenzwart
Copy link
Contributor Author

@MickLesk Hi, is there something I need to do to merge into 'community-scripts/ProxmoxVE'? First time contributor here. Just making sure I didn't miss some part of the process.

@MickLesk
Copy link
Member

Yeah forgotten, because we Check the Issues to pr in VED. Create an Issue with ScriptName, then i can Check at monday 😅

@jeroenzwart jeroenzwart mentioned this pull request Sep 13, 2025
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