Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Filter hosts in settings host-enum #1868

Merged
merged 7 commits into from
Jul 28, 2021
Merged

Conversation

iLLiCiTiT
Copy link
Member

@iLLiCiTiT iLLiCiTiT commented Jul 27, 2021

Description

Host enumerator should be used in most of cases when host names should be filled but sometimes it is required to limit host names to only few of them.

Changes

  • hosts-enum entity has new attribute hosts_filter where can be defined host names which will be available in enum
  • hosts filter is validated
    • at least one host must remain from all hosts
    • if host from filter is not available then warning is logged (Should raise and exception?)
  • added hosts filter for "workfiles on startup" where standalonepublisher is skipped

@iLLiCiTiT iLLiCiTiT self-assigned this Jul 27, 2021
@iLLiCiTiT iLLiCiTiT added backend type: enhancement Enhancements to existing functionality labels Jul 27, 2021
Copy link
Member

@jakubjezek001 jakubjezek001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

love this!

@jakubjezek001 jakubjezek001 removed the request for review from 64qam July 28, 2021 10:28
@iLLiCiTiT iLLiCiTiT merged commit f369392 into develop Jul 28, 2021
@mkolar mkolar deleted the feature/filter_hosts_in_enum branch August 10, 2021 15:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants