diff options
| author | Viacheslav Hletenko <v.gletenko@vyos.io> | 2023-12-11 21:19:24 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-12-11 21:19:24 +0200 |
| commit | bc460276da84c14b4cb541f28f87fb8a96867611 (patch) | |
| tree | 5a5a944c88d415c7d47a91cdd271ec37fc53ec92 | |
| parent | d519d7dcd141cb6c57811f75f9a045c60b02b105 (diff) | |
| parent | 22f53edfcb8bf9b3cf06edf1cb376e841ad854fb (diff) | |
| download | vyos-1x-bc460276da84c14b4cb541f28f87fb8a96867611.tar.gz vyos-1x-bc460276da84c14b4cb541f28f87fb8a96867611.zip | |
Merge pull request #2616 from vyos/mergify/bp/sagitta/pr-2613
T5812: report actual number of revisions instead of max (backport #2613)
| -rw-r--r-- | python/vyos/config_mgmt.py | 7 |
1 files changed, 4 insertions, 3 deletions
diff --git a/python/vyos/config_mgmt.py b/python/vyos/config_mgmt.py index 950f14d4f..fd0fa7a75 100644 --- a/python/vyos/config_mgmt.py +++ b/python/vyos/config_mgmt.py @@ -125,6 +125,7 @@ class ConfigMgmt: get_first_key=True) self.max_revisions = int(d.get('commit_revisions', 0)) + self.num_revisions = 0 self.locations = d.get('commit_archive', {}).get('location', []) self.source_address = d.get('commit_archive', {}).get('source_address', '') @@ -233,7 +234,7 @@ Proceed ?''' msg = '' if not self._check_revision_number(rev): - msg = f'Invalid revision number {rev}: must be 0 < rev < {self.max_revisions}' + msg = f'Invalid revision number {rev}: must be 0 < rev < {self.num_revisions}' return msg, 1 prompt_str = 'Proceed with reboot ?' @@ -560,8 +561,8 @@ Proceed ?''' return len(l) def _check_revision_number(self, rev: int) -> bool: - maxrev = self._get_number_of_revisions() - if not 0 <= rev < maxrev: + self.num_revisions = self._get_number_of_revisions() + if not 0 <= rev < self.num_revisions: return False return True |