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

Add test for rook-ceph which consumes packet_fluo pipeline #1084

Merged
merged 2 commits into from
Oct 30, 2020

Conversation

surajssd
Copy link
Member

No description provided.

@surajssd surajssd force-pushed the surajssd/add-test-code-rook-ceph branch 3 times, most recently from 7c02ccf to 8440d8a Compare October 21, 2020 07:47
@surajssd surajssd force-pushed the surajssd/add-test-code-rook-ceph branch 2 times, most recently from fc456ec to be4ff23 Compare October 29, 2020 08:34
@surajssd surajssd marked this pull request as ready for review October 29, 2020 08:34
@surajssd surajssd changed the title Add test for packet_fluo pipeline Add test for rook-ceph which consumes packet_fluo pipeline Oct 29, 2020
@surajssd surajssd requested review from invidian and ipochi October 29, 2020 11:51
// See the License for the specific language governing permissions and
// limitations under the License.

// +build packet_fluo
Copy link
Member

Choose a reason for hiding this comment

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

What has Rook Ceph to do with packet_fluo ? It seems confusing.

Copy link
Member Author

Choose a reason for hiding this comment

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

That's a pipeline build tag.

Copy link
Member

Choose a reason for hiding this comment

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

I know what it is. I'm saying the name is confusing to me in the context of rook ceph.

test/components/rook/rook-ceph_test.go Outdated Show resolved Hide resolved
test/components/rook/rook-ceph_test.go Show resolved Hide resolved
test/components/rook/rook-ceph_test.go Show resolved Hide resolved
test/components/rook/rook-ceph_test.go Show resolved Hide resolved
@surajssd surajssd force-pushed the surajssd/add-test-code-rook-ceph branch from be4ff23 to 78052d8 Compare October 30, 2020 07:50
Add the tag to `Makefile` and `.golangci.yml`.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
Add tests to verify various control plane components (pods) of rook-ceph
are up and running.

Signed-off-by: Suraj Deshmukh <suraj@kinvolk.io>
@surajssd surajssd force-pushed the surajssd/add-test-code-rook-ceph branch from 78052d8 to 8a236c9 Compare October 30, 2020 11:25
@surajssd surajssd requested a review from invidian October 30, 2020 11:58
Copy link
Member

@invidian invidian left a comment

Choose a reason for hiding this comment

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

OK

@surajssd surajssd merged commit d3d9768 into master Oct 30, 2020
@surajssd surajssd deleted the surajssd/add-test-code-rook-ceph branch October 30, 2020 13:29
invidian added a commit that referenced this pull request Nov 2, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
invidian added a commit that referenced this pull request Nov 5, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
invidian added a commit that referenced this pull request Nov 6, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
invidian added a commit that referenced this pull request Nov 6, 2020
It should've been added in #1084.

Closes #397

Signed-off-by: Mateusz Gozdek <mateusz@kinvolk.io>
@invidian invidian mentioned this pull request Dec 7, 2020
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.

2 participants