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

Revert "List work items part of child iterations (#2146)" #2168

Merged
merged 1 commit into from Jul 13, 2018

Conversation

baijum
Copy link
Contributor

@baijum baijum commented Jul 13, 2018

This reverts commit 68996d1.

This is required to address openshiftio/openshift.io#3974

@jarifibrahim
Copy link
Member

[test]

Copy link
Collaborator

@kwk kwk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@codecov-io
Copy link

codecov-io commented Jul 13, 2018

Codecov Report

Merging #2168 into master will decrease coverage by 0.04%.
The diff coverage is 88.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2168      +/-   ##
==========================================
- Coverage   69.46%   69.41%   -0.05%     
==========================================
  Files         166      166              
  Lines       15446    15422      -24     
==========================================
- Hits        10729    10705      -24     
+ Misses       3748     3747       -1     
- Partials      969      970       +1
Impacted Files Coverage Δ
query/query.go 56.69% <100%> (ø) ⬆️
search/search_repository.go 75.77% <88.23%> (-0.95%) ⬇️
controller/workitem.go 78.64% <0%> (-0.62%) ⬇️
workitem/workitem_repository.go 69.9% <0%> (+0.27%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update eae146f...84028f7. Read the comment docs.

@baijum baijum merged commit d198813 into fabric8-services:master Jul 13, 2018
kwk added a commit to openshiftio/saas-openshiftio that referenced this pull request Jul 16, 2018
**commit** fabric8-services/fabric8-wit@cefe36a
**Author:** Rohit Kumar Rai <rohitkrai03@gmail.com>
**Date:**   Wed Jul 11 17:54:40 2018 +0530

Changed SHA checksum for dep-darwin-amd64 and setting UNAME_S variable (fabric8-services/fabric8-wit#2163)

- For macOS dep package was updated with a new SHA. This SHA value is checked in `Makefile` with a hardcoded SHA checksum to verify dep package.
- Updated the SHA value to new one.
- Added initialization of `$(UNAME_S)` variable `UNAME_S=$(shell uname -s)` since it was being used but never set which lead to explicitly exporting the variable from shell.

Similar PR for fabric8-auth - fabric8-services/fabric8-auth#549

Note: We can probably look into automating the process where SHA value is fetched dynamically instead of hard coding.

**commit** fabric8-services/fabric8-wit@f13e094
**Author:** Elliott Baron <ebaron@redhat.com>
**Date:**   Wed Jul 11 13:46:08 2018 -0400

Add parameter to Delete Space to skip deleting OpenShift resources (fabric8-services/fabric8-wit#2121)

When a user resets their environment, the front-end makes calls to the Delete Space API and Clean Tenant API. It makes these calls asynchronously, and due to both APIs acting on the same resources, I suspect this is the reason we are seeing a variety of errors in openshiftio/openshift.io#3500.

Since the Clean Tenant API cleans out the user's entire namespaces, it is not necessary in this case for Delete Space to delete anything from OpenShift. This PR adds an optional parameter skipCluster, to the Delete Space API, which if true, will not attempt to delete any deployments from OpenShift. The front-end could then use this parameter only when resetting the user's environment. An alternative would be for the front-end to synchronize between deleting spaces and calling Clean Tenant, but this would be less efficient.

Fixes (partially): openshiftio/openshift.io#3500

**commit** fabric8-services/fabric8-wit@cb85aa7
**Author:** Ibrahim Jarif <jarifibrahim@gmail.com>
**Date:**   Fri Jul 13 12:57:43 2018 +0530

Refactor search_blackbox_test.go (fabric8-services/fabric8-wit#2148)

**commit** eae146f7d3cdf1d6c40588608e60d688aaf6ad83
**Author:** Dhriti Shikhar <dhriti.shikhar.rokz@gmail.com>
**Date:**   Fri Jul 13 16:55:40 2018 +0530

Increase paging limit (fabric8-services/fabric8-wit#2166)

**commit** fabric8-services/fabric8-wit@d198813
**Author:** Baiju Muthukadan <baiju.m.mail@gmail.com>
**Date:**   Fri Jul 13 17:46:12 2018 +0530

Revert "List work items part of child iterations (fabric8-services/fabric8-wit#2146)" (fabric8-services/fabric8-wit#2168)

This reverts commit 68996d1555a29a9ef310403403855b47559d5a71.


This is required to address #3974

**commit** fabric8-services/fabric8-wit@a4d9061
**Author:** Michael Kleinhenz <kleinhenz@redhat.com>
**Date:**   Fri Jul 13 16:24:37 2018 +0200

feat(boardview): Board View for WIT. (fabric8-services/fabric8-wit#2111)
aslakknutsen pushed a commit to openshiftio/saas-openshiftio that referenced this pull request Jul 17, 2018
**commit** fabric8-services/fabric8-wit@cefe36a
**Author:** Rohit Kumar Rai <rohitkrai03@gmail.com>
**Date:**   Wed Jul 11 17:54:40 2018 +0530

Changed SHA checksum for dep-darwin-amd64 and setting UNAME_S variable (fabric8-services/fabric8-wit#2163)

- For macOS dep package was updated with a new SHA. This SHA value is checked in `Makefile` with a hardcoded SHA checksum to verify dep package.
- Updated the SHA value to new one.
- Added initialization of `$(UNAME_S)` variable `UNAME_S=$(shell uname -s)` since it was being used but never set which lead to explicitly exporting the variable from shell.

Similar PR for fabric8-auth - fabric8-services/fabric8-auth#549

Note: We can probably look into automating the process where SHA value is fetched dynamically instead of hard coding.

**commit** fabric8-services/fabric8-wit@f13e094
**Author:** Elliott Baron <ebaron@redhat.com>
**Date:**   Wed Jul 11 13:46:08 2018 -0400

Add parameter to Delete Space to skip deleting OpenShift resources (fabric8-services/fabric8-wit#2121)

When a user resets their environment, the front-end makes calls to the Delete Space API and Clean Tenant API. It makes these calls asynchronously, and due to both APIs acting on the same resources, I suspect this is the reason we are seeing a variety of errors in openshiftio/openshift.io#3500.

Since the Clean Tenant API cleans out the user's entire namespaces, it is not necessary in this case for Delete Space to delete anything from OpenShift. This PR adds an optional parameter skipCluster, to the Delete Space API, which if true, will not attempt to delete any deployments from OpenShift. The front-end could then use this parameter only when resetting the user's environment. An alternative would be for the front-end to synchronize between deleting spaces and calling Clean Tenant, but this would be less efficient.

Fixes (partially): openshiftio/openshift.io#3500

**commit** fabric8-services/fabric8-wit@cb85aa7
**Author:** Ibrahim Jarif <jarifibrahim@gmail.com>
**Date:**   Fri Jul 13 12:57:43 2018 +0530

Refactor search_blackbox_test.go (fabric8-services/fabric8-wit#2148)

**commit** eae146f7d3cdf1d6c40588608e60d688aaf6ad83
**Author:** Dhriti Shikhar <dhriti.shikhar.rokz@gmail.com>
**Date:**   Fri Jul 13 16:55:40 2018 +0530

Increase paging limit (fabric8-services/fabric8-wit#2166)

**commit** fabric8-services/fabric8-wit@d198813
**Author:** Baiju Muthukadan <baiju.m.mail@gmail.com>
**Date:**   Fri Jul 13 17:46:12 2018 +0530

Revert "List work items part of child iterations (fabric8-services/fabric8-wit#2146)" (fabric8-services/fabric8-wit#2168)

This reverts commit 68996d1555a29a9ef310403403855b47559d5a71.


This is required to address #3974

**commit** fabric8-services/fabric8-wit@a4d9061
**Author:** Michael Kleinhenz <kleinhenz@redhat.com>
**Date:**   Fri Jul 13 16:24:37 2018 +0200

feat(boardview): Board View for WIT. (fabric8-services/fabric8-wit#2111)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants