mirror of https://github.com/jumpserver/jumpserver
perf: push account without increasing version.
parent
02bdd0f07d
commit
3cd68ba0a9
|
@ -161,7 +161,7 @@ class ChangeSecretManager(AccountBasePlaybookManager):
|
||||||
return inventory_hosts
|
return inventory_hosts
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def need_change_account_version(account, recorder):
|
def require_update_version(account, recorder):
|
||||||
return account.secret != recorder.new_secret
|
return account.secret != recorder.new_secret
|
||||||
|
|
||||||
def on_host_success(self, host, result):
|
def on_host_success(self, host, result):
|
||||||
|
@ -176,6 +176,7 @@ class ChangeSecretManager(AccountBasePlaybookManager):
|
||||||
print("Account not found, deleted ?")
|
print("Account not found, deleted ?")
|
||||||
return
|
return
|
||||||
|
|
||||||
|
version_update_required = self.require_update_version(account, recorder)
|
||||||
account.secret = recorder.new_secret
|
account.secret = recorder.new_secret
|
||||||
account.date_updated = timezone.now()
|
account.date_updated = timezone.now()
|
||||||
|
|
||||||
|
@ -186,7 +187,7 @@ class ChangeSecretManager(AccountBasePlaybookManager):
|
||||||
try:
|
try:
|
||||||
recorder.save()
|
recorder.save()
|
||||||
account_update_fields = ['secret', 'date_updated']
|
account_update_fields = ['secret', 'date_updated']
|
||||||
if self.need_change_account_version(account, recorder):
|
if version_update_required:
|
||||||
account_update_fields.append('version')
|
account_update_fields.append('version')
|
||||||
account.save(update_fields=account_update_fields)
|
account.save(update_fields=account_update_fields)
|
||||||
break
|
break
|
||||||
|
|
|
@ -9,7 +9,7 @@ logger = get_logger(__name__)
|
||||||
class PushAccountManager(ChangeSecretManager, AccountBasePlaybookManager):
|
class PushAccountManager(ChangeSecretManager, AccountBasePlaybookManager):
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def need_change_account_version(account, recorder):
|
def require_update_version(account, recorder):
|
||||||
account.skip_history_when_saving = True
|
account.skip_history_when_saving = True
|
||||||
return False
|
return False
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue