Skip to content

Conversation

@rhamilto
Copy link
Member

@rhamilto rhamilto commented Apr 16, 2019

https://jira.coreos.com/browse/CONSOLE-1480

localhost_9000_command-line-tools

apiVersion: apiextensions.k8s.io/v1beta1
kind: CustomResourceDefinition
metadata:
  name: consoleclidownloads.console.openshift.io
  annotations:
    displayName: ConsoleCLIDownload
    description: Extension for configuring openshift web console command line interface (CLI) downloads.
spec:
  scope: Cluster
  group: console.openshift.io
  versions:
  - name: v1
    served: true
    storage: true
  names:
    plural: consoleclidownloads
    singular: consoleclidownload
    kind: ConsoleCLIDownload
    listKind: ConsoleCLIDownloadList
  additionalPrinterColumns:
  - name: Display name
    type: string
    JSONPath: .spec.displayName
  - name: Age
    type: string
    JSONPath: .metadata.creationTimestamp
  - name: Description
    type: string
    JSONPath: .spec.description
  subresources:
    status: {}
  validation:
    openAPIV3Schema:
      properties:
        spec:
          type: object
          description: Represents console CLI download spec
          required:
          - displayName
          - description
          - links
          properties:
            displayName:
              type: string
              description: Display name of the CLI download
            description:
              type: string
              description: Description of the CLI download (can include markdown)
            links:
              type: array
              description: Objects that hold CLI download link details
              nullable: true
              items:
                type: object
                required:
                - href
                properties:
                  text:
                    type: string
                    description: Text of the link
                  href:
                    type: string
                    description: Absolute secure URL for the link (must use https)
                    pattern: '^https://([\w-]+.)+[\w-]+(/[\w- ./?%&=])?$'

@openshift-ci-robot openshift-ci-robot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Apr 16, 2019
@openshift-ci-robot openshift-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Apr 16, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from c273ca8 to d695f8e Compare April 16, 2019 19:27
@spadgett spadgett added this to the v4.2 milestone Apr 17, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from d695f8e to 1b08f25 Compare April 17, 2019 15:47
@rhamilto rhamilto force-pushed the command-line-tools-page branch 2 times, most recently from f5ab550 to b6e6967 Compare April 17, 2019 17:50
@rhamilto rhamilto changed the title [WIP] Add ConsoleCLITool CRD extension [WIP] Add ConsoleCLIDownload CRD extension Apr 17, 2019
Copy link
Member

@jhadvig jhadvig left a comment

Choose a reason for hiding this comment

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

@rhamilto looks great! Just couple of comments :)

@rhamilto rhamilto force-pushed the command-line-tools-page branch 5 times, most recently from b075dd9 to b657395 Compare April 23, 2019 13:52
@rhamilto rhamilto force-pushed the command-line-tools-page branch from b657395 to ece439d Compare May 2, 2019 17:30
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 2, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from ece439d to a496323 Compare May 2, 2019 17:33
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 2, 2019
@rhamilto
Copy link
Member Author

rhamilto commented May 2, 2019

Just discovered a bug and am not sure how to fix. If the CRD has not been registered with the cluster, the page contents don't render since firehose cannot get resources that don't have a model registered.

No model registered for console.openshift.io~v1~ConsoleCLIDownload

@spadgett, how do I fix?

@rhamilto rhamilto force-pushed the command-line-tools-page branch 3 times, most recently from d912c7b to c8daec2 Compare May 7, 2019 18:23
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label May 7, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from c8daec2 to 893c648 Compare May 7, 2019 18:30
@openshift-ci-robot openshift-ci-robot added needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. and removed needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. labels May 7, 2019
@rhamilto
Copy link
Member Author

rhamilto commented Jun 3, 2019

/retest

1 similar comment
@rhamilto
Copy link
Member Author

rhamilto commented Jun 3, 2019

/retest

@rhamilto rhamilto force-pushed the command-line-tools-page branch from e25a041 to 9a133c6 Compare June 3, 2019 19:54
benjaminapetersen added a commit to benjaminapetersen/console-operator that referenced this pull request Jun 4, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from 9a133c6 to abadbfa Compare June 19, 2019 18:04
@rhamilto
Copy link
Member Author

/retest

@rhamilto rhamilto force-pushed the command-line-tools-page branch from abadbfa to 3cff6e8 Compare June 19, 2019 19:21
benjaminapetersen added a commit to benjaminapetersen/console-operator that referenced this pull request Jun 20, 2019
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
const CommandLineTools = ({obj}) => {
const CommandLineTools: React.FC<CommandLineToolsProps> = ({obj}) => {

then

type CommandLineToolsProps = {
  obj: FirehoseResult;
};

@rhamilto rhamilto force-pushed the command-line-tools-page branch from 3cff6e8 to 0802365 Compare June 25, 2019 13:51
@rhamilto rhamilto changed the title [WIP] Add ConsoleCLIDownload CRD extension Add ConsoleCLIDownload CRD extension Jun 25, 2019
@openshift-ci-robot openshift-ci-robot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 25, 2019
@rhamilto
Copy link
Member Author

@spadgett, PR updated and ready for merge.

Copy link
Member

@spadgett spadgett 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-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2019
@spadgett spadgett removed the kind/api-change Categorizes issue or PR as related to adding, removing, or otherwise changing an API label Jun 25, 2019
@rhamilto rhamilto force-pushed the command-line-tools-page branch from 0802365 to 5b49014 Compare June 25, 2019 15:24
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jun 25, 2019
@rhamilto
Copy link
Member Author

@spadgett Rebased. Please tag again.

@spadgett
Copy link
Member

/lgtm

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

[APPROVALNOTIFIER] This PR is APPROVED

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

Details 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
Copy link
Member

infrastructure failure

/retest

@openshift-merge-robot openshift-merge-robot merged commit 4f4498d into openshift:master Jun 25, 2019
@rhamilto rhamilto deleted the command-line-tools-page branch June 26, 2019 12:16
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/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants