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.

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

execution.properties not working while using the api #192

Closed
orbenda1905 opened this issue Apr 22, 2018 · 2 comments
Closed

execution.properties not working while using the api #192

orbenda1905 opened this issue Apr 22, 2018 · 2 comments
Assignees
Milestone

Comments

@orbenda1905
Copy link

Hi,

I'm creating an execution through the API and I need to add some execution.properties in order to filter in the Kibana. (I'm using the JSystem's script - runScenario.bat)
I did it as it says on the wiki and it doesn't work.
While debugging, I noticed that the properties shows in the ExecutionMetaData but not in the scenario properties like it should be.
When I use JSystem's GUI and edit the remoteDifido.properties file with the execution.properties it works fine (it shows on the scenario properties map while debugging).

Thanks

@itaiag itaiag self-assigned this Oct 29, 2018
@itaiag itaiag added the bug label Oct 29, 2018
@itaiag itaiag added this to the 2.2.05 milestone Oct 29, 2018
@itaiag
Copy link
Contributor

itaiag commented Oct 29, 2018

The execution properties and the scenario properties are actually not exactly the same thing. The execution properties are added as additional columns in the execution list in the main server view while the scenario properties are added to the HTML report in the overview page and are also copied to the Elastic search entities.
The reason that you see the execution properties as scenario properties when running from JSystem or any other framework is that the binder is responsible for adding the execution properties as scenario properties.
When You are creating the execution using the API, you are bypassing the binder mechanism so the execution properties are not included in the scenario properties.

@itaiag
Copy link
Contributor

itaiag commented Oct 29, 2018

Since this is a requirement that I get from many users, I am considering performing the scenario properties update in the server side instead in each one of the binders.

itaiag added a commit that referenced this issue Oct 29, 2018
@itaiag itaiag closed this as completed Mar 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants