[ 
https://issues.apache.org/jira/browse/CASSANDRA-18012?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17632463#comment-17632463
 ] 

Andres de la Peña commented on CASSANDRA-18012:
-----------------------------------------------

{quote}and the -f designation won't actually successfully run to completion.
{quote}
I think you meant {{{}-l{}}}, the flag for setting lower resources. {{-f}} is 
used to force the acceptance of unknown env vars, which is not related to the 
resorce usage.
{quote}all that needs to happen is to remove the -h and -f from the script. 
This would include removing the check for $hires and $check_env_vars variables 
as the code blocks associated with same.
{quote}
{{$check_env_vars}} doesn't have any relation with the resources config. Maybe 
the mention of {{-f}} instead of {{-l}} led to think that it was related to the 
resources config.
{quote}(probably print a warning that it's generating config w/subset of full 
tests that should not be used to validate commits in big warning letters)
{quote}
Perhaps we could also use a different workflow name for each of the two 
configs, that should be hard to miss on the GUI.

> Remove -l / -m / -h designation and have two options: free or paid tier 
> circle config
> -------------------------------------------------------------------------------------
>
>                 Key: CASSANDRA-18012
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-18012
>             Project: Cassandra
>          Issue Type: Task
>          Components: CI
>            Reporter: Josh McKenzie
>            Priority: Normal
>
> Currently the -h designation is wasteful and should not be used, and the -f 
> designation won't actually successfully run to completion.
> We should default to a "free tier" config (probably print a warning that it's 
> generating config w/subset of full tests that should not be used to validate 
> commits in big warning letters) that runs a subset of the test suites for 
> users working on patches to validate their work (unit test only? unit + 
> in-jvm dtest? TBD), and add a flag to generate a config using larger 
> containers + parallelization for the "paid" tier for folks w/paid circleci 
> accounts.,
> It looks like -p is available right now fwiw.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to