Skip to content

[makefile] build Jessie slave unless NOJESSIE is specified#3546

Merged
qiluo-msft merged 2 commits intosonic-net:masterfrom
yxieca:ghm
Oct 2, 2019
Merged

[makefile] build Jessie slave unless NOJESSIE is specified#3546
qiluo-msft merged 2 commits intosonic-net:masterfrom
yxieca:ghm

Conversation

@yxieca
Copy link
Contributor

@yxieca yxieca commented Oct 1, 2019

Provide a method in master branch to build Jessie slave docker. This option is currently on by default unless NOJESSIE=1 is specified. The default for NOJESSIE should probably becomes 1 at some point when we fully understand what needs Jessie slave docker where from the master branch.

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca added the Bug 🐛 label Oct 1, 2019
@yxieca
Copy link
Contributor Author

yxieca commented Oct 2, 2019

retest vsimage please

@yxieca yxieca marked this pull request as ready for review October 2, 2019 02:35
@qiluo-msft qiluo-msft merged commit 1515e39 into sonic-net:master Oct 2, 2019
@echo "+++ Making $@ +++"
ifeq ($(NOJESSIE), 0)
make -f Makefile.work $@
endif
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if-else-endif ?

@yxieca yxieca deleted the ghm branch October 2, 2019 16:41
mssonicbld added a commit that referenced this pull request Oct 12, 2024
…atically (#20458)

#### Why I did it
src/sonic-utilities
```
* 910252c - (HEAD -> master, origin/master, origin/HEAD) [Mellanox] Rename SKU to Mellanox-SN5600-C256X1 (#3546) (3 days ago) [DavidZagury]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Apr 19, 2025
…lly (#22378)

#### Why I did it
src/sonic-swss
```
* e0b2ac6 - (HEAD -> master, origin/master, origin/HEAD) swss: Align STP structures and remove GCC diagnostic pragmas (#3440) (3 hours ago) [Omar Tamer]
* 120ed90 - [fgnhgorch] Fix incorrect state DB updates due to unordered field-val… (#3566) (3 hours ago) [InspurSDN]
* 1552e34 - Move timestamps out of counter table to avoid update too frequently (#3585) (6 hours ago) [Stephen Sun]
* 73e534c - Initialize the last fec ber computed values if not found (#3604) (27 hours ago) [Prince George]
* 504daa1 - [az] Install vpp artifacts (#3609) (27 hours ago) [Kamil Cudnik]
* bc750a5 - [orchagent] Add value validation for RouteOrch, NhgOrch and Srv6Orch (#3596) (4 days ago) [Yuqing Zhao(Alibaba Inc)]
* d1fb9f7 - [DASH] Dash tunnel/PL NSG implementation (#3543) (4 days ago) [Lawrence Lee]
* ffc865e - Fix a pipeline issue due to dependency in docker-sonic-vs (#3603) (4 days ago) [Dashuai Zhang]
* f7ca83d - Fix the missed port status notifications issue (#3595) (7 days ago) [Vivek]
* 5592fbe - [portsorch] pass oper status at hostif creation (#3563) (7 days ago) [Stepan Blyshchak]
* 7bcd21c - [orchagent] Fix issue: typo in high BER (#3600) (8 days ago) [Junchao-Mellanox]
* 3e1f8d8 - [portmgrd] regression: prevent runtime exception (crash) in configuring portchannel at boot (#3432) (8 days ago) [Brad House]
* 492a85a - [DASH] Make DASH vnet orch bulk logic consistent (#3554) (2 weeks ago) [Lawrence Lee]
* 822ab91 - Handle outbound_direction_lookup for DASH_APPLIANCE_TABLE (#3571) (3 weeks ago) [prabhataravind]
* fe9b89e - [DVS] Remove framework package during DVS build (#3577) (3 weeks ago) [Lawrence Lee]
* 596d88c - Added Change for given Route ECMP to fallback on Default Route ECMP (#3389) (3 weeks ago) [abdosi]
* d35c1c7 - Initialize number of counters to 0 before querying the capabilities (#3546) (5 weeks ago) [Stephen Sun]
* 0e0685a - Fix the switch level Voq counter to use switch_id Oid instead of counter name for PACKET_INTEGRITY counter (#3521) (5 weeks ago) [saksarav-nokia]
* ee38b39 - [BFD] Change capability check to get. (#3559) (5 weeks ago) [dypet]
* 45fdedb - Prevent lossless profile creation for 0m cable (#3548) (5 weeks ago) [Jianyue Wu]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants