[ 
https://issues.apache.org/jira/browse/KAFKA-19418?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lianet Magrans updated KAFKA-19418:
-----------------------------------
    Description: 
Currently, the verifiable producer sys test relies on the "is_version" check to 
validate the jar versions of the process it runs. Ex. 
https://github.com/apache/kafka/blob/b8fc96272e746c4d180ad278a86a565dccefc8f7/tests/kafkatest/sanity_checks/test_verifiable_producer.py#L74
 

That check lists processes filtering by a provided name, but the verifiable 
producer does not pass any. This makes that the default "kafka" word is 
"grepped". We've seen this leading to flakiness (the is_version expects only 1 
process, here 
https://github.com/apache/kafka/blob/b8fc96272e746c4d180ad278a86a565dccefc8f7/tests/kafkatest/utils/util.py#L61
 but finds several process meeting the ps/grep condition)

To avoid flakiness we should consider having the verifiable producer passing a 
proc name for it. 

  was:
Currently, the verifiable producer sys test relies on the "is_version" check to 
validate the jar versions of the process it runs. 

That check lists processes filtering by a provided name, but the verifiable 
producer does not pass any. This makes that the default "kafka" word is 
"grepped". We've seen this leading to flakiness (the is_version expects only 1 
process, here 
https://github.com/apache/kafka/blob/b8fc96272e746c4d180ad278a86a565dccefc8f7/tests/kafkatest/utils/util.py#L61
 but finds several process meeting the ps/grep condition)

To avoid flakiness we should consider having the verifiable producer passing a 
proc name for it. 


> Verifiable producer sys test should consider process name to avoid flakiness
> ----------------------------------------------------------------------------
>
>                 Key: KAFKA-19418
>                 URL: https://issues.apache.org/jira/browse/KAFKA-19418
>             Project: Kafka
>          Issue Type: Test
>          Components: clients, system tests
>            Reporter: Lianet Magrans
>            Assignee: Lianet Magrans
>            Priority: Minor
>
> Currently, the verifiable producer sys test relies on the "is_version" check 
> to validate the jar versions of the process it runs. Ex. 
> https://github.com/apache/kafka/blob/b8fc96272e746c4d180ad278a86a565dccefc8f7/tests/kafkatest/sanity_checks/test_verifiable_producer.py#L74
>  
> That check lists processes filtering by a provided name, but the verifiable 
> producer does not pass any. This makes that the default "kafka" word is 
> "grepped". We've seen this leading to flakiness (the is_version expects only 
> 1 process, here 
> https://github.com/apache/kafka/blob/b8fc96272e746c4d180ad278a86a565dccefc8f7/tests/kafkatest/utils/util.py#L61
>  but finds several process meeting the ps/grep condition)
> To avoid flakiness we should consider having the verifiable producer passing 
> a proc name for it. 



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

Reply via email to