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

General improvements #171

Merged
merged 6 commits into from Jul 23, 2021
Merged

General improvements #171

merged 6 commits into from Jul 23, 2021

Commits on Jul 23, 2021

  1. src/client: remove unused methods from Kubernetes interface 

    Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    61afa63 View commit details
    Browse the repository at this point in the history

  2. src/ksm/client: styling improvements 

    Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    15494aa View commit details
    Browse the repository at this point in the history

  3. src/ksm/client/discovery_test.go: automatically fix formatting 

    Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    2f47959 View commit details
    Browse the repository at this point in the history

  4. src/ksm/client: fix panicking test when KSM discovery is broken 

    Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    2054db9 View commit details
    Browse the repository at this point in the history

  5. cmd/kubernetes-discovery/main.go: improvements 

    * Group flag variables together
* Consistently use only single logger
* Improve error messages

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    01cef1b View commit details
    Browse the repository at this point in the history

  6. e2e/cmd: add sanity check for cluster where tests are executed 

    If there is more than one node on the cluster, test result may end with
false negatives.

Signed-off-by: Mateusz Gozdek <mgozdek@microsoft.com>
    @invidian
    invidian committed Jul 23, 2021
    Copy the full SHA
    ea7c570 View commit details
    Browse the repository at this point in the history