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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(plugins/plugin-ibmcloud): initial CodeEngine support #5309

Merged
merged 1 commit into from Aug 6, 2020

Conversation

myan9
Copy link
Contributor

@myan9 myan9 commented Aug 5, 2020

Fixes #5302
Fixes #5303

Description of what you did:

My PR is a:

  • 馃挜 Breaking change
  • 馃悰 Bug fix
  • 馃拝 Enhancement
  • 馃殌 New feature

Please confirm that your PR fulfills these requirements

  • Multiple commits are squashed into one commit.
  • The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
  • All npm dependencies are pinned.

@myan9 myan9 requested a review from starpit as a code owner August 5, 2020 15:36
@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #5309 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5309   +/-   ##
=======================================
  Coverage   59.74%   59.74%           
=======================================
  Files          78       78           
  Lines        2539     2539           
  Branches      627      627           
=======================================
  Hits         1517     1517           
  Misses        788      788           
  Partials      234      234           
Impacted Files Coverage 螖
packages/core/src/index.ts 100.00% <酶> (酶)
packages/core/src/webapp/models/table.ts 80.00% <酶> (酶)
packages/core/src/webapp/views/registrar/modes.ts 50.00% <酶> (酶)

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update 61b169b...d3e9335. Read the comment docs.

listPodsAndExpectOne('nginx', ns)
listPodsAndExpectOne('nginx', ns, `--kubeconfig ${initialKubeConfig}`)
getPodInSidecar('nginx', ns, `--kubeconfig ${initialKubeConfig}`)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

testing get --kubeconfig table, sidecar header, and summary tab

}
}
}).run()

Copy link
Contributor Author

@myan9 myan9 Aug 5, 2020

Choose a reason for hiding this comment

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

test a large duration table auto-gridify, and switch to list view testing the cells

attributes: [{ value: 'foo' }, { value: '21000' }]
}
]
})
Copy link
Contributor Author

Choose a reason for hiding this comment

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

a duration table used for tests

@starpit starpit merged commit 677faca into kubernetes-sigs:master Aug 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for ibm cloud CodeEngine plugin-kubectl does not properly handle --kubeconfig CLI option
2 participants