freeipa

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

Closed: fixed
Reopen Issue

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()

PR: https://github.com/freeipa/freeipa/pull/4313

master:

  • 888c7ba938686a5fbc1c359e43a8e79989dee850 ipatests: update docstring to reflect changes in FileBackup.restore()
  • 9450aef75f2ac064ea182c942d39400287a38bab ipatests: replace utility for editing sssd.conf
  • 3dd679b31dcdab1447b9d19c94019aaf7a0db071 ipatests: use remote_sssd_config to modify sssd.conf
  • e01e7fe6c64ef0d75c7256cc4fa631b0c296edd2 ipatests: remove invalid parameter from sssd.conf

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

ipa-4-6:

  • 714b61f3605f53ecde73dd7e3d23ae92d219f926 ipatests: refactor FileBackup helper
  • e25b10ef3a4da973300cd7d888f1506291fa882d ipatests: update docstring to reflect changes in FileBackup.restore()
  • 7f18f08ca607fdf3b730a6b5e66dc97535007259 ipatests: replace utility for editing sssd.conf
  • aff397b9ef09b1f2dc6c02a6bb85b96fb16b9ded ipatests: use remote_sssd_config to modify sssd.conf
  • 551dabe5f933475e4609b6b23eb1200dec90945b ipatests: remove invalid parameter from sssd.conf
  • 044748b5724f408643fe9f95c3a63d29ca646002 ipatests: temporary disable pylint check no-name-in-module

ipa-4-7:

  • aa722083cea3d3d238d80699de21201242888116 ipatests: update docstring to reflect changes in FileBackup.restore()
  • 7a4d30717e1d4a65e0e9277e6a52e369df3989bf ipatests: replace utility for editing sssd.conf
  • 90d88634ef0b05fdfe6879e03c7c44cd2246668d ipatests: use remote_sssd_config to modify sssd.conf
  • c89dbf2440ef0da37b1f766f7a424408b3202966 ipatests: remove invalid parameter from sssd.conf

ipa-4-8:

  • 9cb8984112ff31721b71dcdd4febcc23c2641691 ipatests: update docstring to reflect changes in FileBackup.restore()
  • 5ff9b6e2a506c3ef1179655ae2d2e479005ec99e ipatests: replace utility for editing sssd.conf
  • 32584ed34f466e8f474e22d778e3e964d0fcd2c4 ipatests: use remote_sssd_config to modify sssd.conf
  • a1695722125674204b6e880b6ac652d78b783c88 ipatests: remove invalid parameter from sssd.conf

ipa-4-6:

  • 46b9139ac9ecbbd89495239e380982514db3a5f4 ipatests: remove workaround for pylint error no-name-in-module
Metadata
None
None
None
None
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.