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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrering til urfave/cli #209

Merged
merged 29 commits into from
Jul 3, 2023
Merged

Migrering til urfave/cli #209

merged 29 commits into from
Jul 3, 2023

Conversation

Kyrremann
Copy link
Contributor

@Kyrremann Kyrremann commented Jun 28, 2023

Denne PR-en bytter fra spf13/cobra og viper til urfave/cli. Det er ikke gjort store endringer utenom å bytte bibliotek. Litt forenkling av kubeconfig da https://github.com/nais/narcos har tatt over behovet for nais naas.

"Modellen" er også blitt endret, nå ligger alt som kun har med cli-commands under ./cmd, mens koden som gjør ting ligger under ./pkgJeg mener dette gjør koden enklere å vedlikeholde og teste. Pluss det gir oss muligheten til å gjenbruke ting på tvers av nais/narc ved behov (som ved kubeconfig-generering).

fixes: #202

@Kyrremann Kyrremann changed the title Urface/cli Migrering til urface/cli Jun 28, 2023
@Kyrremann Kyrremann marked this pull request as draft June 28, 2023 10:01
@Kyrremann Kyrremann self-assigned this Jun 28, 2023
@Kyrremann Kyrremann changed the title Migrering til urface/cli Migrering til urfave/cli Jun 28, 2023
@Kyrremann
Copy link
Contributor Author

Manuelt testet følgende:

  • nais validate
  • nais kubeconfig (default er nå uten onprem!)
  • nais device
  • nais start

Gjenstår å teste aiven og postgres.

Her kan ting feile senere i prosessen fordi man kan ha kjørt
'--update-adc' som har gått ut på tid, eller for en annen konto.
@Kyrremann
Copy link
Contributor Author

Og da er alt testet, mangler å sikre #67 ikke skjer på alle kommandoen til Postgres, da dette feiler ganske langt nede i stacken, så får ikke lagd en catch all.

@Kyrremann Kyrremann marked this pull request as ready for review June 30, 2023 08:49
@Kyrremann
Copy link
Contributor Author

#yolo

@Kyrremann Kyrremann merged commit 2105214 into master Jul 3, 2023
@Kyrremann Kyrremann deleted the urface/cli branch July 3, 2023 11:39
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.

får fss contexts / clustere når jeg kjører nais naas kubeconfig
1 participant