Pr docker factory travis ci test #105
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ction Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I945002c53265ea849ba29a6e3290af3cccb5f4e9 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450048
…ined regex Instead of doing it's own thing, let's use our defined regex and ipaddress to help us out Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I852b2c4022203c8ff9b88aeedc45a2511fe427fc Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450049
If an env does not have ipv6 setup, we don't want to fail so let's try some sanity checking and don't expect it to start properly if things are not defined Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I9b3b4b1909de65e2f3009f9f0bafcfb4999cda55 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450050
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I4e4bb8207e63eff40b43e1ea4dcf168cb58b0ab9 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450061
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ief922097d0f733fb39ff725c96e4ac9989c637dc Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450066
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I2f7cf55598ebe69147fc70d38857d68b980563d1 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/449937
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I6ec2a4166515d681f93e1be08e24d67ca8159ad2 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450082
Still lots of work to do, checking CM cfg, some missing possibilties like bridged and ipv6 only Finally, removed routing through WAN container as it was too flaky and not working for IPv6. We can revisit this later but it's quite possible the default config for a lab would not allow this anyways Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Id25126e32edae234a41aabcdc14a5688c7965f12 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/449831
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ia598e6e2af161482580001c05afc949dd1ba7334
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I429fee6e2daafdcba232c2a2ad9dfea6671463ba Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450088
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I429fee6e2daafdcba232c2a2ad9dfea6671463ba
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ibc5a8a4e4eeb93992913553ed415ef9fc3385ba7
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I8db0a224dd7fa6ee8e750ef508c3ac6a473601ae Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450196
Otherwise this is a simple string Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Id63b338d576ef6ccc2f12116c8fec3376da82b4f Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450221
Not 100% sure how to handle this yet, but let's get an ipv6 addr first so if we do end up getting and ipv4 addr we also get an ipv4 nameserver. In the future this all needs to be cleaned up because the LAN client might not even get an ipv4 address Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I1c35cda11b20795de747edfcbd3a04a5dec99dd0 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450323
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ia59fb9be351f5891436b4ca0263cab290b15b8bb Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450324
snmp_mib_set() - Modified for reg-ex for getting hex-string value
Signed-off-by: Michele Gualco <[email protected]>
add select option by name for GUI Signed-off-by: gerben <[email protected]>
This fixes a bug with v4 vs v6 dhcp client for wan Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I831817b4e8d2ed79e6c56010355c85f292522b27 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450950 Reviewed-by: Michele Gualco <[email protected]>
This is decided later, so we can't set at parse time otherwise we will get a stale address Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I0c878e4ce250b946a8d6824d69e26a2b16ec1228 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450960
This way we won't use the default one unless it's configured as such. Also we will know if things are wrong because the address will be clearly wrong as None is not an ipv6 address Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I9196f3cc02b578f11dc9c49052d631bec19caa23 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450961
Edits the /etc/gai.conf file, this is to give/remove ipv4 preference (by default ipv6 is preferred) No need to restart ipv4/6 See /etc/gai.conf inline comments for more details Signed-off-by: Michele Gualco <[email protected]> Change-Id: I68fe5149bb34f854424244dbbe455faaf12d4ef8 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451191 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
No functional change, just adding more attribute definitions, remove hash size to make things more simple, mark items that need to move to host section Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ie925d3ce2b700b13741662c199d4528116b6ef6f Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/450227 Reviewed-by: Michele Gualco <[email protected]> Reviewed-by: ketan tewari <[email protected]>
This let's us save data from output to the working directory, among other files Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I65ef1e42cdbd9e64e5c6f1fb4a87c32c5a39a1af Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451309
This is causing issues installing packages later, so let's force to ipv4 because we don't test installing packages we just want to use them Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I2123dcfc53cb78a7f9fa96c73c6a982f6550dd0e Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451363
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ic7c36b0fa079bde813af42d38e6b4213a6fe0251 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451378
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I47d6fa5c11232b357583b55f28714723eeb70a92 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451379
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ic62388abe7bff432b19cf9693c7a3bd387d97315 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/451385
Created oe module in devices Signed-off-by: Panimalar Palaniappan <[email protected]> Change-Id: I48c51e5639518eabbfb3f49af32447660ce6c2ab Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/463884 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Michele Gualco <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Mike Anderson <[email protected]> Change-Id: I2387849eaf5bcc96cb4513dd97d2ea4fdda6e0c8 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/463921 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Mike Anderson <[email protected]>
Signed-off-by: Mike Anderson <[email protected]> Change-Id: Ib1386f58e7fa0b68657ba4180956a44c99a78144 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464247 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Mike Anderson <[email protected]>
This has no functional change for boardfarm as it is now because currently you can only run bft from one single directory. But someday soon we will be able to run bft from any directory. When that happens let's set the default output directory to be "results/" within the user's current working directory. Otherwise the default directory will be in some random other directory wherever they install python modules. Not good. Signed-off-by: Mike Anderson <[email protected]> Change-Id: I597a9bf7dd7e195181b711a802531ebaf97b5d0a Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464033 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Shakthi Ravichandran <[email protected]> Change-Id: I9647f91d764eb05184604b8c71571f56a991e22d Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/463781 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
Signed-off-by: Mike Anderson <[email protected]> Change-Id: Ic037c42386e0f247c82c203ccf8feefdd4525b7b Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464424 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Panimalar Palaniappan <[email protected]> Change-Id: I45be3bd38f24f60dfee4b0a933eff2250d911323 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464193 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
Maybe the function get_interface_ip6addr() was never called, because it had 3 undefined variables: AllValidIpv6AddressesRegex, ipaddress, and ip6addr. Now fixed. Signed-off-by: Mike Anderson <[email protected]> Change-Id: I1462d5ce3b9e3a6dfc736f9c28c61ff7d45db620 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464249 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Mike Anderson <[email protected]> Change-Id: I237d715b0855b507270fa36547f368375e8d774c Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464315 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Some of this should not be required since we should not depend on the docsis overlay, so there are some areas for improvemnet Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I00cf1fd95d58647db18b66b289f68164c2e0d548 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464440 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Mike Anderson <[email protected]> Change-Id: If7b55f7ca6d91ea0905f4a8b7c236fa627587a7c Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464248 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I231819dee7876cda2da41a1736c68f5a4e5a014d Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464446 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I4ec6b272dc5a6310da99de73d7297b5ff1de8360 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464447 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
This is in preparation to making boardfarm a standard python module
that we can import and write python scripts for.
Two symlinks have been created: bft, and deploy-boardfarm-nodes.sh
So you can still run "./bft" in the base directory.
No other files were created, only moved.
Near future commits will add '__init__.py' and other files to make
boardfarm a proper module.
An outline of commands for this commit:
mkdir boardfarm
mv * boardfarm/
git rm <files and directories in old location>
git add <files and directories in new location>
Signed-off-by: Mike Anderson <[email protected]>
Change-Id: If8e8d952ee1329f6a764bf5e0e920cc9c16c987e
Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464355
Tested-by: Matthew McClintock <[email protected]>
Reviewed-by: Matthew McClintock <[email protected]>
Probably need to fix another way eventually, just don't want to back out the bigger change Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I0862cb17bc371f25aadcbf98fa50068e359ae2f3 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464465 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I945f9159ef74b5c811a807e52b49978a3a61413f Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464470 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
…ocations w/ overlays Mostly this fixes BFT_OVERLAY being a relative path from the CWDIR. I suggest using full paths going forward though Also, we insert paths for devices in overlays, but we never inserted the path for devices in the main repo, so we need to do that too This should just be a small fix for better improvements later Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Ibf0d70ac93045cfae428ae76a4508436128aef25 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464538 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
Signed-off-by: Michele Gualco <[email protected]> Change-Id: Idf650a30cdda43bbcb4c4f0bbe3188c4b5522931 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464513 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Panimalar Palaniappan <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
The class BaseDevice has a function expect_prompt() to replace use of expect(prompt). This simplifies imports. Signed-off-by: Mike Anderson <[email protected]> Change-Id: Ieb4231a933683a9484f55b2b3c5e6f0454ce5a49 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464570 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Michele Gualco <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
To install boardfarm in developer mode:
pip install -e .
That means changes you make to the files are automatically
available in the installed package.
Or install normally: pip install .
Either way, you can "import boardfarm" anywhere (doesn't
do much at the moment, still a work in progress.) Or run
bft anywhere (also a work in progress).
Signed-off-by: Mike Anderson <[email protected]>
Change-Id: I64c51f29ba6d900617276cd16a75baf3039715f5
Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464580
Tested-by: Matthew McClintock <[email protected]>
Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I6f65683dcda391c90355e355622c3bf25a04f4c9 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464590 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Mike Anderson <[email protected]>
We just started dnsmasq caching nameserver, let's just use that Signed-off-by: Matthew McClintock <[email protected]> Change-Id: If11892ce0a7bd3f42530458d12d9f756cae2eef5 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464702 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: Michele Gualco <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I1e7351edb1bc5598d4e454957b267ac9d90cbb9a Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/463782 Tested-by: Matthew McClintock <[email protected]> Reviewed-by: ketan tewari <[email protected]> Reviewed-by: Matthew McClintock <[email protected]>
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: Id1a6014965f5c712e89c9e6a34d28781c7e4df75 Reviewed-on: https://review.gerrithub.io/c/lgirdk/boardfarm/+/464586 Reviewed-by: Matthew McClintock <[email protected]> Tested-by: Matthew McClintock <[email protected]>
mattsm
force-pushed
the
pr_docker_factory_travis_ci_test
branch
3 times, most recently
from
20aeb28
to
6857d58
Compare
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I87325a662e3f2992ddde86594e6b5e3890e3dbb7
Signed-off-by: Matthew McClintock <[email protected]> Change-Id: I84879ad3c435171cd6d9c99cefcb4632a9f059f6
mattsm
force-pushed
the
pr_docker_factory_travis_ci_test
branch
from
6857d58
to
865f710
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.