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

Fast forward master-next to master #2388

Closed
wants to merge 1,533 commits into from
Closed

Conversation

yaacov
Copy link
Member

@yaacov yaacov commented Aug 18, 2019

Fast forward master-next to master

We already have PR candidates for "master-next" branch, but the current one is way behind "master".

Is it OK to re-create "master-next" from the current "master" branch?

openshift-merge-robot and others added 30 commits July 31, 2019 03:24
Fix bug where update icon is too small
Adding cloudbehl to the OWNERS file(ceph, noobaa)
Fix bug for infinite calls to get imagestream tag in git import
Fix position of loading indicator in LoadingBox component
add builds decorator to topology node
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1724010

Calling the `URLSearchParams` constructor with a sequence only started
to be supported by browsers fairly recently. Although we don't strictly
support these older browsers, the fix is simple.
Fixes https://jira.coreos.com/browse/CONSOLE-1619

The list row filter bars were jutting out into the vertical page gutters
on several pages due to them being double wrapped by `<div
className="row">`.
Bug fix for infotips in Persistent and Object service dashboards
Fix bug where masthead dropdowns have incorrect aria-label
kubevirt: make DNS1123 validation more verbose
Add null check for file in DroppableEditYAML component
…ce-account

fix(pipeline-service-account) copy Service account while rerunning Pipelines
Bug 1734601 - The statefulset created with default yaml will get error in page after add storage to it
Bug 1665010: Add button link co copy webhook URL with secret to the clipboard
Signed-off-by: aruniiird <arun.iiird@gmail.com>
Bug 1732678: Fix text overflow in  bar chart
The ConfigMap provides default `accessMode` and `volumeMode` for storage
classes.
Fix to align with the right edge of the graph and add a small margin
below the button link.
Bug 1741470: Correctly order overview groups
Bug 1739225: Timestamp component always shows dates in the future as relative dates
Ensure that Jest testRegex catches all the specs
Fix bug introduced by PR openshift#2321.

`filterLabels` doesn't include internal labels, so we need to ignore
them when matching.
`olm` is no longer a valid suite name, so change it to a var to avoid confusion.
Fix bug where overview alerts can abut one another
…details-graph

Monitoring: Fix alert details page graph
The old cluster status page has been replaced by the new dashboards
page. Remove the orphaned components.
Switch overview tooltips to PF4 to prevent overflow clipping
Bug 1739831: Fix wrapping issue with node details graphs
Switch timestamp tooltips to PF4 to prevent overflow clipping
Change integration test suite name `olm` to `<suite>`
…r-spacing

apiExplorer: add spacing between dropdown components
@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Aug 18, 2019
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: yaacov
To complete the pull request process, please assign spadgett
You can assign the PR to them by writing /assign @spadgett in a comment when ready.

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

@yaacov
Copy link
Member Author

yaacov commented Aug 18, 2019

@vojtechszocs , do you know who should review/take-ownership this ?

@openshift-ci-robot
Copy link
Contributor

@yaacov: The following tests failed, say /retest to rerun them all:

Test name Commit Details Rerun command
ci/prow/e2e-aws cdeed47 link /test e2e-aws
ci/prow/e2e-aws-console cdeed47 link /test e2e-aws-console

Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@spadgett
Copy link
Member

Let's create a master-4.3 branch instead. We should consider closing out #2269 and #2251 before starting 4.3 features.

@yaacov
Copy link
Member Author

yaacov commented Aug 18, 2019

@spadgett Thanks for the fast response !

Closing per #2388 (comment)

@yaacov yaacov closed this Aug 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.