Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ci]: use build template #1633

Merged
merged 8 commits into from
Feb 10, 2021
Merged

[ci]: use build template #1633

merged 8 commits into from
Feb 10, 2021

Conversation

lguohan
Copy link
Contributor

@lguohan lguohan commented Feb 10, 2021

Signed-off-by: Guohan Lu lguohan@gmail.com

What I did
simplify ci with template

Why I did it

How I verified it

Details if related

Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Signed-off-by: Guohan Lu <lguohan@gmail.com>
@lguohan lguohan merged commit 8739791 into sonic-net:master Feb 10, 2021
@lguohan lguohan deleted the ci branch February 10, 2021 16:37
DavidZagury pushed a commit to DavidZagury/sonic-swss that referenced this pull request Mar 4, 2021
Signed-off-by: Guohan Lu <lguohan@gmail.com>
xumia pushed a commit to xumia/sonic-swss that referenced this pull request Jun 2, 2021
Signed-off-by: Guohan Lu <lguohan@gmail.com>
xumia added a commit that referenced this pull request Jun 18, 2021
* [ci]: use build template (#1633)

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* [ci]: run vstest

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* [ci]: archive swss pytests (#1690)

in other pipelines, they will download swss and pytests. this is make sure the swss.deb and pytests are matched.

Signed-off-by: Guohan Lu <lguohan@gmail.com>

* [ci]: Purge swss before install (#1654)

Observe persistent failure of newly added vs test on testing code change in intfmgrd after migrating to azure pipeline. #1521 passed in legacy Jenkins pipeline.

What we found is that the code change in the swss compiled with PR change is not properly installed in vs docker for a direct installation using dpkg -i. This is confirmed by verifying pipeline artifacts that md5sum value of /user/bin/intfmgrd changes if we install swss deb aritfact into the docker vs artifact, while that of /usr/bin/orchangent stays unchanged.

md5sum before swss install in vs docker

# md5sum /usr/bin/orchagent
28307a7805ea6f3bc5057c0257bf46e6  /usr/bin/orchagent
 
# md5sum /usr/bin/intfmgrd  
fa2b06e20be683286adb47c55635a86d  /usr/bin/intfmgrd
md5sum after swss install

# dpkg -i swss_1.0.0_amd64.deb 
(Reading database ... 19180 files and directories currently installed.)
Preparing to unpack swss_1.0.0_amd64.deb ...
Unpacking swss (1.0.0) over (1.0.0) ...
Setting up swss (1.0.0) ...
# md5sum /usr/bin/orchagent
28307a7805ea6f3bc5057c0257bf46e6  /usr/bin/orchagent

# md5sum /usr/bin/intfmgrd 
e959340709e7aedd7489e69dfd19768f  /usr/bin/intfmgrd

Signed-off-by: Wenda Ni <wonda.ni@gmail.com>

* Fix branch reference error

* Fix branch reference error

Co-authored-by: lguohan <lguohan@gmail.com>
Co-authored-by: Wenda Ni <wonda.ni@gmail.com>
xumia pushed a commit to xumia/sonic-swss that referenced this pull request Jun 25, 2021
Signed-off-by: Guohan Lu <lguohan@gmail.com>
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-swss that referenced this pull request Oct 5, 2021
Signed-off-by: Guohan Lu <lguohan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant