Gitops mode

[knative]

Gitops mode

[rhuss]

@itsmurugappan sorry for the late reply, but this sounds like a good strategy. I don't think you have to implement every method, only those that are needed for this functionality. Also the file name should then be part of that new implementation's struct.

Also I learned this trick today (thanks to @markusthoemmes ):

If you have a big interface to implement but only call a few functions, you can define the struct like this:

type mock struct {
  my.Interface
}

It'll implement all functions with nil-pointer function calls. Then you only have to implement the functions your test actually calls

This might be helpful if you only need to instgal some of the methods (like service create or update)

[sixolet]

Considerations:

• Downloading the right objects from etcd to a directory for the first time may another gitops usecase.
• Anything that reads status and uses that information becomes harder or impossible (ex, locking revisions to a container digest)
• Keeping only the "interesting" fields might make the directory easier to use.

[cppforlife]

Let's have a mode where you can provide a directory instead of a server to connect to

im somewhat skeptical that this would be that useful if users are using gitops workflow (== more familiar with yamls). what commands do you think will benefit from this?

[nimakaviani]

throwing this here since it was discussed during today's call:

knative yaml generator
https://github.com/julz/knightrider

[github-actions]

This issue is stale because it has been open for 90 days with no
activity. It will automatically close after 30 more days of
inactivity. Reopen the issue with /reopen. Mark the issue as
fresh by adding the comment /remove-lifecycle stale.

[itsmurugappan]

/remove-lifecycle stale
/assign

[itsmurugappan]

@rhuss @navidshaikh i think this is the issue we discussed in the last wg. For implementing this am planning to take the below approach, please let me know if this is good

• Implement knclientinterface to work on local directory
• change this line in all commands to return the appropriate client based on cmd flags
  client, err := p.NewServingClient(namespace)

[rhuss]

(sorry closed by accident)