LimitBytes:   &limitBytesGreaterThan1,
			SinceSeconds: &sinceSecondsGreaterThan1,
		},
	}, {
		name: "Invalid podLogOptions with zero or negative LimitBytes",
		podLogOptions: v1.PodLogOptions{
			TailLines:    &positiveLine,
			LimitBytes:   &limitBytesLessThan1,
			SinceSeconds: &sinceSecondsGreaterThan1,
		},
	}, {
		name: "Invalid podLogOptions with zero or negative SinceSeconds",
		podLogOptions: v1.PodLogOptions{

	}
	switch {
	case opts.SinceSeconds != nil && opts.SinceTime != nil:
		allErrs = append(allErrs, field.Forbidden(field.NewPath(""), "at most one of `sinceTime` or `sinceSeconds` may be specified"))
	case opts.SinceSeconds != nil:
		if *opts.SinceSeconds < 1 {
			allErrs = append(allErrs, field.Invalid(field.NewPath("sinceSeconds"), *opts.SinceSeconds, "must be greater than 0"))
		}
	}
	return allErrs
}

{
  "kind": "PodLogOptions",
  "apiVersion": "v1",
  "container": "containerValue",
  "follow": true,
  "previous": true,
  "sinceSeconds": 4,
  "sinceTime": "2005-01-01T01:01:01Z",
  "timestamps": true,
  "tailLines": 7,
  "limitBytes": 8,
  "insecureSkipTLSVerifyBackend": true

apiVersion: v1
container: containerValue
follow: true
insecureSkipTLSVerifyBackend: true
kind: PodLogOptions
limitBytes: 8
previous: true
sinceSeconds: 4
sinceTime: "2005-01-01T01:01:01Z"
tailLines: 7

apiVersion: v1
container: containerValue
follow: true
insecureSkipTLSVerifyBackend: true
kind: PodLogOptions
limitBytes: 8
previous: true
sinceSeconds: 4
sinceTime: "2005-01-01T01:01:01Z"
tailLines: 7

{
  "kind": "PodLogOptions",
  "apiVersion": "v1",
  "container": "containerValue",
  "follow": true,
  "previous": true,
  "sinceSeconds": 4,
  "sinceTime": "2005-01-01T01:01:01Z",
  "timestamps": true,
  "tailLines": 7,
  "limitBytes": 8,
  "insecureSkipTLSVerifyBackend": true

)

func TestPodLogOptions(t *testing.T) {
	sinceSeconds := int64(1)
	sinceTime := metav1.NewTime(time.Date(2000, 1, 1, 12, 34, 56, 0, time.UTC).Local())
	tailLines := int64(2)
	limitBytes := int64(3)

	versionedLogOptions := &v1.PodLogOptions{
		Container:    "mycontainer",
		Follow:       true,
		Previous:     true,
		SinceSeconds: &sinceSeconds,
		SinceTime:    &sinceTime,
		Timestamps:   true,

	store := &genericregistry.Store{
		Storage: genericregistry.DryRunnableStorage{Storage: s},
	}
	logRest := &LogREST{Store: store, KubeletConn: nil}

	negativeOne := int64(-1)
	testCases := []*api.PodLogOptions{
		{SinceSeconds: &negativeOne},
		{TailLines: &negativeOne},
	}

	for _, tc := range testCases {
		_, err := logRest.Get(genericapirequest.NewDefaultContext(), "test", tc)
		if !errors.IsInvalid(err) {

		params.Add("follow", "true")
	}
	if opts.Previous {
		params.Add("previous", "true")
	}
	if opts.Timestamps {
		params.Add("timestamps", "true")
	}
	if opts.SinceSeconds != nil {
		params.Add("sinceSeconds", strconv.FormatInt(*opts.SinceSeconds, 10))
	}
	if opts.SinceTime != nil {
		params.Add("sinceTime", opts.SinceTime.Format(time.RFC3339))
	}
	if opts.TailLines != nil {

	}
	switch {
	case opts.SinceSeconds != nil && opts.SinceTime != nil:
		allErrs = append(allErrs, field.Forbidden(field.NewPath(""), "at most one of `sinceTime` or `sinceSeconds` may be specified"))
	case opts.SinceSeconds != nil:
		if *opts.SinceSeconds < 1 {
			allErrs = append(allErrs, field.Invalid(field.NewPath("sinceSeconds"), *opts.SinceSeconds, "must be greater than 0"))
		}
	}
	return allErrs