Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

new validation plugin: nodup #3461

Closed
markus2330 opened this issue Jul 27, 2020 · 2 comments
Closed

new validation plugin: nodup #3461

markus2330 opened this issue Jul 27, 2020 · 2 comments
Labels
enhancement floss2022w stale

Comments

@markus2330
Copy link
Contributor

markus2330 commented Jul 27, 2020 •
edited

Steps to Reproduce the Problem

sudo kdb mount --with-recommends /etc/hosts system/hosts hosts
sudo kdb set system/hosts/ipv4/test.complang.tuwien.ac.at 192.168.1.1
sudo kdb set system/hosts/ipv4/test.complang.tuwien.ac.at/alias 192.168.1.1
sudo kdb set system/hosts/ipv4/alias 192.168.1.2

Expected Result

An error that a host with the name alias already exists, e.g.:

Sorry, module nodup issued the error C03200:
Validation Semantic: name: system/hosts/ipv4/alias has the same base name as system/hosts/ipv4/test.complang.tuwien.ac.at/alias

Actual Result

192.168.1.2	alias
192.168.1.1	test.complang.tuwien.ac.at alias

Implementation Notes

A "nodup" plugin could check if any keyBaseName is identical and reject such configurations. Unfortunately, hosts already exhausts the number of allowed plugins, so for the concrete test case above to succeed we need to wait for #2963
@kodebach kodebach added the cm2022s for university course label Mar 3, 2021
@markus2330 markus2330 added the 13p thirteen points label Mar 4, 2021
This was referenced Mar 5, 2021
Closed
Merged
@markus2330 markus2330 added blocked something currently cannot be worked on because some other issue needs to be solved first and removed cm2022s for university course 13p thirteen points good first issue labels Sep 15, 2021
@markus2330 markus2330 added 3p three points cm2022s for university course and removed low priority blocked something currently cannot be worked on because some other issue needs to be solved first labels Mar 8, 2022
@markus2330 markus2330 changed the title nodup plugin new validation plugin: nodup Mar 8, 2022
@markus2330 markus2330 removed the 3p three points label Mar 8, 2022
@markus2330 markus2330 changed the title new validation plugin: nodup [CM T2] new validation plugin: nodup Mar 8, 2022
@flo91 flo91 changed the title [CM T2] new validation plugin: nodup new validation plugin: nodup Oct 5, 2022
@flo91 flo91 added floss2022w and removed cm2022s for university course labels Oct 5, 2022
@github-actions GitHub Actions
Copy link

github-actions bot commented Feb 3, 2024

I mark this stale as it did not have any activity for one year. I'll close it in two weeks if no further activity occurs. If you want it to be alive again, ping by writing a message here or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot added the stale label Feb 3, 2024
@github-actions GitHub Actions
Copy link

I closed this now because it has been inactive for more than one year. If I closed it by mistake, please do not hesitate to reopen it or create a new issue with the remainder of this issue.
Thank you for your contributions 💖

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Feb 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement floss2022w stale
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@markus2330 @flo91 @kodebach