Skip to content

Conversation

opudrovs
Copy link
Contributor

@opudrovs opudrovs commented Oct 12, 2022

Closes
weaveworks/weave-gitops#2772
weaveworks/weave-gitops#2811

  • Added support for new WG OSS commands gitops set config and gitops add config in enterprise.

The user ID set with gitops set config will be used with Pendo analytics.

Please see the tickets' acceptance criteria for full examples of running the commands. But in general you can test the commands with make cmd/gitops/gitops and running:

gitops set config analytics true
gitops get config

@opudrovs opudrovs added enhancement New feature or request team/denim labels Oct 12, 2022
@opudrovs opudrovs changed the title Add gitops set config and gitops get config commands from core. Add gitops set config and gitops get config commands from WG OSS. Oct 12, 2022
@opudrovs opudrovs force-pushed the add-gitops-set-config-and-get-config-commands branch from 4376f95 to a39acf2 Compare October 12, 2022 12:00
@opudrovs opudrovs marked this pull request as ready for review October 12, 2022 12:12
@opudrovs opudrovs requested review from a team and foot October 12, 2022 12:12
Copy link
Contributor

@ozamosi ozamosi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good - @foot do you want us to hold off on merging this until v0.9.6 is stable?

@foot
Copy link
Collaborator

foot commented Oct 12, 2022

Code looks good - @foot do you want us to hold off on merging this until v0.9.6 is stable?

Yes please! Will try and do it tomorrow.

@opudrovs opudrovs force-pushed the add-gitops-set-config-and-get-config-commands branch 3 times, most recently from 2c79985 to 0632ca3 Compare October 18, 2022 10:31
@foot
Copy link
Collaborator

foot commented Oct 18, 2022

Hello! main is on #1695 now, so this would be great to merge sans the changes to go.mod/sum

@opudrovs
Copy link
Contributor Author

@ozamosi we are not merging this one yet too, until the legal docs are ready, correct?

@opudrovs opudrovs force-pushed the add-gitops-set-config-and-get-config-commands branch from 0632ca3 to be423b7 Compare October 19, 2022 10:59
@bigkevmcd
Copy link
Contributor

@opudrovs Can we move this forward?

@opudrovs
Copy link
Contributor Author

opudrovs commented Nov 7, 2022

@bigkevmcd I am on vacation now and will be able to check it on Monday. This and other Pendo WG OSS PRs are waiting for a Pendo-related documentation update. If it is urgent please ask @ozamosi or @josefaworks if these PRs are still blocked.

@opudrovs opudrovs force-pushed the add-gitops-set-config-and-get-config-commands branch from be423b7 to 7c0f81a Compare November 24, 2022 02:43
@opudrovs opudrovs force-pushed the add-gitops-set-config-and-get-config-commands branch from 7c0f81a to af1f5ee Compare November 24, 2022 12:17
@opudrovs opudrovs merged commit ab47002 into main Nov 24, 2022
@opudrovs opudrovs deleted the add-gitops-set-config-and-get-config-commands branch November 24, 2022 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants