starred/ProxmoxVE
1
0
Fork 0
mirror of https://github.com/community-scripts/ProxmoxVE.git synced 2026-02-05 00:29:55 +03:00
Code Issues 12 Packages Projects Releases 10 Wiki Activity

scripts add-iptag updates IPs of LXCs but not of VMs #1138

New Issue
Closed
opened 2026-02-04 23:16:19 +03:00 by OVERLORD · 17 comments
OVERLORD commented 2026-02-04 23:16:19 +03:00
Owner
Copy Link

Originally created by @DST73 on GitHub (Jun 22, 2025).

✅ Have you read and understood the above guidelines?

yes

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

add-iptag

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

bash -c "$(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/tools/pve/add-iptag.sh)"

⚙️ What settings are you using?

  • Default Settings
  • Advanced Settings

🖥️ Which Linux distribution are you using?

No response

📝 Provide a clear and concise description of the issue.

The script only updates the IPs of the LXCs but not of my VMs

🔄 Steps to reproduce the issue.

run the scripts within proxmox shell

❌ Paste the full error output (if available).

🖼️ Additional context (optional).

Image

Originally created by @DST73 on GitHub (Jun 22, 2025). ### ✅ Have you read and understood the above guidelines? yes ### 📜 What is the name of the script you are using? add-iptag ### 📂 What was the exact command used to execute the script? bash -c "$(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/tools/pve/add-iptag.sh)" ### ⚙️ What settings are you using? - [x] Default Settings - [ ] Advanced Settings ### 🖥️ Which Linux distribution are you using? _No response_ ### 📝 Provide a clear and concise description of the issue. The script only updates the IPs of the LXCs but not of my VMs ### 🔄 Steps to reproduce the issue. run the scripts within proxmox shell ### ❌ Paste the full error output (if available). ------ ### 🖼️ Additional context (optional). ![Image](https://github.com/user-attachments/assets/08bf8f83-413d-4627-b8bc-f8a86405f3db)
OVERLORD added the bug label 2026-02-04 23:16:19 +03:00
OVERLORD commented 2026-02-04 23:16:34 +03:00
Author
Owner
Copy Link

@MickLesk commented on GitHub (Jun 22, 2025):

When installed?

Service Output?

Config Output?

@MickLesk commented on GitHub (Jun 22, 2025): When installed? Service Output? Config Output?
OVERLORD commented 2026-02-04 23:16:37 +03:00
Author
Owner
Copy Link

@DST73 commented on GitHub (Jun 22, 2025):

Installed some time ago and updated today

Service output:

Image

Config file is empty

@DST73 commented on GitHub (Jun 22, 2025): Installed some time ago and updated today Service output: ![Image](https://github.com/user-attachments/assets/7ed4e7da-42f3-4ca3-9b52-f4b783fcfbc6) Config file is empty
OVERLORD commented 2026-02-04 23:16:41 +03:00
Author
Owner
Copy Link

@MickLesk commented on GitHub (Jun 22, 2025):

@DesertGamer maybe we should add an Guide, its everytime the Same?

@MickLesk commented on GitHub (Jun 22, 2025): @DesertGamer maybe we should add an Guide, its everytime the Same?
OVERLORD commented 2026-02-04 23:16:49 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jun 22, 2025):

This is an interesting behavior and I'm investigating it with a test machine soon.

@DesertGamer commented on GitHub (Jun 22, 2025): This is an interesting behavior and I'm investigating it with a test machine soon.
OVERLORD commented 2026-02-04 23:16:57 +03:00
Author
Owner
Copy Link

@Maxter3185 commented on GitHub (Jun 23, 2025):

Same error here. Installed the script maybe 2 weeks ago, but didn't have any VM at that moment so I didn't noticed that

@Maxter3185 commented on GitHub (Jun 23, 2025): Same error here. Installed the script maybe 2 weeks ago, but didn't have any VM at that moment so I didn't noticed that
OVERLORD commented 2026-02-04 23:17:05 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jun 23, 2025):

Tell me, please, are you trying to update the script instead of installing it from scratch?

@DesertGamer commented on GitHub (Jun 23, 2025): Tell me, please, are you trying to update the script instead of installing it from scratch?
OVERLORD commented 2026-02-04 23:17:15 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jun 23, 2025):

@Maxter3185, @DST73 The fact is that I checked the errors and found that the update algorithm is working incorrectly, I will make the necessary changes in the near future.

@DesertGamer commented on GitHub (Jun 23, 2025): @Maxter3185, @DST73 The fact is that I checked the errors and found that the update algorithm is working incorrectly, I will make the necessary changes in the near future.
OVERLORD commented 2026-02-04 23:17:20 +03:00
Author
Owner
Copy Link

@DST73 commented on GitHub (Jun 23, 2025):

Ok thank you :)

@DST73 commented on GitHub (Jun 23, 2025): Ok thank you :)
OVERLORD commented 2026-02-04 23:17:25 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jul 2, 2025):

Guys, I'm sorry, I didn't have time to do all this at all. I wrote it down for myself and will try to roll out the fix in a couple of hours.

@DesertGamer commented on GitHub (Jul 2, 2025): Guys, I'm sorry, I didn't have time to do all this at all. I wrote it down for myself and will try to roll out the fix in a couple of hours.
OVERLORD commented 2026-02-04 23:17:31 +03:00
Author
Owner
Copy Link

@Maxter3185 commented on GitHub (Jul 2, 2025):

Guys, I'm sorry, I didn't have time to do all this at all. I wrote it down for myself and will try to roll out the fix in a couple of hours.

@DesertGamer No worries! I've kinda fixed it yesterday (at least for me). Looking at the scripts I've found the newer version of the script moved the config files and the script from /opt/lxc-iptag to /opt/iptag, and the config file was missing in the second folder. Restarted the service and it worked.

What I've also found (I know I should open a new thread for this) is that this script is kinda heavy on the cpu time everytime it runs.

Image

This is from the netdata console. I don't know if this is normal behaviour or not.

@Maxter3185 commented on GitHub (Jul 2, 2025): > Guys, I'm sorry, I didn't have time to do all this at all. I wrote it down for myself and will try to roll out the fix in a couple of hours. @DesertGamer No worries! I've kinda fixed it yesterday (at least for me). Looking at the scripts I've found the newer version of the script moved the config files and the script from /opt/lxc-iptag to /opt/iptag, and the config file was missing in the second folder. Restarted the service and it worked. What I've also found (I know I should open a new thread for this) is that this script is kinda heavy on the cpu time everytime it runs. ![Image](https://github.com/user-attachments/assets/beac280b-bad1-4324-9c38-9fc08547bbb5) This is from the netdata console. I don't know if this is normal behaviour or not.
OVERLORD commented 2026-02-04 23:17:36 +03:00
Author
Owner
Copy Link

@DST73 commented on GitHub (Jul 3, 2025):

I tried the same and copied the iptag.conf and restarted the service

cp /opt/lxc-iptag/iptag.conf /opt/iptag

now my VMs get updated as well :)

@DST73 commented on GitHub (Jul 3, 2025): I tried the same and copied the iptag.conf and restarted the service `` cp /opt/lxc-iptag/iptag.conf /opt/iptag `` now my VMs get updated as well :)
OVERLORD commented 2026-02-04 23:17:43 +03:00
Author
Owner
Copy Link

@MickLesk commented on GitHub (Jul 3, 2025):

Maybe we remove the old Install fully and then Install the new one? Not migrating, Just Check If the old exist and full remove it

@DesertGamer

@MickLesk commented on GitHub (Jul 3, 2025): Maybe we remove the old Install fully and then Install the new one? Not migrating, Just Check If the old exist and full remove it @DesertGamer
OVERLORD commented 2026-02-04 23:17:46 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jul 3, 2025):

@MickLesk Look, initially I planned that people who would use the old script with this would be able to seamlessly switch to the new one, for this I wrote an update algorithm, after which I noticed that it was cleaned in the commit, perhaps not intentionally, which is why I just removed this algorithm in the latest version, as it turned out not all, I I would still add a seamless transition algorithm that would be useful to users who have previously used this script.

@DesertGamer commented on GitHub (Jul 3, 2025): @MickLesk Look, initially I planned that people who would use the old script with this would be able to seamlessly switch to the new one, for this I wrote an update algorithm, after which I noticed that it was cleaned in the commit, perhaps not intentionally, which is why I just removed this algorithm in the latest version, as it turned out not all, I I would still add a seamless transition algorithm that would be useful to users who have previously used this script.
OVERLORD commented 2026-02-04 23:17:50 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jul 3, 2025):

Ребята, извините, я вообще не успел все это сделать. Я записал его для себя и постараюсь выкатить исправление через пару часов.

@DesertGamer Без волнений! Я вроде как исправил это вчера (по крайней мере, для меня). Просматривая скрипты, я обнаружил, что новая версия скрипта переместила конфигурационные файлы и скрипт из /opt/lxc-iptag в /opt/iptag, а конфигурационный файл отсутствовал во второй папке. Перезапустил сервис и он заработал.

Что я также обнаружил (я знаю, что мне следует открыть новую тему для этого), так это то, что этот скрипт довольно сильно расходует время процессора каждый раз, когда он запускается.

Image

Это из консоли netdata. Я не знаю, нормальное это поведение или нет.

Please make a separate issue for this and I will definitely see what's the matter.

@DesertGamer commented on GitHub (Jul 3, 2025): > > Ребята, извините, я вообще не успел все это сделать. Я записал его для себя и постараюсь выкатить исправление через пару часов. > > [@DesertGamer](https://github.com/DesertGamer) Без волнений! Я вроде как исправил это вчера (по крайней мере, для меня). Просматривая скрипты, я обнаружил, что новая версия скрипта переместила конфигурационные файлы и скрипт из /opt/lxc-iptag в /opt/iptag, а конфигурационный файл отсутствовал во второй папке. Перезапустил сервис и он заработал. > > Что я также обнаружил (я знаю, что мне следует открыть новую тему для этого), так это то, что этот скрипт довольно сильно расходует время процессора каждый раз, когда он запускается. > > ![Image](https://github.com/user-attachments/assets/beac280b-bad1-4324-9c38-9fc08547bbb5) > > Это из консоли netdata. Я не знаю, нормальное это поведение или нет. Please make a separate issue for this and I will definitely see what's the matter.
OVERLORD commented 2026-02-04 23:17:57 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jul 3, 2025):

Thanks to your messages, the new version has become even better. After it is accepted, you will need to reinstall the script.

Now the function of saving the config has been added (this is optional, you can also overwrite it), as well as updating the service so that you do not worry about your current configuration.

@DesertGamer commented on GitHub (Jul 3, 2025): Thanks to your messages, the new version has become even better. After it is accepted, you will need to reinstall the script. Now the function of saving the config has been added (this is optional, you can also overwrite it), as well as updating the service so that you do not worry about your current configuration.
OVERLORD commented 2026-02-04 23:17:59 +03:00
Author
Owner
Copy Link

@DesertGamer commented on GitHub (Jul 4, 2025):

The update has been downloaded, please use the command:

bash -c "$(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/tools/pve/add-iptag.sh)"

You can save your old config using an additional option when updating, but I strongly recommend overwriting it.

@DesertGamer commented on GitHub (Jul 4, 2025): The update has been downloaded, please use the command: ``` bash -c "$(curl -fsSL https://raw.githubusercontent.com/community-scripts/ProxmoxVE/main/tools/pve/add-iptag.sh)" ``` You can save your old config using an additional option when updating, but I strongly recommend overwriting it.
OVERLORD commented 2026-02-04 23:18:06 +03:00
Author
Owner
Copy Link

@DST73 commented on GitHub (Jul 8, 2025):

Working for me. Thanks!

@DST73 commented on GitHub (Jul 8, 2025): Working for me. Thanks!
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Implemented in VED waiting push to Main
breaking change
bug
bug
bugfix
deferred
delete script
dependencies
enhancement
external
feature
github
help wanted
in project pipeline
invalid
investigation
json
maintenance
needs triage
new script
new script
nice to have
not a script issue
not planned
organization
pull-request
Mirrored from GitHub Pull Request
 question
refactor
rename script
security
update script
website
wontdo
🛑 Failure to comply with the guidelines
No Label bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
OVERLORD
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/ProxmoxVE#1138
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?