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

Validate Node Name #48

Merged
merged 4 commits into from
Jun 8, 2023
Merged

Validate Node Name #48

merged 4 commits into from
Jun 8, 2023

Commits on Jun 1, 2023

  1. Validate CR/node name from the node names in the cluster 

    We want to validate that CR has been created correctly. Also UT functionality
    @razo7
    razo7 committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    91969cf View commit details
    Browse the repository at this point in the history

  2. Add verbs for nodes resources 

    We need to add three new rbac, get, list and watch, for listing the cluster nodes
    @razo7
    razo7 committed Jun 1, 2023
    Configuration menu
    Copy the full SHA
    5b00731 View commit details
    Browse the repository at this point in the history

Commits on Jun 7, 2023

  1. Better error handeling for invalid node name 

    Delete redundant error file, and returning bool with error for IsNodeNameValid function
    @razo7
    razo7 committed Jun 7, 2023
    Configuration menu
    Copy the full SHA
    eec5f74 View commit details
    Browse the repository at this point in the history

Commits on Jun 8, 2023

  1. Fix typos and check error message 

    Updating import names, and check error message of 1 UT
    @razo7
    razo7 committed Jun 8, 2023
    Configuration menu
    Copy the full SHA
    97e101d View commit details
    Browse the repository at this point in the history