Skip to content

Conversation

spadgett
Copy link
Member

No description provided.

dtaylor113 and others added 30 commits February 4, 2019 10:42
…ng string issues

Add .co-break-all to name td
Align line-height for copy-to-clipboard to match height of copy button
Apply table--layout-fixed to images table...
Bug 1684217: Fix runtime error on machine set and machine deployment details pages
Update README for OpenShift 4.0 and CI changes
Change dropdown units for consistency on PVC form
…rators

Show community operators, warn on details
Changed the links for the CRD list page to go to the details page instead of the instances page. Updated the kebab and actions menu with a link to the instances page.

Fixes https://jira.coreos.com/projects/CONSOLE/issues/CONSOLE-1210
Adds a `QueryBrowser` component, which plots a PromQL query as a line
graph along with controls for changing the graph to display the query
for any time range.

Adds the `QueryBrowser` to both the alert details page and the rule
details page. The alert graph shows the single metric for that alert and
the rule graph shows all metrics data for that rule.

Upgrade Plotly to version 1.44.4.

Move the "View in Prometheus UI" link from the top of the page to the
graph area.

Make some changes to the graph Base class.
  - Make the `title` property optional
  - Add the ability to invoke `fetch()` without enabling polling
  - Set a lower limit of 5 seconds on the graph refresh interval to
    prevent too frequent polling when viewing narrow time ranges.
  - Add optional `numSamples` property that defines how many data points
    will be plotted for each metric
- Changed OperatorHub test to check for OwnNamespace installmode
  rather than AllNamespaces
- Updated namespace to match testName for single namespace install
- Added views for selecting namespace on Subscription page

The etcd operator in community-operators changed late last week,
replacing AllNamespaces functionality with SingleNamespace.
This puts a bandaid on the issue.
Operators in community-operators can change frequently, and
because of this OperatorHub tests will be unstable when relying on
external content.
Bandaid fix OperatorHub tests for etcd operator
Bug 1685006: Show mock resource quota page when user has no projects
…externalMetadata_displayName

Bug 1685802 - Fix when no externalMetadata for displayName when Creat…
spadgett and others added 16 commits March 11, 2019 15:59
…a-dev-catalog

Update service catalog tests to provision through developer catalog
Always provide a cluster update path when updates are available
Fix center text in gauge for new plotly version
- Rename machines sidnav item to "Compute"
- Move Nodes into the sidenav Compute section
- Add machines column into Nodes list view
- Remove orphaned code
- Add horizontal separator between Machine Config and Machine Config pools

Console story - https://jira.coreos.com/browse/CONSOLE-1284
Bug 1685661: Reload OAuth config at every point of use
Run OLM and OperatorHub tests as a separate suite in CI
…-version

Bug 1687685 - Add ClusterVersion to global config page
devconsole: setup sub-dir with OWNERS file
…-copy

Add css to truncated container image name to communicate to the user that the entire string can be selected and copied.
@openshift-ci-robot openshift-ci-robot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label Mar 14, 2019
@spadgett
Copy link
Member Author

/retest

level=error msg="Error: Error applying plan:"
level=error
level=error msg="1 error occurred:"
level=error msg="\t* module.vpc.aws_route.to_nat_gw[1]: 1 error occurred:"
level=error msg="\t* aws_route.to_nat_gw.1: Error creating route: timeout while waiting for state to become 'success' (timeout: 2m0s)"

@spadgett spadgett added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Mar 14, 2019
@spadgett
Copy link
Member Author

Approving manually to create initial OWNERS file in master-next

@spadgett
Copy link
Member Author

/retest

@spadgett
Copy link
Member Author

/assign @rhamilto

@rhamilto
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Mar 14, 2019
@openshift-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

Approval requirements bypassed by manually added approval.

This pull-request has been approved by: rhamilto, spadgett

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

The pull request process is described 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

@spadgett spadgett mentioned this pull request Mar 14, 2019
3 tasks
@openshift-merge-robot openshift-merge-robot merged commit c1d04c5 into openshift:master-next Mar 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. 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.