contrail-controller changes for Windows
Affects | Status | Importance | Assigned to | Milestone | ||
---|---|---|---|---|---|---|
Juniper Openstack | Status tracked in Trunk | |||||
R5.0 |
Won't Fix
|
High
|
Pawel Marchewka | |||
Trunk |
In Progress
|
Undecided
|
Pawel Marchewka | |||
OpenContrail |
Fix Committed
|
Undecided
|
Pawel Marchewka |
Bug Description
Merge contrail-controller changes for Windows from github.
Initial changes:
https:/
Further work and cleanup:
https:/
Changed in opencontrail: | |
assignee: | nobody → Pawel Marchewka (pawelmarchewka) |
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #1 |
OpenContrail Admin (ci-admin-f) wrote : | #3 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #4 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #5 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #6 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #9 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #11 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #13 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 382cccd87dd1565
Author: Michał Cłapiński <email address hidden>
Date: Wed Dec 13 17:58:20 2017 +0100
General changes for Windows
Mixed changes related to net,vnsw/
1. boost::
IpYAddress:
and on Windows we compile with c++11.
2. addr.to_
Difference between boost::array and std::array
3. variable length arrays to dynamic allocation with smart pointers
4. s_addr to s_address, DELETE to DEL, EVENT_MAX to EVENT_MAXIMUM
s_addr, DELETE, EVENT_MAX are reserved on Windows
5. bind to ::bind
To avoid confusion with std::bind
6. and more various bugs
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I7b963703d6d0ae
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #14 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 442663f4e8d7c15
Author: Pawel Marchewka <email address hidden>
Date: Fri Dec 15 12:36:45 2017 +0100
Changes for windows
Changes for windows:
1. Do not build usock_server on windows
2. Use uint8_t instead of u_int8_t
3. Added cast in setsocktopt
4. Changed size_t to std::size_t in some methods (to match to
declaration)
Change-Id: I1336413e541e25
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #15 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #16 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #18 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #19 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #21 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #22 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #26 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 05212ee13413504
Author: Mateusz Jablonski <email address hidden>
Date: Thu Dec 14 17:04:12 2017 +0100
General changes for Windows
Mixed changes related to schema, xmpp, vnsw/agent/{kstate, vgw, test-xml}:
1. boost::
IpYAddress:
otherwise and on Windows we compile with c++11.
2. add linking with advapi32 and gdi32 on Windows
(needed by OpenSSL)
3. do not link with librt on Windows
4. use multiplatform boost::filesystem instead of open/stat/read
5. add missing 'return' to some functions
6. add missing include
7. and more various bugs
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Ifa9a6182300a16
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #27 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #28 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #29 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #30 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #31 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #32 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #33 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #35 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit e83c61b714058b4
Author: Michał Cłapiński <email address hidden>
Date: Tue Dec 19 15:53:04 2017 +0100
src/vnsw/agent changes for Windows
1. Created empty class VnswInterfaceLi
2. Disabled some libs and flags in SConscript for Windows
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Ifa57e5a2b3f0e8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #36 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 1a587b0ae387717
Author: Pawel Marchewka <email address hidden>
Date: Tue Dec 19 14:36:11 2017 +0100
Agent services changes for Windows
Agent services changes for Windows:
1. Variable length arrays is only supported in gcc and it is not a part
of C++ standard. We decided to use dynamic allocation and scoped_array.
2. Empty (zero element) array at the end of structure is not portable.
3. Use boost::
uses std::array instead of boost::array
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I938ae8b6b9e174
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #37 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 50c2fdbe648edd2
Author: Pawel Marchewka <email address hidden>
Date: Fri Dec 15 14:48:33 2017 +0100
Don't build nexthop_client and nexthop_server on Windows.
NexthopDBServer uses UnixDomainSocke
Windows.
Change-Id: Ibf117424629b66
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #38 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #39 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #40 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #41 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #42 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #43 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #44 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #45 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #47 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #48 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #49 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #50 |
Review in progress for https:/
Submitter: Katarzyna Rybacka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #51 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 8b3cc8b40588295
Author: Mateusz Jablonski <email address hidden>
Date: Thu Dec 21 13:13:58 2017 +0100
Fix for signed-unsigned comparison
Fix for signed-unsigned comparison in test_xml.cc
Change-Id: I191267d18c8e44
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #52 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 37dbe7724b0c4d9
Author: Pawel Marchewka <email address hidden>
Date: Thu Dec 21 12:08:16 2017 +0100
Boost thread linking issue
Fix for boost thread linking issue in tests.
Change-Id: Icef7f3fcee5ea2
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #53 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit c846bf5fc491977
Author: Katarzyna Rybacka <email address hidden>
Date: Tue Dec 19 16:25:51 2017 +0100
Agent dns changes for Windows
Agent dns changes for Windows:
1. Changed 'libxml2' to 'xml2', so it could be included on Windows either.
2. Move boost_chrono not to be included on Windows because std::chrono is used
3. Changed lib prefixes and suffixes to be exact for every platform
4. Removed cmn_sources due to dependency cycle in scons
5. Ifndefed two of bind_tests because Named Config is not used on Windows
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I623012d492a83f
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #54 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #55 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #56 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #57 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit c3242d424531e35
Author: Pawel Marchewka <email address hidden>
Date: Thu Dec 14 11:23:59 2017 +0100
Partial SNAT support for Windows
Partial SNAT support for Windows:
1. Added vrouter_hyperv.py and vrouter_
Those files setup SNAT VM and setup HYPERV
2. Moved 'validate_uuid' function to separate file common.py
to not install whole vrouter_netns.py on Windows
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I85d9cb7b3932b5
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #58 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit c1586d32c629088
Author: Michał Cłapiński <email address hidden>
Date: Tue Dec 19 19:11:01 2017 +0100
src/vnsw/
1. Used env['LIBPREFIX'] and env['LIBSUFFIX'] instead of 'lib' and '.a'
2. Added code in SCons and .xml for MSI generation using WiX
3. Used forward_adapter while registering boost::factory
4. Added config for Windows Service Wrapper
5. Disabled vhost creation on Windows
Change-Id: I2f1189b45aa8f0
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #59 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #60 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #61 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #62 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #63 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 49147ab5d9a9d0a
Author: Katarzyna Rybacka <email address hidden>
Date: Wed Dec 20 15:43:32 2017 +0100
Agent ifmap changes for Windows
Agent ifmap changes for Windows:
1. Changed DELETE to DEL because DELETE is reserved on Windows.
2. Changed deprecated posix function index() to recommended strchr().
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I578d942b6d879e
Depends-On: Icef7f3fcee5ea2
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #64 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit fa9ef31576ff55f
Author: Pawel Marchewka <email address hidden>
Date: Thu Dec 21 13:18:49 2017 +0100
Changes for Windows
1. Redefinition of strtok_r to strtok_s on Windows
2. Use memset instead of deprecated bzero.
Change-Id: I7fbc331f475853
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #65 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit b2854e582728bbe
Author: Mateusz Jablonski <email address hidden>
Date: Tue Dec 19 17:14:06 2017 +0100
General changes for Windows
Mixed changes related to bgp, db, http, vnsw/agent/cmn, vnsw/agent/uve:
1. change all occurrences of u_intX_t to portable uintX_t
2. add missing include net/if.h header
(for if_nameindex() and if_freenameindex() functions)
3. add missing include unistd.h header
(for sysconf() function)
4. define and use NOEXCEPT (for C++ version-independent noexcept)
5. add const specifier where needed
6. change && in macro to nested macros for MSVC compatibility
7. change closesocket to close_socket
(closesocket() is a Windows built-in function)
8. add empty implementation of uve/vm_stat* classes
(vm_stat is currently not implemented on Windows)
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Iee4b80e105ab2c
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #66 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 122017d811f0535
Author: Pawel Marchewka <email address hidden>
Date: Thu Dec 21 13:26:11 2017 +0100
Changes for windows related to oper directory
1. Added base/os.h which should include redefinition of strtok_r to
strtok_s on Windows
2. Renamed DELETE to DEL
3. Added NULL implementation of instance_task
4. Added os specific implementation of GetOsSpecificPa
5. Added posix_fcntl.h with fcntl stub method - from windows stubs
6. Added optional os_guid_ to Interface - used on windows
7. Use memset instead of bzero
8. Use full scope name of boost::
confilct with std::placeholders
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I0d045dd4a6ec7e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #67 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #68 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #69 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #70 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
tags: | added: vrouter windows |
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #71 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit b825ef4fddb8bd0
Author: Michał Cłapiński <email address hidden>
Date: Tue Jan 2 15:59:52 2018 +0100
Multithreaded build fix for Windows
1. Fixed flakiness in multithreaded build for Windows
2. Added NOGDI flag to compilation
Change-Id: Ib88461e4af5fef
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #72 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 95b45fe7739d320
Author: Pawel Marchewka <email address hidden>
Date: Fri Dec 29 14:26:40 2017 +0100
Added boost_thread to libs (changes for Windows)
Added boost_thread to libs. Boost::thread should be used instead of
pthread, since it is portable. Pthread was not removed from libs since it is still
being used by some not ported (to Windows) controler compoments.
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Ie75ad05b48ff29
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #73 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 80e4ab60418ba14
Author: Mateusz Jablonski <email address hidden>
Date: Wed Dec 20 12:38:27 2017 +0100
src/vnsw/agent/test changes for Windows
1. Create empty implementation of test_util_tap.cc on Windows
2. Used forward_adapter while registering boost::factory
3. Use portable __attribute_
(defined in I5a902da559cf63
4. Change all occurrences of sizeof(pkt.payload) to 0
(sizeof of flexible array is not supported on MSVC)
5. Change pthread to boost::thread
6. Add missing 'return' to some functions
7. Use bind() from global namespace explicitly
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Ie15b27ccd9850b
Depends-On: I5a902da559cf63
Depends-On: I191267d18c8e44
Depends-On: Ie75ad05b48ff29
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #74 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #75 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 9e11a6f922466dd
Author: Pawel Marchewka <email address hidden>
Date: Tue Jan 2 12:47:09 2018 +0100
Added boost_thread to libs (changes for Windows)
Added boost_thread to libs. Boost::thread should be used instead of pthread,
since it is portable. Pthread was not removed from libs since it is still
being used by some not ported (to Windows) controller compoments.
Change-Id: Id4908c8d6fc19e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #76 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit efabdc9a74965e1
Author: Pawel Marchewka <email address hidden>
Date: Fri Dec 29 14:38:07 2017 +0100
Use boost::thread in tests (changes for Windows)
Use boost::thread in tests instead of pthread, since it is portable.
Added boost_thread to libs. Pthread was not removed since it is still
required by some not ported components.
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I68f8637fc9ff2f
Depends-On: Ie75ad05b48ff29
Depends-On: Id4908c8d6fc19e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #77 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #78 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #80 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #81 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #83 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit b72a8e3661dee62
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 5 17:03:31 2018 +0100
Fix type casting in Windows code
Initial work:
https:/
Change-Id: I27d919ac9210e7
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #84 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #85 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #86 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 2b65cff07ada8af
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 5 17:14:22 2018 +0100
Partial fixes for Agent compilation on Windows
1. Change `define` in CCFLAGS to portable version
2. Add new include directory on Windows
3. Remove Windows version of Rapidjson SConscript
(After update updating Rapidjson to v1.1.0)
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: I74847a044c7ecb
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #87 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 2e303a1b4531cfb
Author: Michał Cłapiński <email address hidden>
Date: Thu Jan 4 19:43:49 2018 +0100
Minor fixes for Windows
1. Casted void* to char* before addition
2. Removed one unnecessary define (Windows only)
3. Updated Visual Studio project files
4. Add missing include
Change-Id: I01e72b87320840
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #88 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #89 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #91 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit e4f287083279311
Author: Mateusz Jablonski <email address hidden>
Date: Mon Jan 8 17:03:23 2018 +0100
Partial fixes for Agent compilation on Windows
1. Add new files to windowsstubs
2. Add type casting because of different `{get,set}sockopt` signatures
3. Remove unnecessary files
Change-Id: I64a7d957ad0014
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #92 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit e418b474b371c43
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 5 16:55:54 2018 +0100
CCFLAGS changes for Windows
Initial changes:
https:/
Further work and cleanup:
https:/
Change-Id: Idf7f920bc7b723
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #93 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 0456f0957e2450b
Author: Mateusz Jablonski <email address hidden>
Date: Mon Jan 8 17:21:39 2018 +0100
Partial fixes for Agent compilation on Windows
1. Compile some files on specific platforms only
2. Remove unnecessary files, includes and constants
3. Use std::chrono on Windows
(__cplusplus value is invalid on Windows)
4. Do not use SIGCHLD on Windows
5. Add type casting because of different `{get,set}sockopt` signatures
6. Change boost::
(because of name conflict with eof() function)
7. Remove incorrect implementation of GetContrailVers
Change-Id: Id523d3bae12028
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #94 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #95 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 0a0a983bdf11070
Author: Michał Cłapiński <email address hidden>
Date: Tue Jan 9 20:40:13 2018 +0100
Fixed line endings
1. Fixed line endings (Windows-style to Unix-style)
2. Removed useless file
Change-Id: I9de46f28daf8da
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #96 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #99 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #101 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #103 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #106 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #107 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 6ab3a5cb7fe08b9
Author: Michał Cłapiński <email address hidden>
Date: Tue Jan 9 16:41:04 2018 +0100
Updated lib/SConscript for Windows
Updated lib/SConscript with calls to correct Windows-only third-party-
Change-Id: I588fbe1b1f7301
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #108 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #109 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 7cfd83ad081b641
Author: Michał Cłapiński <email address hidden>
Date: Thu Jan 11 14:22:05 2018 +0100
Fixed recvmsg function on Windows
Added one missing argument - bool msg_wait
Change-Id: Ib8a9b663cb465d
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #110 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit a4a5400e383c388
Author: Michał Cłapiński <email address hidden>
Date: Wed Jan 10 11:43:00 2018 +0100
Fixed endlines in base
Fixed line endings (Windows-style to Unix-style)
Change-Id: Ib4ad921a488703
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #111 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #113 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit b3adbb7b81bbaf1
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 11 17:48:28 2018 +0100
Add VS_BUILDMODE env variable
Changes required for building some libraries on Windows
Change-Id: Ie194c570dca341
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #114 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #115 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit bff616286aa4f6a
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jan 12 10:54:37 2018 +0100
Use env['VS_BUILDMODE'] instead of env['OPT'] on Windows
Change-Id: I567242de0242ac
Depends-On: Ie194c570dca341
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #116 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 8c2480401bd730c
Author: Michał Cłapiński <email address hidden>
Date: Wed Jan 10 20:21:11 2018 +0100
Various fixes for Windows
1. INFDEFed code for huge pages and SIGHUP
2. Moved to C++ standard independent Ip?Address:
3. Fixed bug where std::map with C array as key tried to be created
(C arrays don't meet the requirements for map's key - to be copyable
and assignable)
4. Changed DELETE to DEL (DELETE is reserved on Windows)
Change-Id: I5555d11f13e311
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #117 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #118 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #119 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #120 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #121 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #122 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #123 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #125 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #127 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #128 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 3211b4dac22b45a
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 11 12:50:03 2018 +0100
Add SConscripts implementation for Windows
1. rabbitmq-c
2. SimpleAmqpClient
3. Required changes in other SConscripts
4. Use env['VS_BUILDMODE'] instead of env['OPT']
Depends-On: Ie194c570dca341
Depends-On: I588fbe1b1f7301
Change-Id: I39b2a497a802c8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #129 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #130 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #131 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #132 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #133 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #135 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #136 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #137 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #138 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #141 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #142 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 5aeb76bee032483
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 18 13:08:21 2018 +0100
Fix some tests compilation for Windows
Change-Id: I839b470bb1f0d1
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #143 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit 8e02dbd26e0aadd
Author: Mateusz Jablonski <email address hidden>
Date: Thu Jan 18 12:59:22 2018 +0100
Fix some tests compilation for Windows
Change-Id: If51d7a8d8aadb6
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #144 |
Reviewed: https:/
Committed: http://
Submitter: Zuul (<email address hidden>)
Branch: master
commit eec65458b9d5fae
Author: Michał Cłapiński <email address hidden>
Date: Wed Jan 17 17:56:17 2018 +0100
Moved files on Windows (preparation for shmem)
Change-Id: I56a58bb29ac5fa
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #145 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #146 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #147 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #148 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #151 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #152 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #153 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #157 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #158 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ba50c2aac412559
Author: Michał Cłapiński <email address hidden>
Date: Mon Jan 22 17:06:08 2018 +0100
Revert removing staging of address.h
Basically a revert of change nr 39006
It was removed because this file had two Install() calls that
caused a minor warning. Unfortunetly the other call is in
contrail-analytics - a repository that is not downloaded on
Windows. Also contrail-common is a more basic dependency so
it should have this call and contrail-analytics should not.
Change-Id: I6b3b279c955d16
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #159 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 93344d642260ba0
Author: Michał Cłapiński <email address hidden>
Date: Mon Jan 22 17:13:09 2018 +0100
Remove duplicate staging of address.h
The address.h from controller/src/net has two Install() calls that cause
a minor scons warning to be issued.
It's been already removed from contrail-common but it's actually needed
there, so it should be restored there (see depends-on) and removed from here.
Change-Id: I1bb568a13a39d9
Depends-On: I6b3b279c955d16
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #160 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #161 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #162 |
Review in progress for https:/
Submitter: Ananth Suryanarayana (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #164 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 6069d3df783584f
Author: Michał Cłapiński <email address hidden>
Date: Mon Jan 15 13:41:18 2018 +0100
Fix agent build on Windows
1. Simplified some SConscripts for Windows
2. GCC only __PRETTY_FUNCTION__ to standard __FUNCTION__
3. POSIX only random() to standard rand()
4. u_int8_t to standard uint8_t
5. Deprecated bzero to standard memset
6. POSIX only sysconf to platform independent boost solution
7. and more less significant fixes (mostly for C++11)
Change-Id: I5b6a9849aca56c
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #165 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #166 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #167 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit bf89391663bcb8c
Author: Michał Cłapiński <email address hidden>
Date: Wed Jan 17 17:40:32 2018 +0100
Level of indirection for shared memory on Windows
Platform independent:
1. Created a function for calculating bridge otable size
Windows only:
1. Written code for multiple shmem instances
2. Written code for multiple shmem devices
Change-Id: Ibd11c5afe1b41d
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #168 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #169 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit a6f6de5993f6c29
Author: Michał Cłapiński <email address hidden>
Date: Wed Jan 17 17:47:13 2018 +0100
Implemented shared memory mmap in utils on Windows
1. Moved current vr_table_map to unix_util.c
2. Implemented Windows only mmap in windows_util.c
3. Fixed bug where adding vif with util caused mcast_vrf
to be set to 0 instead of -1
4. Moved vr_netlink_port from vr_util to nl_util because it caused
linking issues
Change-Id: I218b2a8bcf6a23
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #170 |
Review in progress for https:/
Submitter: Michal Krasnoborski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #172 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #173 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #174 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #175 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #176 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #177 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #181 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d1ab9296e171907
Author: Michał Krasnoborski <email address hidden>
Date: Wed Jan 24 16:51:42 2018 +0100
Fix vtest on Windows
1. Add all_tests_run.ps1 script
2. Propagate error from vt_tree_traverse
3. Detect when response_process callback has not been called
4. Set netlink family when using VR_NETLINK_
5. Memzero source buffer in callbacks to avoid deallocating strings
Before this change dealocated memory was read.
6. Change nr 5 would cause mem leaks so a struct with special free
function was added
7. Add vt_packet stub for Windows (packet tests are ignored for now)
8. Renamed vrouter_
is not a simple getter and it has a lot of side effects.
#define will be removed in a follow-up change.
Change-Id: Icfe51fbd31415f
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #182 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ce0e6080d68c82b
Author: Michal Clapinski <email address hidden>
Date: Fri Feb 2 13:53:32 2018 +0100
Rename vrouter_
Since this function is not a simple getter and has side effects it
had to be renamed.
Change-Id: I04a6d353281534
Depends-On: Icfe51fbd31415f
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #183 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 6559b3856fda844
Author: Michal Clapinski <email address hidden>
Date: Fri Feb 2 13:57:05 2018 +0100
Remove old define for vrouter_
After renaming this function in contrail-controller we can safely delete
this define.
Change-Id: I61454fa828c649
Depends-On: I04a6d353281534
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #184 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2394003becda894
Author: Michal Clapinski <email address hidden>
Date: Tue Feb 6 18:42:12 2018 +0100
Don't compile KSyncSockTcp under Windows
1. Moved KSyncSockTcp class implementation to new file and enabled its
compilation only under Linux.
2. Enabled linking to tcmalloc only under Linux
Change-Id: I15447a320c632b
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #185 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #186 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #187 |
Review in progress for https:/
Submitter: Michal Kostrzewa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #188 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #189 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit f424b92aad7cb6d
Author: Michal Clapinski <email address hidden>
Date: Fri Feb 9 14:54:32 2018 +0100
Fixes for contrail-windows
Corrected the Linux-only types to standard C types
Change-Id: I5bbf594724d2ff
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #190 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8d46016aab153c8
Author: Michal Kostrzewa <email address hidden>
Date: Mon Feb 12 10:27:18 2018 +0100
Refactor formatting of powershell scripts.
% is an alias for ForEach-Object. It is recommended to use the verbose form.
Change-Id: I4dae2cf59c137b
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #191 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 003f2a9ec3e8893
Author: Mateusz Jablonski <email address hidden>
Date: Mon Feb 12 11:03:15 2018 +0100
Fix SConscript for Windows
Change-Id: I9ac1e7a1ea2230
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #192 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #194 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #195 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #196 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #197 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #201 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #204 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #206 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 11a284cc2c1d9f8
Author: Michal Clapinski <email address hidden>
Date: Mon Feb 19 19:04:15 2018 +0100
Add cmocka SConscript for Windows
Change-Id: Icef126ecda3cb9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #207 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #208 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #209 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 6ce6e721063de7b
Author: Michal Clapinski <email address hidden>
Date: Mon Feb 19 18:52:31 2018 +0100
Framework to unit test kernel-space code in userspace
Change-Id: I3b01439773a9cd
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #210 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #211 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #212 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #213 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #214 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #215 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #216 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #218 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 9b93c6925258dac
Author: Michał Cłapiński <email address hidden>
Date: Wed Feb 28 15:38:42 2018 +0100
Install correct version of setuptools before running them
Change-Id: I64baeca845cbbb
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #219 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 078bc64898a74e0
Author: Pawel Marchewka <email address hidden>
Date: Wed Feb 21 10:50:42 2018 +0100
Pip version doesn't need to be checked on Windows
Moreover we don't use targets related to python virtualenv on Windows
Change-Id: I7d42f95e485352
Depends-On: I64baeca845cbbb
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #220 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit c01d225abe24dd7
Author: Michal Clapinski <email address hidden>
Date: Wed Feb 28 16:10:49 2018 +0100
Add #build/bin to path before running tests
This is needed for cmocka.dll
Change-Id: I854ea35cf1a6e4
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #221 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #225 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8e38a1660965ab8
Author: Michal Clapinski <email address hidden>
Date: Wed Feb 21 15:29:42 2018 +0100
Remove POSIX and GCC extensions
1. bzero is not a standard C function, used memset
2. bcmp is not a standard C function, used memcmp
3. bcopy is not a standard C function, used memmove
4. Designed initializers are not a standard C++ feature, removed it
5. Structures with zero length arrays cannot be used as fields in structs
in standard C, wrote get functions instead
6. Used standard types instead of POSIX only ones
7. Generated a patch to help with future porting
Change-Id: I91f97287a8d5f2
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #226 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #227 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #229 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #231 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #233 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #234 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #235 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #237 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit ba273177b538790
Author: Pawel Marchewka <email address hidden>
Date: Fri Mar 9 14:19:48 2018 +0100
Use void * instead of custom void_t
void_t may conflict with std::void_t, that is not defined as void *
Change-Id: I0873c923e891a7
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #238 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit c0d13a86c36f755
Author: Michał Cłapiński <email address hidden>
Date: Tue Mar 6 15:37:30 2018 +0100
Fixed line endings
Converted Windows-style line endings to Unix-style
Change-Id: I7ebcae812e864b
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #239 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #240 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #241 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #242 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #243 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #244 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 7d660044fc97a44
Author: Mateusz Jablonski <email address hidden>
Date: Mon Mar 12 11:18:45 2018 +0100
Prepare vr_table_map to be used by Agent
The vr_table_map function returns error message now, so it will be
possible to use it with LOG() macro in Agent.
Change-Id: Ia15939f015f347
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #245 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #246 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #247 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #248 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #249 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #250 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #251 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #252 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #253 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #254 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 6160a9048bf50d3
Author: Mateusz Jablonski <email address hidden>
Date: Wed Mar 14 14:51:58 2018 +0100
Fix 'agent:pkt' test compilation on Windows
1. Fix 'agent:pkt' test compilation on Windows
2. Reformat code
(lines are still shorter than 100 characters,
but looks more readable now).
Change-Id: I481445724be5cb
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #255 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #256 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #257 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #258 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #259 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 1b269e9a1334045
Author: Mateusz Jablonski <email address hidden>
Date: Tue Mar 13 17:40:16 2018 +0100
Fixes for minor problems found while fixing bridge table on Windows
1. Add 'const' where needed
2. Remove unused defines/variables
3. Add 'void' to some C functions without arguments
4. Remove code duplication by using 'get_platform' function
Change-Id: I6b1687f32c1703
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #260 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 681d1230b68bf4e
Author: Michal Clapinski <email address hidden>
Date: Wed Mar 7 12:31:04 2018 +0100
Thrift patch for Windows was missing some changes
1. Added missing ctime_r related changes to the thrift patch
2. Simplified fetch_packages.py
Change-Id: I7de2a0021bebf8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #261 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #262 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #265 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #266 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 017501a9f0558ae
Author: Mateusz Jablonski <email address hidden>
Date: Tue Mar 13 19:18:35 2018 +0100
Create OS dependent vr_table_unlink function
Partial-Bug: #1737177
Change-Id: Ia5f0c6bb075532
OpenContrail Admin (ci-admin-f) wrote : | #267 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 14d4de9aa3d97c4
Author: Mateusz Jablonski <email address hidden>
Date: Fri Mar 9 12:59:32 2018 +0100
Fix XMPP server address parsing
1. Fix parsing address without port provided.
2. Move parsing logic to separate file.
3. Add unit tests.
4. Use proper type for port variables.
5. Fix formatting (mostly tabs/spaces/
changed automatically by code editor.
6. Fix some includes
Depends-On: I0873c923e891a7
Change-Id: Ic9e4645f0fc65e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #268 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #269 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #270 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #271 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #272 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #274 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d28756d8b7af341
Author: Michal Clapinski <email address hidden>
Date: Fri Mar 16 15:17:15 2018 +0100
Fix utility bugs introduced by change nr 40478
1. Remove bug where AF_BRIDGE was appended to flags
3. Remove flag NH_FLAG_
in vtest scenarios
Change-Id: I08aabf29821776
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #275 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 1354ea6f0e19acb
Author: Mateusz Jablonski <email address hidden>
Date: Tue Mar 13 18:00:23 2018 +0100
Fix bridge table implementation on Windows
1. Use constants instead of hardcoded values
2. Remove code duplication by using 'vr_table_map' function
3. Fix formatting in some includes
Depends-On: I6b1687f32c1703
Change-Id: I108674d5dd8c6b
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #276 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d9b122757dad3a3
Author: Mateusz Jablonski <email address hidden>
Date: Tue Mar 13 19:22:09 2018 +0100
Use vr_table_unlink function
Change-Id: I33a7274ff3db04
Depends-On: Ia5f0c6bb075532
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #277 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d1a4e9b7f5dbc26
Author: Mateusz Jablonski <email address hidden>
Date: Tue Mar 13 19:09:44 2018 +0100
Remove unused extern constant declaration
Depends-On: I108674d5dd8c6b
Change-Id: I0078de40715927
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #278 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #279 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #280 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #284 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #285 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit de42434c5d53b99
Author: Mateusz Jablonski <email address hidden>
Date: Fri Mar 16 10:24:57 2018 +0100
Refactor 'nl_util' structure for Windows and *nix
This is first part of multiple 'nl_util' changes.
For now, the only change is removing unused fields.
The main reason of all this changes is removing all
Windows/Linux 'ifdefs/ifndefs' from
'nl_util/vr_util' headers and sources.
Change-Id: Ie6b4bbbaef5cfc
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #286 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #287 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #288 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #289 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #290 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #291 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 52069a397263231
Author: Mateusz Jablonski <email address hidden>
Date: Fri Mar 16 15:07:45 2018 +0100
Refactor 'nl_util' structure for Windows and *nix
This is one of multiple 'nl_util' changes.
This change includes only better formatting
and removing unused 'return' keywords.
The main reason of all this changes is removing all
Windows/Linux 'ifdefs/ifndefs' from
'nl_util/vr_util' headers and sources.
Change-Id: Id782ee6d848015
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #292 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 48c8d4d98c31d7a
Author: Mateusz Jablonski <email address hidden>
Date: Fri Mar 16 15:21:32 2018 +0100
Refactor 'nl_util' structure for Windows and *nix
This is one of multiple 'nl_util' changes.
This change includes moving 'vrouter_
function to OS dependent source files.
The main reason of all this changes is removing all
Windows/Linux 'ifdefs/ifndefs' from
'nl_util/vr_util' headers and sources.
Change-Id: I581a65b198cca6
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #293 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 424fa50892cc6c5
Author: Mateusz Jablonski <email address hidden>
Date: Fri Mar 16 15:48:59 2018 +0100
Refactor 'nl_util' structure for Windows and *nix
This is one of multiple 'nl_util' changes.
This change includes:
1. Making *nix specific fields from nl_client
defined only on *nix systems.
2. Adding code for closing Windows KSync pipe.
3. Moving 'nl_free' OS specific code to separate files.
4. Moving 'nl_parse_reply' OS specific code to separate files.
5. Moving 'reset_cl_sock' OS specific code to separate files.
The main reason of all this changes is removing all
Windows/Linux 'ifdefs/ifndefs' from
'nl_util/vr_util' headers and sources.
Change-Id: I6bbda8550af003
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #294 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 40ad15c8a622345
Author: Mateusz Jablonski <email address hidden>
Date: Wed Mar 21 15:11:42 2018 +0100
Refactor 'vr_get_nl_client'
Move all Windows code from 'vr_get_nl_client'
to 'nl_socket' and 'nl_connect'.
Change-Id: I323724a9ea370f
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #295 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0b32f6e2a262d93
Author: Mateusz Jablonski <email address hidden>
Date: Wed Mar 21 16:06:28 2018 +0100
Use 'nl_socket' and 'nl_connect' for KSync pipe on Windows
Change-Id: I05cbe2194fea2a
Depends-On: I323724a9ea370f
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #296 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2b9e7550198f082
Author: Michal Clapinski <email address hidden>
Date: Wed Mar 14 20:58:10 2018 +0100
Handle vhost as VmInterface in GetOsParams
Implemented virtual method ObtainOsSpecifi
for setting os params depending on the child class
Change-Id: I7cc79754c39857
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #297 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #298 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #299 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #301 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #303 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #305 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #307 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #308 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #309 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #310 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #311 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #312 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #313 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #314 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #315 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #316 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #317 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #318 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #319 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 663137715e288e8
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:24:35 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I4042246eabd9f8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #320 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 0460741b8eacbd3
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:19:57 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I9c71f1558154e9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #321 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 499694a627be0a2
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:07:03 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I765f17008d1470
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #322 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 22a4f3cecc25a43
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 13 17:06:03 2018 +0200
Change Boost and OpenSSL paths in Thrift project on Windows
Boost and OpenSSL libraries will be moved from third-party to system directory.
Change-Id: I1556be2892aae0
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #323 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #324 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #325 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #326 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #327 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit e7bf0fca7bafd83
Author: Mateusz Jablonski <email address hidden>
Date: Fri Apr 13 17:06:03 2018 +0200
Change Boost and OpenSSL paths in Thrift project on Windows
Boost and OpenSSL libraries will be moved from third-party to system directory.
Change-Id: I1556be2892aae0
Partial-Bug: #1737177
(cherry picked from commit 22a4f3cecc25a43
OpenContrail Admin (ci-admin-f) wrote : | #328 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit b6f73f06db29114
Author: Pawel Marchewka <email address hidden>
Date: Mon Apr 16 17:50:45 2018 +0200
windows: fix held udp flows
Change-Id: I76e7a967bb499a
Closes-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #329 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2cc691779b3c380
Author: Pawel Marchewka <email address hidden>
Date: Mon Apr 16 17:50:45 2018 +0200
windows: fix held udp flows
Change-Id: I76e7a967bb499a
Closes-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #330 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 68898872c17d9f7
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:17:51 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: Icf690ad6cac40a
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #331 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 7740346693d7315
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:07:03 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I765f17008d1470
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #332 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 3708ee56ff69ac8
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:17:51 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: Icf690ad6cac40a
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #333 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit a0f7b5acb0b3877
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:14:43 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I02fe6dcee44e90
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #334 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit c4152505c450932
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:24:35 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I4042246eabd9f8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #335 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8c874075d10dbda
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:19:57 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I9c71f1558154e9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #336 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 16006ee8ecfad44
Author: Pawel Marchewka <email address hidden>
Date: Fri Apr 13 12:16:48 2018 +0200
Use boost and openssl from system on windows
Use boost and openssl from system instead of third_party on windows
Change-Id: Id798344bd11b02
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #337 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 403a95ab1c1830d
Author: Pawel Marchewka <email address hidden>
Date: Fri Apr 13 12:16:48 2018 +0200
Use boost and openssl from system on windows
Use boost and openssl from system instead of third_party on windows
Change-Id: Id798344bd11b02
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #338 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #339 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #340 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 302dcf27334f908
Author: Michal Clapinski <email address hidden>
Date: Tue Apr 17 16:14:43 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I02fe6dcee44e90
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #341 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #342 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #343 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #344 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #345 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8d8bc1d5d2e8819
Author: Pawel Marchewka <email address hidden>
Date: Wed Apr 25 12:23:53 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
GenerateDS works only with python2, hence change in shebang.
Change-Id: Ia04035064a75d6
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #346 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit fc945f961a40f63
Author: Pawel Marchewka <email address hidden>
Date: Wed Apr 25 10:18:00 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I92e03a285b70e9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #347 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #348 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #349 |
Review in progress for https:/
Submitter: Pawel Marchewka (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #350 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit c03ff0bd030747a
Author: Pawel Marchewka <email address hidden>
Date: Wed Apr 25 10:18:00 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
Change-Id: I92e03a285b70e9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #351 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 04f6d5b7f8b3a7c
Author: Pawel Marchewka <email address hidden>
Date: Wed Apr 25 12:23:53 2018 +0200
Add python3 support
Since cpython 2 has bugs that won't be fixed on Windows, we have to move
to python 3
GenerateDS works only with python2, hence change in shebang.
Change-Id: Ia04035064a75d6
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #352 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #353 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #354 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit fa3b8cab84ef967
Author: Pawel Marchewka <email address hidden>
Date: Fri Apr 27 13:29:11 2018 +0200
Updated windows_
Added rabbitmq-c and SimpleAmqpClient
Fixed xml formatting
Change-Id: I1fe065d5409489
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #355 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #356 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #357 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #359 |
Review in progress for https:/
Submitter: Michal Kostrzewa (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #360 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d9a4ffd6ba49aa9
Author: Michal Clapinski <email address hidden>
Date: Fri May 25 20:31:08 2018 +0200
Add new virtual-
Also don't shout error on 404
Change-Id: I43b63b6c4abf07
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #361 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 94c6a44e9dd9976
Author: Pawel Marchewka <email address hidden>
Date: Fri Apr 27 13:29:11 2018 +0200
Updated windows_
Added rabbitmq-c and SimpleAmqpClient
Fixed xml formatting
Change-Id: I1fe065d5409489
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #362 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #363 |
Review in progress for https:/
Submitter: Michal Krasnoborski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #365 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #366 |
Review in progress for https:/
Submitter: Michal Krasnoborski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #367 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 87e910201054297
Author: Michał Krasnoborski <email address hidden>
Date: Mon Jun 4 16:10:10 2018 +0200
More robust error handling of go-plugin-
1. Catch and propagade errors of h.Serve
2. Reorder and split waiting on pipe
3. Move writing plugin spec to after we have waited for pipe.
Change-Id: Iade6c740b429dd
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #368 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #369 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #370 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 27796a1bd582766
Author: Michal Clapinski <email address hidden>
Date: Fri Jun 8 19:18:06 2018 +0200
Fix missing sconscripts warnings on Windows
We don't clone those repositories on Windows, so we shouldn't use their
sconscripts.
Change-Id: Ib6114613fe3d22
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #371 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2aaf8c42fd7ca83
Author: Mateusz Jablonski <email address hidden>
Date: Mon May 21 16:23:48 2018 +0200
Remove ASSERTMSG
Remove ASSERTMSG which reises during kernel debugging
making debugging process very frustrating.
Change-Id: If2cf0f4f8856f1
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #372 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2bb99a727839c57
Author: Michal Clapinski <email address hidden>
Date: Fri Jun 8 19:52:58 2018 +0200
Simplify/correct SConscripts
Fix SConscripts warnings on Windows
Change-Id: Iffcb5a79e28954
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #373 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #375 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #377 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #378 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #379 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #380 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #381 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #382 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit de1ebe16d125383
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jun 22 14:44:10 2018 +0200
Refactoring of Windows NBLs - Part 1
1. Add VR_PACKET_WRAPPER structure containing vr_packet and WIN_PACKET
2. Use NET_BUFFER_LIST indirectly through WIN_PACKET
3. Fake WIN_PACKET to fix tests compilation (unnecessary stub code
will be removed in next PRs)
Change-Id: I1ff3d749b6d947
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #383 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #384 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #385 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #386 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #387 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #388 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #389 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #390 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #391 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #394 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #396 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #397 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #399 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #400 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #401 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #402 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #403 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #404 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #405 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #406 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #407 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #408 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #409 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 390b9c39ea69e98
Author: Michal Clapinski <email address hidden>
Date: Fri Jun 29 19:05:02 2018 +0200
Don't send ARP probes
When we leave the SPA (source IP) field empty in an ARP request, the
request begins to be an ARP probe. When container tries to assign
itself an IP address and receives an ARP probe, it thinks that somebody
else has that address already assigned and the address is marked as a
duplicate. To stop this behavior we should send ARP requests from normal
IP address. The idea to use service address was taken from
ArpEntry:
Change-Id: Iceec678ed12566
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #410 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #411 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 0625ce33a43d6ad
Author: Dariusz Sosnowski <email address hidden>
Date: Tue Jun 26 14:08:40 2018 +0200
Introduce WinPacketClone with unit tests
- Divide win_packet.c into two files
- win_packet.c - for NDIS-independent logic
- win_packet_raw.c - only NDIS-dependent code
- Extend WIN_PACKET interface to allow cloning and establishing
parent/child relationships
- Implement WinPacketClone unit tests
Change-Id: I468772725236a1
Depends-On: I1ff3d749b6d947
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #412 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2d527e5716e639d
Author: Dariusz Sosnowski <email address hidden>
Date: Fri Jun 29 12:37:12 2018 +0200
Introduce WinPacketFreeRe
This change is a part of Windows NBLs refactoring.
- Implement WinPacketFreeRe
- Implement WinPacketFreeRe
Change-Id: I349da7e9f002de
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #413 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit cd6f3a2b0d3659c
Author: Mateusz Jablonski <email address hidden>
Date: Tue Jul 3 13:10:42 2018 +0200
Use WinPacketFreeRe
Change-Id: I0ac93523b1f829
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #414 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit b3d243e3108bdb5
Author: Mateusz Jablonski <email address hidden>
Date: Wed Jun 27 15:31:53 2018 +0200
win_pclone/
Change-Id: I120f06f4e030f3
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #415 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #419 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #420 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #421 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #422 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: R5.0
commit 02a40d9333f9d1c
Author: Michal Clapinski <email address hidden>
Date: Fri Jun 29 19:05:02 2018 +0200
Don't send ARP probes
When we leave the SPA (source IP) field empty in an ARP request, the
request begins to be an ARP probe. When container tries to assign
itself an IP address and receives an ARP probe, it thinks that somebody
else has that address already assigned and the address is marked as a
duplicate. To stop this behavior we should send ARP requests from normal
IP address. The idea to use service address was taken from
ArpEntry:
Change-Id: Iceec678ed12566
Partial-Bug: #1737177
(cherry picked from commit 390b9c39ea69e98
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #423 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : [Review update] R5.0 | #424 |
Review in progress for https:/
Submitter: Jarek Lukow (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #425 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 50c8296222079dd
Author: Mateusz Jablonski <email address hidden>
Date: Mon Jul 9 12:30:53 2018 +0200
Refactor SConscript for test/windows directory
Change-Id: I2d2c4d955abf40
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #426 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 8582f487c674767
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jul 13 16:23:02 2018 +0200
Refactor WinPacket
- Add WinPacketRaw for all OS-dependent operations
- Refactor WinPacket
Change-Id: I74d2b92e565f2e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #427 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit b6e1aeca6b4b22b
Author: Mateusz Jablonski <email address hidden>
Date: Fri Jul 13 16:35:00 2018 +0200
Refactor NBL splitting
- Add WinMultiPacket
- Add WinSubPacket
- Add WinPacketList
- Add WinPacketSplitM
Change-Id: I011b572bfad035
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #428 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 32b751950ae5692
Author: Mateusz Jablonski <email address hidden>
Date: Wed Jul 18 12:18:17 2018 +0200
Post-refactor changes
- Change Fake_WinPacketL
- Remove dead code
- Add comment with logic explanation
- Remove unused packets definitions
- Update .gitignore
Change-Id: Ib4d4d3f063ca96
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #429 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #430 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #431 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #433 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #437 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #438 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 96a04515f3cf3e7
Author: Mateusz Jablonski <email address hidden>
Date: Thu Aug 9 16:42:32 2018 +0200
Fix floating IP for Windows
IPv4 packet coming from virtual interface may have
TcpChecksum or UdpChecksum flag set. In this case
TCP/UDP checksum is partially calculated so we need
to set VP_FLAG_
Change-Id: I14b060cd5f0997
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #439 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit c3aee6f186235cc
Author: Michał Cłapiński <email address hidden>
Date: Tue Aug 14 15:09:58 2018 +0200
Unify checking OS in SConscripts
We were using two methods for checking OS, sys.platform and
platform.system(). Now we only used platform.system() as its
output is more user friendly, eg. 'Windows' instead of 'win32'
Change-Id: I59df572358da40
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #440 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #441 |
Review in progress for https:/
Submitter: Dariusz Sosnowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #442 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 571d0d21d116f76
Author: Michal Clapinski <email address hidden>
Date: Fri Aug 3 18:29:26 2018 +0200
Don't build unnecessary modules on Windows
Change-Id: I4982cfd3ceb5bc
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #443 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 2dba0f92ad71802
Author: Michal Clapinski <email address hidden>
Date: Fri Aug 3 18:23:15 2018 +0200
Don't build unnecessary api-lib on Windows
Change-Id: Ia441d608457b17
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #444 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit cb547e1ab74ee36
Author: Dariusz Sosnowski <email address hidden>
Date: Mon Aug 13 11:27:29 2018 +0200
windows: Unit tests for atomic operation wrappers
Initial work:
https:/
Change-Id: I6ad28206d224ff
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #445 |
Review in progress for https:/
Submitter: Michal Krasnoborski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #446 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #448 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #449 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #450 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #451 |
Review in progress for https:/
Submitter: Jacek Izykowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #454 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #455 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 5b4eeaee41022ff
Author: Mateusz Jablonski <email address hidden>
Date: Thu Sep 6 15:59:11 2018 +0200
Pass proper handle to NdisAllocateIoW
A handle to wrong object has been passed to
NdisAllocateIoW
which resulted in executing asynchronous callback synchronously.
In some rare cases it leads to recursively acquire
lock and finally causes a deadlock.
Change-Id: Ide96756bf1a65c
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #456 |
Review in progress for https:/
Submitter: Jacek Izykowski (<email address hidden>)
description: | updated |
OpenContrail Admin (ci-admin-f) wrote : | #460 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #461 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #462 |
Review in progress for https:/
Submitter: Jacek Izykowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #464 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #465 |
Review in progress for https:/
Submitter: Jacek Izykowski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #466 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 57826baf2806682
Author: Michal Clapinski <email address hidden>
Date: Mon Sep 17 16:59:34 2018 +0200
Poll for interface MAC address
Sometimes it takes a while for an interface to get the physical
address so it's better to poll.
Change-Id: I6171d63a217de3
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #467 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #468 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 43e802164d80162
Author: Michał Krasnoborski <email address hidden>
Date: Thu Aug 30 17:19:59 2018 +0200
Handle LSO in windows vrouter
Change-Id: I886f6582037ce8
Partial-Bug: #1737177
Co-authored-by: Michal Krasnoborski <email address hidden>
Co-authored-by: Mateusz Jablonski <email address hidden>
Co-authored-by: Jacek Izykowski <email address hidden>
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #469 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #471 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #472 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #474 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 07a81d070d2e501
Author: Mateusz Jablonski <email address hidden>
Date: Tue Sep 18 11:49:42 2018 +0200
Rewrite function with improper license
Change-Id: I7a90bd6892995e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #475 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #476 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #478 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 535ae429fa4e8c6
Author: Arkadiusz Kudan <email address hidden>
Date: Mon Sep 17 18:09:04 2018 +0200
Fix vDNS responding on Windows
Sometimes agent didn't respond to VMs' DNS queries
in vDNS mode. It was related to memory with the response being freed,
before Pkt0Interface:
With this fix, we will ensure that the memory won't be
freed until the handler of async_write function is called.
Change-Id: I07734c22be4134
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #479 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d362b168999011a
Author: Arkadiusz Kudan <email address hidden>
Date: Wed Sep 19 14:59:01 2018 +0200
Implement default DNS mode on Windows
In default DNS mode VMs get list of DNS resolvers
from OS on which vRouter agent is running.
Right now, this list is read from /etc/resolv.conf file,
so it doesn't work on Windows.
To get the list of known DNS servers on Windows, we have to use
"iphlpapi" module, which is IP helper API.
This commit will logically divide implementation of building
the list of DNS resolvers in matter of OS the agent is running on.
Change-Id: Iad1b1dcd2237a4
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #480 |
Review in progress for https:/
Submitter: Przemys?aw Marcinkiewicz (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #481 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #482 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #483 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #484 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #486 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 77534f537d847e4
Author: Michał Cłapiński <email address hidden>
Date: Fri Sep 28 18:50:12 2018 +0200
Fix endlines in the whole repository
This repository had Windows-style line endings all over.
This change converts them to Linux-style line endings.
Change-Id: I642e46abd54db3
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #487 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit e47e71b4f2b82d1
Author: Michał Cłapiński <email address hidden>
Date: Fri Sep 28 18:54:07 2018 +0200
Return -1 in posix stubs
We were returning 0 in stub functions without setting anything
in the passed structures. Therefore the results were dependent
on random memory. Now we return -1, so nobody should use those
structures.
Change-Id: I7c767d17a98fd8
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #488 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #489 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #490 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 5d1e5ab014dbce0
Author: Michal Clapinski <email address hidden>
Date: Mon Oct 1 15:41:14 2018 +0200
Port mirror and vrmemstats utils for Windows
Just enabled building them.
Change-Id: I12dabf6932be3e
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : | #491 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit 16300d899a2d013
Author: Michal Clapinski <email address hidden>
Date: Fri Sep 28 19:09:39 2018 +0200
Fix default paths on Windows
There are hardcoded paths to /var and /etc in agent.
This PR fixes them for Windows.
Change-Id: If9b152c83fd977
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #492 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #498 |
Review in progress for https:/
Submitter: Mateusz Jablonski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #499 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : | #500 |
Review in progress for https:/
Submitter: Arkadiusz Kudan (<email address hidden>)
OpenContrail Admin (ci-admin-f) wrote : A change has been merged | #501 |
Reviewed: https:/
Committed: http://
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master
commit d749fd24c130cd8
Author: Arkadiusz Kudan <email address hidden>
Date: Mon Oct 22 12:47:54 2018 +0200
Delete Agent's Windows service creation
This commit deletes vRouter agent's service creation
from MSI. Right now, we stick to NSSM for Windows service creation
of Contrail components, so MSI creates a "dead" service.
Change-Id: I6309dac661c4f9
Partial-Bug: #1737177
OpenContrail Admin (ci-admin-f) wrote : [Review update] master | #502 |
Review in progress for https:/
Submitter: Micha? C?api?ski (<email address hidden>)
Changed in opencontrail: | |
status: | New → Fix Committed |
OpenContrail Admin (ci-admin-f) wrote : | #503 |
Review in progress for https:/
Submitter: Michal Clapinski (<email address hidden>)
Review in progress for https:/ /review. opencontrail. org/38302
Submitter: Michal Clapinski (<email address hidden>)