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

chore: minor tweaks to be consistent w/ new configbump build #926

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

nickboldt
Copy link
Contributor

@nickboldt nickboldt commented Sep 18, 2023

What does this PR do?

chore: minor tweaks to be consistent w/ new configbump build

Change-Id: If5170c7d93625ef37d49a108ccce6192eced1550
Signed-off-by: Nick Boldt nboldt@redhat.com

Screenshot/screencast of this PR

N/A

What issues does this PR fix or reference?

eclipse-che/che#21988

How to test this PR?

N/A

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

@che-bot
Copy link
Contributor

che-bot commented Sep 18, 2023

Click here to review and test in web IDE: Contribute

@nickboldt nickboldt changed the title chore: minor tweaks to be consistent w/ new... chore: minor tweaks to be consistent w/ new configbump build Sep 18, 2023
Change-Id: If5170c7d93625ef37d49a108ccce6192eced1550
Signed-off-by: Nick Boldt <nboldt@redhat.com>
@nickboldt
Copy link
Contributor Author

/retest

uses: docker/login-action@v2
with:
registry: quay.io
username: ${{ secrets.QUAY_USERNAME }}
password: ${{ secrets.QUAY_PASSWORD }}
-
name: "Docker docker.io Login"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

removed this because I didn't think we need to push to docker.io and shouldn't need authentication to pull images... but test is now failing? https://github.com/eclipse-che/che-dashboard/actions/runs/6223681358/job/16890210564?pr=926

@nickboldt
Copy link
Contributor Author

/retest

@github-actions
Copy link

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

@openshift-ci
Copy link

openshift-ci bot commented Oct 3, 2023

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: nickboldt, 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

@nickboldt nickboldt merged commit 443c826 into main Oct 3, 2023
15 checks passed
@nickboldt nickboldt deleted the tweaks_21988 branch October 3, 2023 14:20
@devstudio-release
Copy link

Build 3.10 :: dashboard_3.x/355: 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/355: SUCCESS

Upstream sync done; /DS_CI/sync-to-downstream_3.x/4871 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/1971: Console, Changes, Git Data

@devstudio-release
Copy link

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

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

@devstudio-release
Copy link

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

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

@devstudio-release
Copy link

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

@devstudio-release
Copy link

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

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

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.

5 participants