Skip to content
This repository has been archived by the owner on Mar 11, 2021. It is now read-only.

Stabilize/disable tests related to if_modified_since_header #1655

Closed
sbose78 opened this issue Sep 21, 2017 · 2 comments
Closed

Stabilize/disable tests related to if_modified_since_header #1655

sbose78 opened this issue Sep 21, 2017 · 2 comments
Assignees

Comments

@sbose78
Copy link
Member

sbose78 commented Sep 21, 2017

  --- FAIL: TestChildren/not_modified_using_if_modified_since_header (0.01s)
        workitem_testing.go:849: invalid response status code: got 200, expected 304

https://ci.centos.org/job/devtools-fabric8-wit/876/

@joshuawilson
Copy link
Member

What is the context for this? Is this a task or an issue?

@xcoulon
Copy link
Contributor

xcoulon commented Sep 21, 2017

@joshuawilson when running the platform tests locally or on Jenkins, this kind of random error occurs from time to time. Usually, kicking a new build get rids of the failure, but we should get this stabilised for good, instead.

xcoulon added a commit to xcoulon/fabric8-wit that referenced this issue Oct 2, 2017
…services#1655)

Using the `Last-Modified` response header when possible
Also, making sure that the list of spaces is always ordered by
`updated_at DESC` to get the most recents spaces first.

Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
xcoulon added a commit to xcoulon/fabric8-wit that referenced this issue Oct 2, 2017
…services#1655)

Using the `Last-Modified` response header when possible
Also, making sure that the list of spaces is always ordered by
`updated_at DESC` to get the most recents spaces first.

Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
xcoulon added a commit to xcoulon/fabric8-wit that referenced this issue Oct 2, 2017
…services#1655)

Using the `Last-Modified` response header when possible
Also, making sure that the list of spaces is always ordered by
`updated_at DESC` to get the most recents spaces first.

Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
xcoulon added a commit to xcoulon/fabric8-wit that referenced this issue Oct 2, 2017
…services#1655)

Using the `Last-Modified` response header when possible
Also, making sure that the list of spaces is always ordered by
`updated_at DESC` to get the most recents spaces first.

Fixes fabric8-services#1655

Signed-off-by: Xavier Coulon <xcoulon@redhat.com>
@xcoulon xcoulon closed this as completed in 43bd1d6 Oct 3, 2017
xcoulon pushed a commit that referenced this issue Jan 15, 2018
…ion)

- add new API endpoints:

/apps/environments/{envName}
/apps/environments/{envName}/applications/{appName}/pods
/apps/spaces/{spaceID}
/apps/spaces/{spaceID}/environments
/apps/spaces/{spaceID}/applications/{appName}
/apps/spaces/{spaceID}/applications/{appName}/deployments/{deployName}
/apps/spaces/{spaceID}/applications/{appName}/deployments/{deployName}/control
/apps/spaces/{spaceID}/applications/{appName}/deployments/{deployName}/stats
/apps/spaces/{spaceID}/applications/{appName}/deployments/{deployName}/statseries

- Strip nested vendor directories during glide install
- Also specify --strip-vcs when calling "glide install --strip-vendor" 
  (this is required because the Jenkins build uses an earlier version of Glide)

- Many, many changes incorporated from code review by xcoulon

- Update Glide config to newest fabric8-auth.
- Use generated Auth amnd WIT clients
- Add initial unit tests

- environment variable FABRIC8_WIT_API_URL added to override WIT URL (defaults to localhost) for local testing

- Add github.com/ajg/form package to try to fix Jenkins build

- Omit account/tenant generated code from unit tests

Co-authored-by: Elliot Baron <ebaron@redhat.com>

fixes openshiftio/openshift.io#1655
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants