Tariq Hasan

Introduced error constants and replaced reflect with cmp

23

**What this PR does / why we need it**: Following the suggestion https://github.com/kubeflow/katib/pull/2281#pullrequestreview-1926741952 from @tenzen-y this code change provides a more thorough comparison of errors through the introduction of error...

Make test fields private in Go unit tests

2

**What this PR does / why we need it**: Following the [suggestion](https://github.com/kubeflow/katib/pull/2289#discussion_r1566264061) to keep the test fields private within Go unit tests, this PR implements the necessary changes. The unit...

Replaced hpcloud with nxadm for tail package in Go

1

**What this PR does / why we need it**: This issue helps to migrate the dependency of the `tail` package in `katib` from https://github.com/hpcloud/tail to https://github.com/nxadm/tail. This change is required...

### What you would like to be added? We need to add unit test for the `tune` function in the `katib_client` module. ### Why is this needed? Parent issue: [Add...

Upgrade node from 12 to 16

22

# Motivation This PR is an offshoot from https://github.com/kubeflow/katib/pull/2384 in `kubeflow/katib`. https://github.com/kubeflow/katib/pull/2384 attempts to resolve errors in e2e tests for cypress in Katib UI and requires that the node version...

fix: Resolve errors in e2e tests for cypress in Katib UI

26

**What this PR does / why we need it**: This PR fixed the errors in e2e tests for cypress. **Which issue(s) this PR fixes** _(optional, in `fixes #(, fixes #,...

Replace `go mock` with `client-go fake client` in unit tests

23

### What you would like to be added? `go mock` should be replaced with `client-go fake client` in unit tests. ### Why is this needed? Reference: https://github.com/kubeflow/katib/pull/2289#discussion_r1718769690 ### Love this...