Skip to content

Commit

Permalink
fix: kebab case pod name on kubectl run
Browse files Browse the repository at this point in the history
  • Loading branch information
edosrecki committed Jan 11, 2023
1 parent 13b0e82 commit e5253fe
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions src/lib/configurations/index.ts
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import exitHook from 'exit-hook'
import { omit } from 'lodash'
import { omit, kebabCase } from 'lodash'
import {
deletePod,
portForward,
Expand Down Expand Up @@ -40,7 +40,7 @@ export const deleteConfiguration = (configuratioName: string): void => {

export const execConfiguration = (configuration: Configuration) => {
const pod = {
name: `sql-proxy-${configuration.configurationName}-${randomString()}`,
name: `sql-proxy-${kebabCase(configuration.configurationName)}-${randomString()}`,
context: configuration.kubernetesContext,
namespace: configuration.kubernetesNamespace,
serviceAccount: configuration.kubernetesServiceAccount,
Expand Down
2 changes: 1 addition & 1 deletion src/lib/updates.ts
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ export const notifyForUpdates = () => {
updateCheckInterval: oneDayMs,
})

if (update) {
if (update && update.current !== update.latest) {
const text =
`${yellow(update.current)}${green(update.latest)}\n\n` +
`${blue('brew')} upgrade ${name}\n` +
Expand Down

0 comments on commit e5253fe

Please sign in to comment.