Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 4, 2025

✍️ Description

The update script referenced config/local.yaml instead of config/packages/local.yaml, causing sed to fail with "No such file or directory" during updates.

Changed:

  • Line 59: config/local.yamlconfig/packages/local.yaml

This aligns with the backup (line 48), restore (line 56), and install script paths.

🔗 Related PR / Issue

Link: #6126

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • 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.
Original prompt

This section details on the original issue you should resolve

<issue_title>Kimai local.yml not found error on update command</issue_title>
<issue_description>### ✅ Have you read and understood the above guidelines?

yes

🔎 Did you run the script with verbose mode enabled?

Yes, verbose mode was enabled and the output is included below

📜 What is the name of the script you are using?

kimai.sh

📂 What was the exact command used to execute the script?

update

⚙️ What settings are you using?

  • Default Settings
  • Advanced Settings

🖥️ Which Linux distribution are you using?

Debian 12

📈 Which Proxmox version are you on?

8.4.1

📝 Provide a clear and concise description of the issue.

Step:

sed -i '/^admin_lte:/,/^[^[:space:]]/d' config/local.yaml

there is no local.yaml under config/local.yaml. It must be config/packages/local.yaml or?

🔄 Steps to reproduce the issue.

execute update in container

❌ Paste the full error output (if available).

Updating Kimaised: can't read config/local.yaml: No such file or directory
✖️ in line 59: exit code 2 (Misuse of shell builtins (e.g. syntax error)): while executing command sed -i '/^admin_lte:/,/^[^[:space:]]/d' config/local.yaml

🖼️ Additional context (optional).

No response</issue_description>

<agent_instructions>typo, check against kimai-install.sh</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@MickLesk Yep Typo. Fix in progress. Don't be surprised, I'm handing this over to Copilot because we are currently testing it in the maintainer program, and he is happy (😄) to check such minor details. We check it anyway using the dual control principle.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI changed the title [WIP] Fix local.yml not found error on update command Fix kimai.sh update script path typo for local.yaml Dec 4, 2025
Copilot AI requested a review from MickLesk December 4, 2025 13:13
@MickLesk MickLesk marked this pull request as ready for review December 4, 2025 13:36
@MickLesk MickLesk requested a review from a team as a code owner December 4, 2025 13:36
@github-actions github-actions bot added update script A change that updates a script bugfix labels Dec 4, 2025
@MickLesk MickLesk merged commit 5333e9f into main Dec 4, 2025
7 checks passed
probers1 pushed a commit to probers1/ProxmoxVE that referenced this pull request Dec 5, 2025
…s#9645)

* Initial plan

* Fix typo in kimai.sh: correct config path to config/packages/local.yaml

Co-authored-by: MickLesk <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: MickLesk <[email protected]>
@MickLesk MickLesk deleted the copilot/fix-local-yaml-not-found-error branch December 8, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Kimai local.yml not found error on update command

3 participants