commit-confirm script not found
Closed, ResolvedPublicBUG
Actions

Assigned To

Authored By

	FileGo
	Apr 14 2021, 3:02 PM

Description

commit-confirm returns error in latest 1.4 rolling release:

$ commit-confirm
commit-confirm will automatically reboot in 10 minutes unless changes are confirmed.
Proceed? [y]y
Reboot scheduled for commit-confirm. Confirm your changes to cancel the reboot.
sudo: /opt/vyatta/sbin/commit-confirm-notify.py: command not found

$ ls /opt/vyatta/sbin/commit*
ls: cannot access '/opt/vyatta/sbin/commit*': No such file or directory

Details

Version: 1.4-rolling-202104132216
Is it a breaking change?: Unspecified (possibly destroys the router)

Related Objects
Search...

Status	Subtype	Assigned	Task
In progress	FEATURE REQUEST	None	T3355 Remove all remaining legacy Vyatta code
Resolved	FEATURE REQUEST	erkin	T3285 Schedule reboots through systemd-shutdownd instead of atd
Resolved	BUG	erkin	T3472 commit-confirm script not found

Event Timeline

FileGo created this task.Apr 14 2021, 3:02 PM

FileGo renamed this task from commit-confirm scrpt not found to commit-confirm script not found.

c-po assigned this task to erkin.Apr 14 2021, 4:05 PM

Is there a specific reason not to put it in vyos-1x instead?

I've merged the PR to alleviate the urgent problem, but I believe we may use the rewrite as an opportunity to also move at least some config management stuff to vyos-1x.

That's a good point. I just might do that.

erkin added a parent task: T3285: Schedule reboots through systemd-shutdownd instead of atd.Apr 16 2021, 11:07 PM

Restricted Repository Identity mentioned this in rVYOSONEX981153d79783: Merge pull request #812 from erkin/current.Apr 17 2021, 11:00 AM

erkin mentioned this in rVYOSONEX3604ccf91f06: T3472: Move over commit-confirm-notify.py from vyatta-config-mgmt.Apr 17 2021, 11:00 AM

c-po changed the task status from Open to Needs testing.Apr 17 2021, 7:03 PM

c-po triaged this task as Urgent! priority.

c-po merged a task: T3482: commit-confirm unable to call its notify python script.

c-po added subscribers: Alfa80, pasik.

erkin closed this task as Resolved.Apr 24 2021, 12:02 PM

c-po moved this task from Open to Finished on the VyOS 1.4 Sagitta board.May 24 2021, 7:38 PM

Restricted Repository Identity mentioned this in rVYOSONEX6c8c4e212b40: Merge pull request #2034 from erkin/current.Jun 10 2023, 5:46 AM

erkin mentioned this in rVYOSONEXfbcb22c1850c: T3472: Print warning when commit-confirm is run as regular user.Jun 10 2023, 5:46 AM