Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Retry request when token is missed #942

Merged
merged 5 commits into from
Oct 9, 2023
Merged

fix: Retry request when token is missed #942

merged 5 commits into from
Oct 9, 2023

Conversation

tolusha
Copy link
Contributor

@tolusha tolusha commented Oct 6, 2023

What does this PR do?

Retry request when token is missed

What issues does this PR fix or reference?

eclipse-che/che#22353

Is it tested? How?

N/A

Release Notes

N/A

Docs PR

@che-bot
Copy link
Contributor

che-bot commented Oct 6, 2023

Click here to review and test in web IDE: Contribute

Copy link
Contributor

@olexii4 olexii4 left a comment

Choose a reason for hiding this comment

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

LGTM

@openshift-ci openshift-ci bot added lgtm and removed lgtm labels Oct 6, 2023
@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-942

Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
Signed-off-by: Anatolii Bazko <abazko@redhat.com>
@openshift-ci
Copy link

openshift-ci bot commented Oct 6, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: akurinnoy, olexii4, tolusha

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@tolusha
Copy link
Contributor Author

tolusha commented Oct 6, 2023

/retest

@github-actions
Copy link

github-actions bot commented Oct 6, 2023

Docker image build succeeded: quay.io/eclipse/che-dashboard:pr-942

@tolusha
Copy link
Contributor Author

tolusha commented Oct 9, 2023

/retest

@codecov
Copy link

codecov bot commented Oct 9, 2023

Codecov Report

Merging #942 (7cb56e9) into main (b1b21bd) will increase coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is 83.82%.

@@            Coverage Diff             @@
##             main     #942      +/-   ##
==========================================
+ Coverage   82.30%   82.32%   +0.02%     
==========================================
  Files         364      365       +1     
  Lines       38046    38154     +108     
  Branches     2450     2456       +6     
==========================================
+ Hits        31314    31412      +98     
- Misses       6706     6716      +10     
  Partials       26       26              
Flag Coverage Δ
unittests 82.32% <83.82%> (+0.02%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...-frontend/src/services/backend-client/eventsApi.ts 100.00% <100.00%> (ø)
...rd-frontend/src/services/backend-client/podsApi.ts 100.00% <100.00%> (ø)
...tend/src/services/backend-client/userProfileApi.ts 100.00% <100.00%> (ø)
...services/backend-client/devWorkspaceTemplateApi.ts 45.00% <50.00%> (ø)
...end/src/services/backend-client/devWorkspaceApi.ts 57.69% <63.63%> (+0.27%) ⬆️
...ontend/src/services/backend-client/gitConfigApi.ts 57.57% <20.00%> (-3.72%) ⬇️
.../services/backend-client/personalAccessTokenApi.ts 32.85% <20.00%> (ø)
...ontend/src/services/backend-client/axiosWrapper.ts 92.07% <92.07%> (ø)

@tolusha tolusha merged commit 56bca30 into main Oct 9, 2023
17 checks passed
@tolusha tolusha deleted the 22353_401 branch October 9, 2023 08:48
@devstudio-release
Copy link

Build 3.10 :: dashboard_3.x/360: Console, Changes, Git Data

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: dashboard_3.x/360: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4912 triggered

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

@devstudio-release
Copy link

Build 3.10 :: copyIIBsToQuay/1988: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.10 :: sync-to-downstream_3.x/4913: SUCCESS

Build container: devspaces-operator-bundle synced; /DS_CI/get-sources-rhpkg-container-build_3.x/4735 triggered; /job/DS_CI/job/dsc_3.x triggered;

@devstudio-release
Copy link

Build 3.10 :: operator-bundle_3.x/2139: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4913 triggered

@devstudio-release
Copy link

Build 3.10 :: dsc_3.x/1458: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.10 :: update-digests_3.x/4562: SUCCESS

Detected new images: rebuild operator-bundle
* dashboard; /DS_CI/operator-bundle_3.x/2139 triggered

@devstudio-release
Copy link

Build 3.10 :: dsc_3.x/1458: SUCCESS

3.10.0-CI

@devstudio-release
Copy link

Build 3.10 :: copyIIBsToQuay/1988: SUCCESS

3.10
arches = x86_64, s390x, ppc64le;
  * LATEST DS OPERATOR BUNDLE = <a href=https://quay.io/repository/devspaces/devspaces-operator-bundle?tab=tags>registry-proxy.engineering.redhat.com/rh-osbs/devspaces-operator-bundle:3.10-116
  * LATEST DWO OPERATOR BUNDLE = <a href=https://quay.io/repository/devworkspace/devworkspace-operator-bundle?tab=tags>registry-proxy.engineering.redhat.com/rh-osbs/devworkspace-operator-bundle:0.21-7
+ s390x-rhel8 IIB(s) copied:
  + quay.io/devspaces/iib:3.10-v4.14-592011-533291-s390x
  + quay.io/devspaces/iib:3.10-v4.14-s390x
  + quay.io/devspaces/iib:next-v4.14-s390x
  + quay.io/devspaces/iib:3.10-v4.13-592010-533286-s390x
  + quay.io/devspaces/iib:3.10-v4.13-s390x
  + quay.io/devspaces/iib:next-v4.13-s390x
  + quay.io/devspaces/iib:3.10-v4.12-592286-533281-s390x
  + quay.io/devspaces/iib:3.10-v4.12-s390x
  + quay.io/devspaces/iib:next-v4.12-s390x
  + quay.io/devspaces/iib:3.10-v4.11-592282-533277-s390x
  + quay.io/devspaces/iib:3.10-v4.11-s390x
  + quay.io/devspaces/iib:next-v4.11-s390x
+ ppc64le-rhel8 IIB(s) copied:
  + quay.io/devspaces/iib:3.10-v4.14-592011-533291-ppc64le
  + quay.io/devspaces/iib:3.10-v4.14-ppc64le
  + quay.io/devspaces/iib:next-v4.14-ppc64le
  + quay.io/devspaces/iib:3.10-v4.13-592010-533286-ppc64le
  + quay.io/devspaces/iib:3.10-v4.13-ppc64le
  + quay.io/devspaces/iib:next-v4.13-ppc64le
  + quay.io/devspaces/iib:3.10-v4.12-592286-533281-ppc64le
  + quay.io/devspaces/iib:3.10-v4.12-ppc64le
  + quay.io/devspaces/iib:next-v4.12-ppc64le
  + quay.io/devspaces/iib:3.10-v4.11-592282-533277-ppc64le
  + quay.io/devspaces/iib:3.10-v4.11-ppc64le
  + quay.io/devspaces/iib:next-v4.11-ppc64le
+ x86_64-rhel8 IIB(s) copied:
  + quay.io/devspaces/iib:3.10-v4.14-592011-533291-x86_64
  + quay.io/devspaces/iib:3.10-v4.14-x86_64
  + quay.io/devspaces/iib:next-v4.14-x86_64
  + quay.io/devspaces/iib:3.10-v4.13-592289-533286-x86_64
  + quay.io/devspaces/iib:3.10-v4.13-x86_64
  + quay.io/devspaces/iib:next-v4.13-x86_64
  + quay.io/devspaces/iib:3.10-v4.12-592286-533281-x86_64
  + quay.io/devspaces/iib:3.10-v4.12-x86_64
  + quay.io/devspaces/iib:next-v4.12-x86_64
  + quay.io/devspaces/iib:3.10-v4.11-592282-533277-x86_64
  + quay.io/devspaces/iib:3.10-v4.11-x86_64
  + quay.io/devspaces/iib:next-v4.11-x86_64

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants