diff --git a/RELEASE_GUIDE.md b/RELEASE_GUIDE.md index 896d0139f3..50f8b759e2 100644 --- a/RELEASE_GUIDE.md +++ b/RELEASE_GUIDE.md @@ -6,6 +6,13 @@ Lens releases are built by CICD automatically on git tags. The typical release p 1. If doing a patch release checkout the `release/vMAJOR.MINOR` branch for the appropriate `MAJOR`/`MINOR` version and manually `cherry-pick` the PRs required for the patch that were commited to master. If there are any conflicts they must be resolved manually. If necessary, get assistance from the PR authors. +This can be helped (if you have the `gh` CLI installed) by running: +``` +gh api -XGET "/repos/lensapp/lens/pulls?state=closed&per_page=100" | jq -r '[.[] | select(.milestone.title == "") | select((.merged_at | type) == "string")] | sort_by(.merged_at) | map(.merge_commit_sha) | join(" ")' +``` + +But you will probably need to verify that all the PRs have correct milestones. + 1. From a clean and up to date `master` (or `release/vMAJOR.MINOR` if doing a patch release) run `npm version --git-tag-version false` where `` is one of the following: - `major` - `minor` @@ -14,14 +21,27 @@ Lens releases are built by CICD automatically on git tags. The typical release p - `preminor [--preid=]` - `prepatch [--preid=]` - `prerelease [--preid=]` - + where `` is generally one of: - `alpha` - `beta` - `rc` - + This assumes origin is set to https://github.com/lensapp/lens.git. If not then set GIT_REMOTE to the remote that is set to https://github.com/lensapp/lens.git. For example run `GIT_REMOTE=upstream npm version ...` 1. Open the PR (git should have printed a link to GitHub in the console) with the contents of all the accepted PRs since the last release. The PR description needs to be filled with the draft release description. From https://github.com/lensapp/lens click on Releases, the draft release should be first in the list, click `Edit` and copy/paste the markdown to the PR description. Add the `skip-changelog` label and click `Create Pull Request`. If this is a patch release be sure to set the PR base branch to `release/vMAJOR.MINOR` instead of `master`. + + It might also help, if the release drafter isn't updating correctly. To grab the data for the PR description using `gh` and `jq`. You can run the following command to get all the titles: + + ``` + gh api -XGET "/repos/lensapp/lens/pulls?state=closed&per_page=100" | jq -r '[.[] | select(.milestone.title == "") | select((.merged_at | type) == "string")] | sort_by(.merged_at) | map([.title, " (**#", .number, "**) ", .user.html_url] | join ("")) | join("\n")' | pbcopy + ``` + + And if you want to specify just bug fixes then you can add the following to the end of the `[ ... ]` section in the above command (before the `sort_by`) to just have bug fixes. Switch to `== "enhancement"` for enhancements and `all()` with `. != "bug && . != "enhancement"` for maintanence sections. + + ``` + | select(any(.labels | map(.name)[]; . == "bug")) + ``` + 1. After the PR is accepted and passes CI (and before merging), go to the same branch and run `make tag-release` (set GIT_REMOTE if necessary). This additionally triggers the azure jobs to build the binaries and put them on S3. 1. If the CI fails at this stage the problem needs to be fixed. Sometimes an azure job fails due to outside service issues (e.g. Apple signing occasionally fails), in which case the specific azure job can be rerun from https://dev.azure.com/lensapp/lensapp/_build. Otherwise changes to the codebase may need to be done and committed to the release branch and pushed to https://github.com/lensapp/lens. CI will run again. As well the release tag needs to be manually set to this new commit. You can do something like: - `git push origin :refs/tags/vX.Y.Z-beta.N` (removes the tag from https://github.com/lensapp/lens) @@ -43,4 +63,4 @@ Other tasks - generate a changelog from the prerelease descriptions (for major/minor releases) - announce the release on lens and lens-hq slack channels (release is announced automatically on the community slack lens channel through the above publishing process) - announce on lens-hq that master is open for PR merges for the next release (for major/minor releases) - - update issues on github (bump those that did not make it into the release to a subsequent release) (for major/minor/patch releases) \ No newline at end of file + - update issues on github (bump those that did not make it into the release to a subsequent release) (for major/minor/patch releases) diff --git a/src/common/k8s-api/kube-json-api.ts b/src/common/k8s-api/kube-json-api.ts index 3776634a47..59c8a7e980 100644 --- a/src/common/k8s-api/kube-json-api.ts +++ b/src/common/k8s-api/kube-json-api.ts @@ -21,7 +21,6 @@ import { JsonApi, JsonApiData, JsonApiError } from "./json-api"; import type { Response } from "node-fetch"; -import type { Cluster } from "../../main/cluster"; import { LensProxy } from "../../main/lens-proxy"; import { apiKubePrefix, isDebugging } from "../vars"; @@ -73,7 +72,7 @@ export interface KubeJsonApiError extends JsonApiError { } export class KubeJsonApi extends JsonApi { - static forCluster(cluster: Cluster): KubeJsonApi { + static forCluster(clusterId: string): KubeJsonApi { const port = LensProxy.getInstance().port; return new this({ @@ -82,7 +81,7 @@ export class KubeJsonApi extends JsonApi { debug: isDebugging, }, { headers: { - "Host": `${cluster.id}.localhost:${port}`, + "Host": `${clusterId}.localhost:${port}`, }, }); } diff --git a/src/extensions/renderer-api/k8s-api.ts b/src/extensions/renderer-api/k8s-api.ts index 4a084e6373..ecae80fb5a 100644 --- a/src/extensions/renderer-api/k8s-api.ts +++ b/src/extensions/renderer-api/k8s-api.ts @@ -55,6 +55,7 @@ export { ClusterRole, clusterRoleApi } from "../../common/k8s-api/endpoints"; export { ClusterRoleBinding, clusterRoleBindingApi } from "../../common/k8s-api/endpoints"; export { CustomResourceDefinition, crdApi } from "../../common/k8s-api/endpoints"; export { KubeObjectStatusLevel } from "./kube-object-status"; +export { KubeJsonApi } from "../../common/k8s-api/kube-json-api"; // types export type { ILocalKubeApiConfig, IRemoteKubeApiConfig, IKubeApiCluster } from "../../common/k8s-api/kube-api"; diff --git a/src/main/shell-session/node-shell-session.ts b/src/main/shell-session/node-shell-session.ts index 1f1a6cab03..b865fc411d 100644 --- a/src/main/shell-session/node-shell-session.ts +++ b/src/main/shell-session/node-shell-session.ts @@ -64,7 +64,7 @@ export class NodeShellSession extends ShellSession { const args = ["exec", "-i", "-t", "-n", "kube-system", this.podName, "--"]; const nodeApi = new NodesApi({ objectConstructor: Node, - request: KubeJsonApi.forCluster(this.cluster), + request: KubeJsonApi.forCluster(this.cluster.id), }); const node = await nodeApi.get({ name: this.nodeName }); const nodeOs = node.getOperatingSystem(); diff --git a/src/main/shell-session/shell-session.ts b/src/main/shell-session/shell-session.ts index b171986d56..677a637cc0 100644 --- a/src/main/shell-session/shell-session.ts +++ b/src/main/shell-session/shell-session.ts @@ -158,6 +158,8 @@ export abstract class ShellSession { cwd, env, name: "xterm-256color", + // TODO: Something else is broken here so we need to force the use of winPty on windows + useConpty: false, })); } diff --git a/src/renderer/api/terminal-api.ts b/src/renderer/api/terminal-api.ts index 8cca03538b..5f1acc9516 100644 --- a/src/renderer/api/terminal-api.ts +++ b/src/renderer/api/terminal-api.ts @@ -144,18 +144,6 @@ export class TerminalApi extends WebSocketApi { this.socket.binaryType = "arraybuffer"; } - destroy() { - if (!this.socket) return; - const controlCode = String.fromCharCode(4); // ctrl+d - - this.sendMessage({ type: TerminalChannels.STDIN, data: controlCode }); - setTimeout(() => super.destroy(), 2000); - } - - reconnect() { - super.reconnect(); - } - sendMessage(message: TerminalMessage) { return this.send(serialize(message)); } diff --git a/src/renderer/components/cluster-manager/cluster-status.tsx b/src/renderer/components/cluster-manager/cluster-status.tsx index e926a75d84..79df15c412 100644 --- a/src/renderer/components/cluster-manager/cluster-status.tsx +++ b/src/renderer/components/cluster-manager/cluster-status.tsx @@ -153,7 +153,7 @@ export class ClusterStatus extends React.Component { return (
-

{this.entity.getName()}

+

{this.entity?.getName() ?? this.cluster.name}

{this.renderStatusIcon()} {this.renderAuthenticationOutput()} {this.renderReconnectionHelp()} diff --git a/src/renderer/components/layout/__tests__/sidebar-cluster.test.tsx b/src/renderer/components/layout/__tests__/sidebar-cluster.test.tsx index 195368408d..c256c03e76 100644 --- a/src/renderer/components/layout/__tests__/sidebar-cluster.test.tsx +++ b/src/renderer/components/layout/__tests__/sidebar-cluster.test.tsx @@ -57,18 +57,25 @@ describe("", () => { }); it("renders cluster avatar and name", () => { - const { getByText } = render(); + const { getByText, getAllByText } = render(); expect(getByText("tc")).toBeInTheDocument(); - expect(getByText("test-cluster")).toBeInTheDocument(); + + const v = getAllByText("test-cluster"); + + expect(v.length).toBeGreaterThan(0); + + for (const e of v) { + expect(e).toBeInTheDocument(); + } }); - it("renders cluster menu", async () => { + it("renders cluster menu", () => { const { getByTestId, getByText } = render(); const link = getByTestId("sidebar-cluster-dropdown"); fireEvent.click(link); - expect(await getByText("Add to Hotbar")).toBeInTheDocument(); + expect(getByText("Add to Hotbar")).toBeInTheDocument(); }); }); diff --git a/src/renderer/components/layout/setting-layout.tsx b/src/renderer/components/layout/setting-layout.tsx index 9f458658e8..282eed0dcb 100644 --- a/src/renderer/components/layout/setting-layout.tsx +++ b/src/renderer/components/layout/setting-layout.tsx @@ -26,6 +26,7 @@ import { observer } from "mobx-react"; import { cssNames, IClassName } from "../../utils"; import { navigation } from "../../navigation"; import { Icon } from "../icon"; +import { catalogURL } from "../../../common/routes"; export interface SettingLayoutProps extends React.DOMAttributes { className?: IClassName; @@ -39,7 +40,13 @@ export interface SettingLayoutProps extends React.DOMAttributes { const defaultProps: Partial = { provideBackButtonNavigation: true, contentGaps: true, - back: () => navigation.goBack(), + back: () => { + if (navigation.length <= 1) { + navigation.push(catalogURL()); + } else { + navigation.goBack(); + } + }, }; /** diff --git a/src/renderer/components/layout/sidebar-cluster.module.css b/src/renderer/components/layout/sidebar-cluster.module.css index 635443500d..2b57854776 100644 --- a/src/renderer/components/layout/sidebar-cluster.module.css +++ b/src/renderer/components/layout/sidebar-cluster.module.css @@ -61,4 +61,51 @@ .avatar { font-weight: 500; margin-right: 1.25rem; -} \ No newline at end of file +} + +.loadingAvatar { + position: relative; + pointer-events: none; + &:after { + content: ""; + position: absolute; + left: 0; + top: 0; + width: 0; + height: 100%; + background: transparentize(white, .85); + animation: waiting 1.5s infinite linear; + } +} + +.loadingClusterName { + position: relative; + pointer-events: none; + width: 80%; + height: 16px; + &:after { + content: ""; + position: absolute; + left: 0; + top: 0; + width: 0; + height: 100%; + background: transparentize(white, .85); + animation: waiting 1.5s infinite linear; + } +} + +@keyframes waiting { + 0% { + left: 0; + width: 0; + } + 50% { + left: 25%; + width: 75%; + } + 75% { + left: 100%; + width: 0; + } +} diff --git a/src/renderer/components/layout/sidebar-cluster.tsx b/src/renderer/components/layout/sidebar-cluster.tsx index baa77f168e..a4479f1a24 100644 --- a/src/renderer/components/layout/sidebar-cluster.tsx +++ b/src/renderer/components/layout/sidebar-cluster.tsx @@ -31,6 +31,7 @@ import { Icon } from "../icon"; import { navigate } from "../../navigation"; import { Menu, MenuItem } from "../menu"; import { ConfirmDialog } from "../confirm-dialog"; +import { Tooltip } from "../tooltip"; const contextMenu: CatalogEntityContextMenuContext = observable({ menuItems: [], @@ -60,11 +61,25 @@ function onMenuItemClick(menuItem: CatalogEntityContextMenu) { } } +function renderLoadingSidebarCluster() { + return ( +
+ +
+
+ ); +} + export function SidebarCluster({ clusterEntity }: { clusterEntity: CatalogEntity }) { const [opened, setOpened] = useState(false); if (!clusterEntity) { - return null; + return renderLoadingSidebarCluster(); } const onMenuOpen = () => { @@ -95,6 +110,7 @@ export function SidebarCluster({ clusterEntity }: { clusterEntity: CatalogEntity const { metadata, spec } = clusterEntity; const id = `cluster-${metadata.uid}`; + const tooltipId = `tooltip-${id}`; return (
-
+
{metadata.name}
+ + {metadata.name} +