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

Configurable Discovery #1

Open
wants to merge 12 commits into
base: master
Choose a base branch
from
Open

Configurable Discovery #1

wants to merge 12 commits into from

Conversation

agomez08
Copy link

[x] This contribution adheres to CONTRIBUTING.md.

TODO: Check the above box with an 'x' indicating you've read and followed CONTRIBUTING.md.

What does this Pull Request accomplish?

The user is now capable of choosing how targets are discovered from the network. User can select from the next options:

  1. Automatic discover. Default behavior.
  2. Only add targets provided in a configuration file.
  3. Only add a LinuxRT target that is connected over USB.
  4. Do not discover targets and manually add them 1 by 1.

Why should this Pull Request be merged?

Some users complain that RAD takes a long time to discover targets on the network. A big part of them are not needing to automatically discover targets from the network since they already know their IP Addresses. With this feature, users can save time by configuring how RAD adds targets.

What testing has been done?

  1. Tested the 4 discovery methods for several target types.
  2. Automatically discovering 6 targets took 21 seconds on my computer. Providing a list with these IP addresses causes that RAD takes 10 seconds adding them. If it is only 1 target the time is reduced to close to 3-4 seconds.
  3. Compiled Exe with LV 2014. Tested retrieving/deploying an image to VxWorks, LinuxRT and Pharlap targets. This modification shouldn't affect any operation related to retrieving and deploying images.

…initializing. Otherwise, it looks like it is frozen for a few seconds.
…et once the user provides a valid IP Address. Adding a progress dialog so the user can tell RAD is connecting to the target after pressing Get Info. Otherwise, it looks like it is frozen for a few seconds.
…ll contain the settings related to how targets are discovered from the network and will be loaded from a configuration file.

-Added extra code in the rad_Load Config VI to load from the configuration file rad_config.ini the settings related to how targets are discovered in the network.
-Added extra code in the rad_Save Config VI to save to the configuration file rad_config.ini the settings related to how targets are discovered in the network.
…scovered from the network. The name of this state is “Configure Discovery” and it is called from the “Init – Load Config” state. For now, it is only called when RAD starts.

-The user can select from 4 different options:
a.	Automatically discover targets in my local network. This is the normal default behavior from RAD. It tries to discover all the available targets on the local network.
b.	Discover targets defined in a configuration file. It will only try to connect and list the targets defined in a file. The expected format for this file is 1 target per line, listed by IP Address.
c.	Only add 1 target that is connected over USB. This only applies for Linux RT targets that can be connected through USB with the IP Address 172.22.11.2.
d.	Do not discover targets. I will add my targets manually 1 by 1. No targets will be searched from the network. The user can add them manually 1 by 1 later.
This is implemented in the rad_Configure Discovery Settings.vi by modifying the User Added Targets and Target Types parameters in the main state cluster.
…when needed while the utility is running and not only when it launches.
…tly showing up as disconnected. With this we limit the possibility of trying a retrieve or deploy.
…d tested deploy/retrive of images for LinuxRT, Pharlap and VxWorks.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant