Skip to content
This repository has been archived by the owner on Oct 27, 2022. It is now read-only.

Add label filtering to CTS #28

Merged
merged 12 commits into from
Jul 4, 2019
Merged

Conversation

sjanota
Copy link

@sjanota sjanota commented Jun 28, 2019

Description

Changes proposed in this pull request:

  • matchLabels in CTS changed to matchLabelExpressions and uses k8s syntax for label filtering
  • definition service filters TDs by labels
  • definition service ensures that unique TDs are returned
  • sample added

Related issue(s)

Fixes #7

config/samples/advanced/testsuite-select-by-labels.yaml Outdated Show resolved Hide resolved
pkg/fetcher/definition.go Outdated Show resolved Hide resolved
pkg/fetcher/definition.go Outdated Show resolved Hide resolved
pkg/fetcher/definition_test.go Outdated Show resolved Hide resolved
pkg/fetcher/definition_test.go Show resolved Hide resolved
pkg/fetcher/definition_test.go Show resolved Hide resolved
pkg/fetcher/definition_test.go Outdated Show resolved Hide resolved
pkg/fetcher/definition_test.go Show resolved Hide resolved
pkg/fetcher/definition.go Show resolved Hide resolved
pkg/fetcher/definition.go Show resolved Hide resolved
pkg/fetcher/definition_test.go Show resolved Hide resolved
pkg/fetcher/definition.go Outdated Show resolved Hide resolved
pkg/fetcher/definition.go Outdated Show resolved Hide resolved
@aszecowka
Copy link
Contributor

@sjanota please update Documentation

Co-Authored-By: Klaudia Grzondziel <35192450+klaudiagrz@users.noreply.github.com>
Copy link
Contributor

@aszecowka aszecowka left a comment

Choose a reason for hiding this comment

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

good job!

@sjanota sjanota merged commit dee8148 into kyma-incubator:master Jul 4, 2019
@sjanota sjanota deleted the label-selector branch July 4, 2019 11:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement Selective Testing
4 participants