Use smaller number of validators when running tests locally

[chatton]

Summary

Currently, we have hard coded values for validators to run in our tests.

These numbers are good for CI, but when iterating on tests locally can be pain unless manually changed to a lower number. We can lower these to 1, 0 for example when not running in CI.

in github actions, a CI=true environment variable is set, we could leverage this existing value to conditionally set these values.

---

For Admin Use

• Not duplicate issue
• Appropriate labels applied
• Appropriate contributors tagged/assigned

[expertdicer]

I think this should be closed as I saw that someone already do this

[DimitrisJim]

I think this should be closed as I saw that someone already do this

Nice find! This was indeed addressed in aabb07a of #3079:

ibc-go/e2e/testsuite/testsuite.go

Lines 565 to 573 in aabb07a

	// getValidatorsAndFullNodes returns the number of validators and full nodes respectively that should be used for
	// the test. If the test is running in CI, more nodes are used, when running locally a single node is used to
	// use less resources and allow the tests to run faster.
	func getValidatorsAndFullNodes() (int, int) {
	if testconfig.IsCI() {
	return 4, 1
	}
	return 1, 0
	}