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

Replace Blue Ocean in default display URL (or remove the Blue Ocean plugins) #2833

Closed
uhafner opened this issue Mar 17, 2022 · 35 comments
Closed

Comments

@uhafner
Copy link

uhafner commented Mar 17, 2022

Service(s)

ci.jenkins.io

Summary

All our plugin builds on ci.jenkins.io use GitHub checks to report builds results directly in pull requests. The checks provide a details URL in the pull request to directly navigate to the results in Jenkins. Currently, the display URL of the Jenkins instance is shown here. However, this details URL currently is configured to show the Blue Ocean view in Jenkins rather than the classic Jenkins view. Can you please change that so that the Blue Ocean view is not shown by default anymore. (In my opinion we can remove Blue Ocean totally, as this project has been abandoned years ago.)

(And yes: I know that I can customize the URL when logged in. But this is the main entry point for PR contributors and they should actually see Jenkins without logging in by using the current version without struggling with the deprecated Blue Ocean views.)

Reproduction steps

Example:

@uhafner uhafner added the triage Incoming issues that need review label Mar 17, 2022
@dduportal
Copy link
Contributor

🤔 I assume that removing BlueOcean would solve the URL problem, but can't be sure. We might want to try on release.ci or infra.ci.

This is a good point that you make given the state of BlueOcean.
(Personnaly it will be hard for me to not have Blue Ocean. The Jenkins "classic" UI is barely usable for viewing pipelines but maybe I should start learning to use the pipeline graph view from Tim).

@timja
Copy link
Member

timja commented Mar 17, 2022

Pipeline graph is very much WIP and really needs someone with more time to look into it.

blueocean had a whole team of people to build it...

Personally I use the Blue Ocean view a lot still...

@dduportal
Copy link
Contributor

I have no idea how to setup the URL of the GitHub checks, in the scenario of "keeping BlueOcean". Where should I look? (e.g. is it the "check" plugin, or th" github check" plugin, or "github" or jenkins management page, or something else)

@KalleOlaviNiemitalo
Copy link

KalleOlaviNiemitalo commented Mar 17, 2022

Seems to be configurable with the JENKINS_DISPLAYURL_PROVIDER environment variable or the jenkins.displayurl.provider Java property: JENKINS-41578, jenkinsci/display-url-api-plugin#7, https://github.com/jenkinsci/display-url-api-plugin/blob/f306dd90f7aa7117a446adbc116e0b9693a54d84/src/main/java/org/jenkinsci/plugins/displayurlapi/DisplayURLProvider.java#L178-L184

@KalleOlaviNiemitalo
Copy link

The implementation rereads the property every time, so if you call System.setProperty('jenkins.displayurl.provider', 'org.jenkinsci.plugins.displayurlapi.ClassicDisplayURLProvider') in the script console, then that should take effect immediately. Unless the HTTP redirect response is cached somewhere.

@timja
Copy link
Member

timja commented Mar 17, 2022

I think this should probably be discussed on the mailing list.

@basil
Copy link
Collaborator

basil commented Mar 20, 2022

+1 from me for changing the default from Blue Ocean to the standard UI. I almost always switch from Blue Ocean to the standard UI, so this would save me a click. I still do use Blue Ocean occasionally when I need to read the console log of a build with parallel stages, so I do not think we should remove the Blue Ocean plugin yet.

@dduportal dduportal added this to the infra-team-sync-next milestone Apr 8, 2022
@dduportal dduportal removed the triage Incoming issues that need review label Apr 8, 2022
@dduportal
Copy link
Contributor

Delaying this issue due to not enough bandwidth.

@dduportal
Copy link
Contributor

@dduportal
Copy link
Contributor

We've not seen any "no" answer (here and mailing list). Since this change is revertable, we'll proceed to apply it.

@dduportal
Copy link
Contributor

Change appplied successfully (and automatically) on ci.jenkins.io. Many thanks @smerle33 .

Check: https://github.com/jenkins-infra/pipeline-library/runs/6349066600 => click on the link View more details on ci.jenkins.io redirects to https://ci.jenkins.io/job/Infra/job/pipeline-library/job/master/255/ which is the classic UI.

@KalleOlaviNiemitalo
Copy link

Huh, that link doesn't even redirect; rather, it points directly to the classic UI. So if this change is ever reverted, then these links on GitHub will keep pointing to the classic UI.

@timja
Copy link
Member

timja commented Aug 19, 2023

FYI the issue that prevented this change was fixed in https://github.com/jenkinsci/display-url-api-plugin/releases/tag/display-url-api-2.3.9

@rosenkrmnv
Copy link

rosenkrmnv commented Mar 19, 2024

this fix did not help for me, neither any of the provided workarounds. Any other ideas? its quite annoying

@timja
Copy link
Member

timja commented Mar 19, 2024

this fix did not help for me, neither any of the provided workarounds. Any other ideas? its quite annoying

Can you explain steps to reproduce?

@lemeurherve
Copy link
Member

lemeurherve commented May 8, 2024

As I'd like these display/redirect links to redirect to the main build pages instead of the pipeline graph views, I tried again, checking first by changing only my user settings, then by temporarily changing the default notification URL setting of infra.ci.jenkins.io (private instance) from "default" to "Jenkins Classic":

image

Using the following redirect link as example: https://infra.ci.jenkins.io/job/kubernetes-jobs/job/kubernetes-management/job/main/34726/display/redirect

Here are the redirections depending on the setting value:

(I've reverted infra.ci.jenkins.io to its original setting value after the check)

As it now works as intended, I propose to set default notification URL on all instances ci.jenkins.io, infra.ci.jenkins.io and weekly.ci.jenkins.io to "Jenkins Classic", any objection?

@timja
Copy link
Member

timja commented May 8, 2024

+1 from me

@NotMyFault
Copy link
Member

+1

@lemeurherve lemeurherve self-assigned this May 8, 2024
@dduportal
Copy link
Contributor

+1

@lemeurherve
Copy link
Member

Change applied on infra.ci.jenkins.io and weekly.ci.jenkins.io but rollbacked on ci.jenkins.io as I encountered the following error:

image

@dduportal
Copy link
Contributor

Deferring issue to the 2024-06-11 milestone, time for Jay to get ready around JCasc.

@dduportal
Copy link
Contributor

Update:

@dduportal
Copy link
Contributor

Update:

ci.jenkins.io is also set up with the specified setting: we can close.

Congrats @jayfranco999 on your first finished issue!

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

No branches or pull requests

9 participants