1
0
mirror of https://github.com/lensapp/lens.git synced 2025-05-20 05:10:56 +00:00

Remove Helm release update checker (#420)

Signed-off-by: Lauri Nevala <lauri.nevala@gmail.com>
This commit is contained in:
Lauri Nevala 2020-06-09 14:33:46 +03:00 committed by GitHub
parent deb4bf2f7b
commit 247677b740
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
4 changed files with 6 additions and 73 deletions

View File

@ -198,21 +198,4 @@ export class HelmRelease implements ItemObject {
const chartVersion = versions.find(chartVersion => chartVersion.version === version); const chartVersion = versions.find(chartVersion => chartVersion.version === version);
return chartVersion ? chartVersion.repo : ""; return chartVersion ? chartVersion.repo : "";
} }
getLastVersion(): string | null {
const chartName = this.getChart();
const versions = helmChartStore.versions.get(chartName);
if (!versions) {
return null; // checking new version state
}
if (versions.length) {
return versions[0].version; // versions already sorted when loaded, the first is latest
}
return this.getVersion();
}
hasNewVersion() {
const lastVersion = this.getLastVersion();
return lastVersion && lastVersion !== this.getVersion();
}
} }

View File

@ -19,7 +19,6 @@ import { AceEditor } from "../ace-editor";
import { Button } from "../button"; import { Button } from "../button";
import { releaseStore } from "./release.store"; import { releaseStore } from "./release.store";
import { Notifications } from "../notifications"; import { Notifications } from "../notifications";
import { Icon } from "../icon";
import { createUpgradeChartTab } from "../dock/upgrade-chart.store"; import { createUpgradeChartTab } from "../dock/upgrade-chart.store";
import { getDetailsUrl } from "../../navigation"; import { getDetailsUrl } from "../../navigation";
import { _i18n } from "../../i18n"; import { _i18n } from "../../i18n";
@ -190,14 +189,12 @@ export class ReleaseDetails extends Component<Props> {
<DrawerItem name={<Trans>Chart</Trans>} className="chart"> <DrawerItem name={<Trans>Chart</Trans>} className="chart">
<div className="flex gaps align-center"> <div className="flex gaps align-center">
<span>{release.getChart()}</span> <span>{release.getChart()}</span>
{release.hasNewVersion() && ( <Button
<Button primary
primary label={_i18n._(t`Upgrade`)}
label={_i18n._(t`Upgrade`)} className="box right upgrade"
className="box right upgrade" onClick={this.upgradeVersion}
onClick={this.upgradeVersion} />
/>
)}
</div> </div>
</DrawerItem> </DrawerItem>
<DrawerItem name={<Trans>Updated</Trans>}> <DrawerItem name={<Trans>Updated</Trans>}>
@ -211,12 +208,6 @@ export class ReleaseDetails extends Component<Props> {
<span> <span>
{release.getVersion()} {release.getVersion()}
</span> </span>
{!release.getLastVersion() && (
<Icon svg="spinner" small/>
)}
{release.hasNewVersion() && (
<span><Trans>New version available:</Trans> <b>{release.getLastVersion()}</b></span>
)}
</div> </div>
</DrawerItem> </DrawerItem>
<DrawerItem name={<Trans>Status</Trans>} className="status" labelsOnly> <DrawerItem name={<Trans>Status</Trans>} className="status" labelsOnly>

View File

@ -37,7 +37,6 @@ export class HelmReleaseMenu extends React.Component<Props> {
const { release, toolbar } = this.props; const { release, toolbar } = this.props;
if (!release) return; if (!release) return;
const hasRollback = release && release.getRevision() > 1; const hasRollback = release && release.getRevision() > 1;
const hasNewVersion = release.hasNewVersion();
return ( return (
<> <>
{hasRollback && ( {hasRollback && (
@ -46,12 +45,6 @@ export class HelmReleaseMenu extends React.Component<Props> {
<span className="title"><Trans>Rollback</Trans></span> <span className="title"><Trans>Rollback</Trans></span>
</MenuItem> </MenuItem>
)} )}
{hasNewVersion && (
<MenuItem onClick={this.upgrade}>
<Icon material="build" interactive={toolbar} title={_i18n._(t`Upgrade`)}/>
<span className="title"><Trans>Upgrade</Trans></span>
</MenuItem>
)}
</> </>
) )
} }

View File

@ -5,9 +5,7 @@ import kebabCase from "lodash/kebabCase";
import { observer } from "mobx-react"; import { observer } from "mobx-react";
import { Trans } from "@lingui/macro"; import { Trans } from "@lingui/macro";
import { RouteComponentProps } from "react-router"; import { RouteComponentProps } from "react-router";
import { autobind, interval } from "../../utils";
import { releaseStore } from "./release.store"; import { releaseStore } from "./release.store";
import { helmChartStore } from "../+apps-helm-charts/helm-chart.store";
import { IReleaseRouteParams, releaseURL } from "./release.route"; import { IReleaseRouteParams, releaseURL } from "./release.route";
import { HelmRelease } from "../../api/endpoints/helm-releases.api"; import { HelmRelease } from "../../api/endpoints/helm-releases.api";
import { ReleaseDetails } from "./release-details"; import { ReleaseDetails } from "./release-details";
@ -15,9 +13,7 @@ import { ReleaseRollbackDialog } from "./release-rollback-dialog";
import { navigation } from "../../navigation"; import { navigation } from "../../navigation";
import { ItemListLayout } from "../item-object-list/item-list-layout"; import { ItemListLayout } from "../item-object-list/item-list-layout";
import { HelmReleaseMenu } from "./release-menu"; import { HelmReleaseMenu } from "./release-menu";
import { Icon } from "../icon";
import { secretsStore } from "../+config-secrets/secrets.store"; import { secretsStore } from "../+config-secrets/secrets.store";
import { when } from "mobx";
enum sortBy { enum sortBy {
name = "name", name = "name",
@ -33,31 +29,16 @@ interface Props extends RouteComponentProps<IReleaseRouteParams> {
@observer @observer
export class HelmReleases extends Component<Props> { export class HelmReleases extends Component<Props> {
private versionsWatcher = interval(3600, this.checkVersions);
componentDidMount() { componentDidMount() {
// Watch for secrets associated with releases and react to their changes // Watch for secrets associated with releases and react to their changes
releaseStore.watch(); releaseStore.watch();
this.versionsWatcher.start();
when(() => releaseStore.isLoaded, this.checkVersions);
} }
componentWillUnmount() { componentWillUnmount() {
releaseStore.unwatch(); releaseStore.unwatch();
this.versionsWatcher.stop();
} }
// Check all available versions every 1 hour for installed releases.
// This required to show "upgrade" icon in the list and upgrade button in the details view.
@autobind()
checkVersions() {
const charts = releaseStore.items.map(release => release.getChart());
return charts.reduce((promise, chartName) => {
const loadVersions = () => helmChartStore.getVersions(chartName, true);
return promise.then(loadVersions, loadVersions);
}, Promise.resolve({}))
};
get selectedRelease() { get selectedRelease() {
const { match: { params: { name, namespace } } } = this.props; const { match: { params: { name, namespace } } } = this.props;
return releaseStore.items.find(release => { return releaseStore.items.find(release => {
@ -130,7 +111,6 @@ export class HelmReleases extends Component<Props> {
]} ]}
renderTableContents={(release: HelmRelease) => { renderTableContents={(release: HelmRelease) => {
const version = release.getVersion(); const version = release.getVersion();
const lastVersion = release.getLastVersion();
return [ return [
release.getName(), release.getName(),
release.getNs(), release.getNs(),
@ -138,20 +118,6 @@ export class HelmReleases extends Component<Props> {
release.getRevision(), release.getRevision(),
<> <>
{version} {version}
{!lastVersion && (
<Icon
small svg="spinner"
className="checking-update"
tooltip={<Trans>Checking update</Trans>}
/>
)}
{release.hasNewVersion() && (
<Icon
material="new_releases"
className="new-version"
tooltip={<Trans>New version: {lastVersion}</Trans>}
/>
)}
</>, </>,
release.appVersion, release.appVersion,
{ title: release.getStatus(), className: kebabCase(release.getStatus()) }, { title: release.getStatus(), className: kebabCase(release.getStatus()) },