Giter Club home page Giter Club logo

Comments (7)

scothis avatar scothis commented on September 21, 2024 1

Just FYI, for tilt we require the --export (in general) to be able to return the workload in the cluster. Tilt requires the uid field which is what it then bases its tracking off of.

The idea behind --export is that it takes a resource in the cluster and makes it appropriate to write into a gitops store and share. It intentionally strips off the uid and other cluster specific fields. You want something more like -o yaml for create/update/apply

from apps-cli-plugin.

heyjcollins avatar heyjcollins commented on September 21, 2024

comments originally submitted by @paulcwarren

Just FYI, for tilt we require the --export (in general) to be able to return the workload in the cluster. Tilt requires the uid field which is what it then bases its tracking off of.


We would also require the ability to --quiet any other output that isn't the eventual workload yaml.

from apps-cli-plugin.

heyjcollins avatar heyjcollins commented on September 21, 2024

@paulcwarren - do you see any issue with --export implying --yes and --quiet?

from apps-cli-plugin.

scothis avatar scothis commented on September 21, 2024

do you see any issue with --export implying --yes and --quiet?

I do, --yes is an explicit opt-in to not be prompted before potentially destructive actions. Making it implicit weakens its purpose. In the case here of using a script to drive the command, the script can specify the extra flags it cares about. One flag changing the implicit value of an unrelated flag is a good way to surprise users.

from apps-cli-plugin.

heyjcollins avatar heyjcollins commented on September 21, 2024

@scothis - I agree with the sentiment in your previous comment and have updated the issue accordingly.
--yes is not implied if a user includes --export (users must provide --yes to skip prompts).

from apps-cli-plugin.

scothis avatar scothis commented on September 21, 2024

It's worth double checking that the prompts are on stderr so that stdout can be piped to another tool with the exported content.

from apps-cli-plugin.

heyjcollins avatar heyjcollins commented on September 21, 2024

After further clarifying discussions with @paulcwarren and my team, I've decided against implementing this and we'll stick with the 2 command approach which works today.
apps workload apply... followed by apps workload get -o yaml/json to get the uid.

from apps-cli-plugin.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.