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.

Sign up for GitHub

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

Jump to bottom

cli: update demo error message for expired license #40434

Closed
awoods187 opened this issue Sep 3, 2019 · 2 comments · Fixed by #81737
Closed

cli: update demo error message for expired license #40434

awoods187 opened this issue Sep 3, 2019 · 2 comments · Fixed by #81737
Labels
A-cli-client CLI commands that pertain to using SQL features A-demo A-partitioning A-sql-execution Relating to SQL execution. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) S-3-ux-surprise Issue leaves users wondering whether CRDB is behaving properly. Likely to hurt reputation/adoption.

Comments

@awoods187
Copy link
Contributor

awoods187 commented Sep 3, 2019 •
edited by cockroach-jira-scripts
Loading

Demo now uses enterprise licenses by default. It comes with a one hour limit. If you attempt to use a feature that requires an enterprise license, you will receive this message:

pq: Use of replication zones on indexes or partitions requires an enterprise license. Your evaluation license expired on September 3, 2019. If you're interested in getting a new license, please contact [email protected] and we can help you out.

This may be confusing to demo users since they do not expressly get a license and its confusing that the license message will be for the same day.

We should enhance this message to say something like this for demo:

Your enterprise license for Cockroach Demo expired after 1 hour. To obtain a full trial license click here or reach out to [email protected] and we can help you out.

Jira issue: CRDB-6341
@awoods187 awoods187 added A-sql-execution Relating to SQL execution. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) S-3-ux-surprise Issue leaves users wondering whether CRDB is behaving properly. Likely to hurt reputation/adoption. labels Sep 3, 2019
@awoods187 awoods187 mentioned this issue Sep 3, 2019
Closed
12 tasks
@rohany
Copy link
Contributor

rohany commented Sep 11, 2019

cockroach demo has been changed to give a 24 hour license now. Will this still be an issue?

@awoods187
Copy link
Contributor Author

I still think we should do this but it isnt a high priority issue

@knz knz added A-cli-client CLI commands that pertain to using SQL features and removed A-cli labels Mar 20, 2021
@knz knz mentioned this issue May 24, 2022
Merged
@craig craig bot closed this as completed in 63924f9 May 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-cli-client CLI commands that pertain to using SQL features A-demo A-partitioning A-sql-execution Relating to SQL execution. C-enhancement Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception) S-3-ux-surprise Issue leaves users wondering whether CRDB is behaving properly. Likely to hurt reputation/adoption.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

cli/demo: fix enterprise features for multitenancy knz/cockroach
3 participants
@knz @rohany @awoods187