Skip to content

Commit

Permalink
use ubuntu latest
Browse files Browse the repository at this point in the history
Signed-off-by: Yashash H L <[email protected]>
  • Loading branch information
yhl25 committed Sep 13, 2024
1 parent 2fb827f commit 02fca5d
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 39 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ on:
jobs:
ui:
name: UI
runs-on: ubuntu-24.04
runs-on: ubuntu-latest
timeout-minutes: 6
steps:
- name: Checkout code
Expand Down Expand Up @@ -131,7 +131,7 @@ jobs:

lint:
name: Lint
runs-on: ubuntu-24.04
runs-on: ubuntu-latest
timeout-minutes: 10
env:
GOPATH: /home/runner/go
Expand Down Expand Up @@ -184,7 +184,7 @@ jobs:

e2e-tests:
name: E2E Tests
runs-on: ubuntu-24.04
runs-on: ubuntu-latest
needs: [ build-rust-amd64 ]
timeout-minutes: 20
strategy:
Expand Down
34 changes: 0 additions & 34 deletions pkg/sdkclient/options.go
Original file line number Diff line number Diff line change
Expand Up @@ -16,17 +16,9 @@ limitations under the License.

package sdkclient

import (
"time"

"github.com/numaproj/numaflow/pkg/apis/numaflow/v1alpha1"
)

type Options struct {
udsSockAddr string
maxMessageSize int
readBatchSize int
readTimeout time.Duration
}

// UdsSockAddr returns the UDS sock addr.
Expand All @@ -39,23 +31,11 @@ func (o *Options) MaxMessageSize() int {
return o.maxMessageSize
}

// ReadBatchSize returns the read batch size.
func (o *Options) ReadBatchSize() int {
return o.readBatchSize
}

// ReadTimeout returns the read timeout.
func (o *Options) ReadTimeout() time.Duration {
return o.readTimeout
}

// DefaultOptions returns the default options.
func DefaultOptions(address string) *Options {
return &Options{
maxMessageSize: DefaultGRPCMaxMessageSize,
udsSockAddr: address,
readBatchSize: v1alpha1.DefaultReadBatchSize,
readTimeout: v1alpha1.DefaultReadTimeout,
}
}

Expand All @@ -75,17 +55,3 @@ func WithMaxMessageSize(size int) Option {
opts.maxMessageSize = size
}
}

// WithReadBatchSize sets the read batch size.
func WithReadBatchSize(size int) Option {
return func(opts *Options) {
opts.readBatchSize = size
}
}

// WithReadTimeout sets the read timeout.
func WithReadTimeout(timeout time.Duration) Option {
return func(opts *Options) {
opts.readTimeout = timeout
}
}
1 change: 0 additions & 1 deletion rust/monovertex/build.rs
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
fn main() {
tonic_build::configure()
.build_server(true)
.protoc_arg("--experimental_allow_proto3_optional")
.compile(
&[
"proto/source.proto",
Expand Down
3 changes: 2 additions & 1 deletion test/e2e/functional_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ func (s *FunctionalSuite) TestCreateSimplePipeline() {
w := s.Given().Pipeline("@testdata/simple-pipeline.yaml").
When().
CreatePipelineAndWait()
defer w.DeletePipelineAndWait()
//defer w.DeletePipelineAndWait()
pipelineName := "simple-pipeline"

w.Expect().
Expand Down Expand Up @@ -146,6 +146,7 @@ func (s *FunctionalSuite) TestCreateSimplePipeline() {
assert.Fail(s.T(), "timed out waiting for rate to be calculated")
}
timer.Stop()
time.Sleep(300 * time.Second)
}

func (s *FunctionalSuite) TestUDFFiltering() {
Expand Down

0 comments on commit 02fca5d

Please sign in to comment.