#8219 ipatests: unify editing of sssd.conf
Closed: fixed 4 years ago by sorlov. Opened 4 years ago by sorlov.

When tests need to modify sssd.conf they currently do in one of three ways
with tasks.modify_sssd_conf() - this function validates changes against schema but allows editing only domain sections of config files
with tasks.remote_ini_file() which can change any section but does not check for logical errors like wrong option name
* with direct file editing using sed

We should have a single tool for this task with following properties
it checks changes with schema
it has a simple and convenient interface
* it allows to make arbitrary changes to sssd.conf if they conform to schema.

Whenever we need to produce invalid config file for the testing purposes we still can use remote_ini_file()


master:

  • 888c7ba ipatests: update docstring to reflect changes in FileBackup.restore()
  • 9450aef ipatests: replace utility for editing sssd.conf
  • 3dd679b ipatests: use remote_sssd_config to modify sssd.conf
  • e01e7fe ipatests: remove invalid parameter from sssd.conf

Metadata Update from @sorlov:
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

4 years ago

ipa-4-6:

  • 714b61f ipatests: refactor FileBackup helper
  • e25b10e ipatests: update docstring to reflect changes in FileBackup.restore()
  • 7f18f08 ipatests: replace utility for editing sssd.conf
  • aff397b ipatests: use remote_sssd_config to modify sssd.conf
  • 551dabe ipatests: remove invalid parameter from sssd.conf
  • 044748b ipatests: temporary disable pylint check no-name-in-module

ipa-4-7:

  • aa72208 ipatests: update docstring to reflect changes in FileBackup.restore()
  • 7a4d307 ipatests: replace utility for editing sssd.conf
  • 90d8863 ipatests: use remote_sssd_config to modify sssd.conf
  • c89dbf2 ipatests: remove invalid parameter from sssd.conf

ipa-4-8:

  • 9cb8984 ipatests: update docstring to reflect changes in FileBackup.restore()
  • 5ff9b6e ipatests: replace utility for editing sssd.conf
  • 32584ed ipatests: use remote_sssd_config to modify sssd.conf
  • a169572 ipatests: remove invalid parameter from sssd.conf

ipa-4-6:

  • 46b9139 ipatests: remove workaround for pylint error no-name-in-module

Login to comment on this ticket.

Metadata