Port nodemgr to Windows

Bug #1783539 reported by Michał Cłapiński
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juniper Openstack
Status tracked in Trunk
Trunk
Fix Committed
Undecided
Michał Cłapiński
OpenContrail
Fix Committed
Undecided
Michał Cłapiński

Bug Description

nodemgr isn't platform independent and requires significant changes to work on Windows

Tags: windows
Changed in opencontrail:
assignee: nobody → Michał Cłapiński (michal.clapinski)
Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/44937
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44938
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44941
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44943
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/44941
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/45011
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/44941
Committed: http://github.com/Juniper/contrail-controller/commit/1af9785f55b948a56e5a70060832688823d1d120
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 1af9785f55b948a56e5a70060832688823d1d120
Author: Michal Clapinski <email address hidden>
Date: Wed Jul 25 19:45:39 2018 +0200

Adjust nodemgr for Windows

Where convenient I've implemented some level of indirection,
where inconvenient I've used platform checking to skip some
Linux only code. Real code checking running processes and so
on will be submitted in next changes.

Change-Id: I5abb1b37621c295eac56ff096c5a348f999588c9
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/44943
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/44937
Committed: http://github.com/Juniper/contrail-common/commit/3c131839bc9019f339ddb9765c6dfc9ed4ffe020
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 3c131839bc9019f339ddb9765c6dfc9ed4ffe020
Author: Michal Clapinski <email address hidden>
Date: Wed Jul 25 13:12:27 2018 +0200

Don't record port on Windows

This won't work on Windows and is not needed.

Change-Id: I9b49d7d02925d9dafbefa39eb220712fe757bc7a
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/44938
Committed: http://github.com/Juniper/contrail-build/commit/9c54232a198a3c5a1102d1ad14f9dee392ee1345
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 9c54232a198a3c5a1102d1ad14f9dee392ee1345
Author: Michal Clapinski <email address hidden>
Date: Wed Jul 25 13:21:24 2018 +0200

Port to Windows/Python3

Minor fixes related to Windows and Python 3

Change-Id: Idec35bf0e66869dae5d817b3ca389fea3b927e52
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/45011
Committed: http://github.com/Juniper/contrail-controller/commit/ef10d00372b22e8c433383d4cb003caa7c9753db
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit ef10d00372b22e8c433383d4cb003caa7c9753db
Author: Michal Clapinski <email address hidden>
Date: Fri Jul 27 16:50:38 2018 +0200

Missing pyyaml requirement

pyyaml wasn't being install with nodemgr as it should

Change-Id: I2ee943d35f8e14d60d519a58b4e441ca5dba67a0
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/44943
Committed: http://github.com/Juniper/contrail-controller/commit/7fed0208770413399fde5e76ad2046b6bc440b16
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 7fed0208770413399fde5e76ad2046b6bc440b16
Author: Michal Clapinski <email address hidden>
Date: Wed Jul 25 21:41:29 2018 +0200

Implement checking if agent is up on Windows

Very simple implementation using psutil

Change-Id: I2b7c65d6d677f0f57e79277ac2298f0b73729b94
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/45211
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/45211
Submitter: Mateusz Jablonski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/45211
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/45266
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/45211
Committed: http://github.com/Juniper/contrail-controller/commit/fae95d28551e7c4062d984488b07190734e1d061
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit fae95d28551e7c4062d984488b07190734e1d061
Author: Michal Clapinski <email address hidden>
Date: Thu Aug 2 17:20:01 2018 +0200

Implement proper status reporting in nodemgr by events on Windows

Reporting by events was missing so I had to add some new code to
windows_process_manager.py. Also I wanted to reuse some code from
docker_process_manager.py so I moved it to common_process_manager.py

Change-Id: If191e8cfd6ca18fd8b4899c7bcaa259d6950d628
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/45332
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Review in progress for https://review.opencontrail.org/45333
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/45266
Committed: http://github.com/Juniper/contrail-third-party/commit/28100980fb363eab2bf40c706bb3d47ad0798f3d
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 28100980fb363eab2bf40c706bb3d47ad0798f3d
Author: Michal Clapinski <email address hidden>
Date: Fri Aug 3 19:11:12 2018 +0200

Download fysom on Windows for nodemgr

Change-Id: Iba61e916b6258029565a06bb8bd33d83a8b11fde
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/45332
Committed: http://github.com/Juniper/contrail-controller/commit/5f09d5ed1e01b95fce920021d3159124a98a0779
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 5f09d5ed1e01b95fce920021d3159124a98a0779
Author: Michal Clapinski <email address hidden>
Date: Mon Aug 6 20:31:46 2018 +0200

Create scons alias for nodemgr with dependencies

From now on `scons nodemgr-full` will build nodemgr and all dependencies.

Change-Id: I06beb95d1d9d12e31198c1ad00254f53c5e6c55b
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote :

Reviewed: https://review.opencontrail.org/45333
Committed: http://github.com/Juniper/contrail-common/commit/0b6a40ab2c8f6ba56917e128c8969859a05d1abf
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit 0b6a40ab2c8f6ba56917e128c8969859a05d1abf
Author: Michal Clapinski <email address hidden>
Date: Mon Aug 6 20:38:06 2018 +0200

Create scons alias for nodemgr with dependencies

From now on `scons nodemgr-full` will build nodemgr and all dependencies.

Change-Id: Ie8d41984c7160d6d296b402151cfffd1d5174aa5
Partial-Bug: #1783539

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : [Review update] master

Review in progress for https://review.opencontrail.org/47437
Submitter: Micha? C?api?ski (<email address hidden>)

Revision history for this message
OpenContrail Admin (ci-admin-f) wrote : A change has been merged

Reviewed: https://review.opencontrail.org/47437
Committed: http://github.com/Juniper/contrail-controller/commit/c27957cda3ba582c6dd023e31cbca1319b94996d
Submitter: Zuul v3 CI (<email address hidden>)
Branch: master

commit c27957cda3ba582c6dd023e31cbca1319b94996d
Author: Michal Clapinski <email address hidden>
Date: Fri Nov 2 17:22:38 2018 +0100

Fix vrouter agent service name

Change-Id: I05fd94c6775feb242513c673c23ce2f36e28646b
Partial-Bug: #1783539

Changed in opencontrail:
status: New → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.