Making the host ip device configurable

[MaciejKaszynski]

• Currently the IP address of the QEMU VM is hard-coded as the parameter has the default value that is never set this adds an optional config field host_ip.
  • This is backward compatible as the default will still be the old IP address.
• For the QEMU instance also made the second network device optional and dependent on having the host_diagnostic_ip optional configuration field.
• Made the data_router_config configuration field optional so that if nothing is configured no dlt is created.
• Made all the targets take in an ECU config as a parameter instead of having each config field as a parameter.

[draganbjedov]

You are missing with. DltReceive is context manager. In other targets as well.

[MaciejKaszynski]

Fixed on 94d6916

[draganbjedov]

Suggested change

	if target_config.data_router_config:
	dlt = DltReceive(
	target_ip=target_config.ip_address,
	protocol=Protocol.UDP,
	data_router_config=target_config.data_router_config,
	binary_path=test_config.dlt_receive_path,
	)
	dlt.start_process()
	target = TargetQvp(test_config.ecu, test_config.os)
	target.register_processors(qvp_process)
	yield target
	target.teardown()
	finally:
	if dlt:
	dlt.kill_process()
	if target_config.data_router_config:
	dlt = DltReceive(
	target_ip=target_config.ip_address,
	protocol=Protocol.UDP,
	data_router_config=target_config.data_router_config,
	binary_path=test_config.dlt_receive_path,
	)
	with dlt if dlt else nullcontext():
	target = TargetQvp(test_config.ecu, test_config.os)
	target.register_processors(qvp_process)
	yield target
	target.teardown()

[MaciejKaszynski]

Done on e5f27a5

[draganbjedov]

Please squash related commits and then I can approve.

[MaciejKaszynski]

Please squash related commits and then I can approve.

Done

[draganbjedov]

@MaciejKaszynski Did you tested this by running ITF tests from reference_integration?

[MaciejKaszynski]

@MaciejKaszynski Did you tested this by running ITF tests from reference_integration?

Locally I get a pass for qnx_qemu image. I used bazel test --config=qemu-integration //:test_persistency_qemu //:test_ssh_qemu //:test_scrample_qemu I think those are the only itf ones.

[draganbjedov]

You need to rebase. We have changed ITF structure #30