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.

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

test #271

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

test #271

Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 24 additions & 24 deletions .github/workflows/actions.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,27 +47,27 @@ jobs:
run: make install
- name: test
run: make lint
# TODO: restore this test
# integration-tests:
# runs-on: ubuntu-18.04
# defaults:
# run:
# working-directory: go/src/github.com/lyft/flinkk8soperator
# env:
# GOPATH: "/home/runner/work/flinkk8soperator/flinkk8soperator/go/"
# steps:
# - name: checkout
# uses: actions/checkout@v2
# with:
# fetch-depth: 1
# path: go/src/github.com/lyft/flinkk8soperator
# - name: install go
# uses: actions/setup-go@v2
# with:
# go-version: 1.12
# - name: install
# run: integ/install.sh
# - name: setup
# run: integ/setup.sh
# - name: test
# run: sudo "PATH=$PATH" "GOPATH=$GOPATH" integ/test.sh
# TODO: restore this test
integration-tests:
runs-on: ubuntu-18.04
defaults:
run:
working-directory: go/src/github.com/lyft/flinkk8soperator
env:
GOPATH: "/home/runner/work/flinkk8soperator/flinkk8soperator/go/"
steps:
- name: checkout
uses: actions/checkout@v2
with:
fetch-depth: 1
path: go/src/github.com/lyft/flinkk8soperator
- name: install go
uses: actions/setup-go@v2
with:
go-version: 1.12
- name: install
run: integ/install.sh
- name: setup
run: integ/setup.sh
- name: test
run: sudo "PATH=$PATH" "GOPATH=$GOPATH" "GOGC=off" integ/test.sh
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ FlinkK8sOperator is a [Kubernetes operator](https://coreos.com/operators/) that

## Project Status

*Beta*
*Beta**

The operator is in use for some less-critical jobs at Lyft. At this point the focus is on testing and stability. While in
Beta, we will attempt to limit the number of backwards-incompatible changes, but they may still occur as necessary.
Expand Down
5 changes: 2 additions & 3 deletions integ/checkpoint_failure_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,6 @@ package integ

import (
"fmt"
"io/ioutil"
"os"
"time"

"github.com/lyft/flinkk8soperator/pkg/apis/app/v1beta1"
Expand Down Expand Up @@ -65,7 +63,7 @@ func failingJobTest(s *IntegSuite, c *C, testName string, causeFailure func()) {
log.Info("All pods torn down")
}

// Tests that we correctly handle updating a job with task failures
/* Tests that we correctly handle updating a job with task failures
func (s *IntegSuite) TestJobWithTaskFailures(c *C) {
failingJobTest(s, c, "taskfailure", func() {
f, err := os.OpenFile(s.Util.CheckpointDir+"/fail", os.O_RDONLY|os.O_CREATE, 0666)
Expand All @@ -82,3 +80,4 @@ func (s *IntegSuite) TestCheckpointTimeout(c *C) {
c.Assert(err, IsNil)
})
}
*/
303 changes: 0 additions & 303 deletions integ/job_cancellation_test.go

This file was deleted.

Loading