Updating sonic motd with official logo by rodnymolina · Pull Request #1487 · sonic-net/sonic-buildimage

rodnymolina · 2018-03-13T05:28:56Z

This one should satisfy most of the initial concerns: no more than 80-cols wide, and no more than 80-lines long. Let me know if any suggestion.

jleveque · 2018-03-14T23:35:37Z

files/image_config/environment/motd

+                         \___ \| | | |  \| | | |
+                          ___) | |_| | |\  | | |___
+                         |____/ \___/|_| \_|_|\____|
+


I like it, but I feel it's too big. Also, I think all text underneath should be centered on the logo. Also, do we need to keep the "You are on" line?

Here's more of what I'm thinking:

,,,,, ,,,,,,,,,,,,. /***,,,,,,,, /(///***,,,,. (#(((///***, , **##((///. ## ****//* *#####, ****///// %%%#### (((######, .***/////(((#%%%%%####(((((((######, .**////(((((##%%#(/((((((((######, .&%///(((((###(//((((((((#/#####, %&&&&%%%#((((###(//(((((, /#####, ***#&%%%# (###(///( /#####, ****/// *(, *(#####, ****/////( ((((###### ***/////((((/ ,(((((((##### .////(((((### //(((((((((#* /(((((###(//((((((( *((###(//(((* .##(//. ____ ___ _ _ _ ____ / ___| / _ \| \ | (_)/ ___| \___ \| | | | \| | | | ___) | |_| | |\ | | |___ |____/ \___/|_| \_|_|\____| -- Software for Open Networking in the Cloud -- Unauthorized access and/or use are prohibited. All access and/or use are subject to monitoring. Help: http://azure.github.io/SONiC/

I think this logo is trying to match the gradual change of the color
Actually I have a black/white version of the logo
I think that might be more suitable in the motd

@jleveque i like your idea to center all text, looks good. Not sure about the smaller logo though, i tried that one on a real device and didn't look that good IMO.

@stcheng please share the one you have so that we can try it.

also feel it is a little big

I feel should be with 80-col width and 40~50 rows.

@lguohan, my current proposal is already meeting your size criteria: 66 columns and 38 rows. Please keep in mind that in reality t looks considerably smaller than what it appears in this github ascii file. Have no issues making it smaller if you all agree, but keep in mind that the image won't look that neat anymore. Please let me know.

jleveque

This PR has been open for almost a full year. I think we're long overdue for freshening up the MOTD! My vote at this point is simply to remove the first three lines (the "You are on" line and two blank lines following it), bringing the entire MOTD (logo and text) to 40 rows total and center the text at the bottom. Here's my suggestion, line for line:

                            ,,,,
                        .,,,,,,,,,,.
                     *,,,,,,,,,,,,,,,,,.
                 *****,,,,,,,,,,,,,,.
              ///*****,,,,,,,,,,,.
          *((////*****,,,,,,,,
      ,#(((((////*****,,,,
    (###(((((////*****.                                 #.
   **###(((((/////*                                     ##.
  *****((((((//,                                     /####(
  *******//(/                                    ,#########
  *******/////.            #%%#%%,            ((###########
  ,******/////////     *%%%%%%%#####(     ,(((((###########
   ******////////((((%%%%%%%%%%%#######(((((((((###########
    ******///////(((((((#%%%%%%%####((((((((((((###########
       ,*////////((((((((####%##(//(((((((((((((###########
         /%//////((((((((#####////(((((((((((((############
      #&&&&&%%///((((((((#####////((((((((((((,  ##########
   .#&&&&&&&%%%%%#(((((((#####////((((((((/      ##########
  .***&&&&&&%%%%%%*  /(((#####////(((((.         ##########
  *******%&&%%(         .#####/////*             ##########
  ********//#              .##//*               /##########
  *******///////                            .(((##########(
  .******////////((.                     /((((((#########
    *****////////(((((/              .((((((((((########(
      ****///////(((((((((        ,(((((((((((((######,
         ,///////((((((((####/.////((((((((((((((#(
             *//(((((((((#####////(((((((((((((.
                .((((((((#####////(((((((((*
                    /((((#####////((((((
                       *(#####////((/
                           ###///,

                  ____   ___  _   _ _  ____
                 / ___| / _ \| \ | (_)/ ___|
                 \___ \| | | |  \| | | |
                  ___) | |_| | |\  | | |___
                 |____/ \___/|_| \_|_|\____|


       -- Software for Open Networking in the Cloud --
       
        Unauthorized access and/or use are prohibited.
       All access and/or use are subject to monitoring.
       
            Help:  http://azure.github.io/SONiC/

swss: * d7643f2 2020-11-01 | [tlm_teamd]: Make the destionation for std::transform() to use std::back_inserter() for allocating new space for the copied objects (sonic-net#1490) (HEAD, origin/master, origin/HEAD) [pavel-shirshov] * 7fa7cd6 2020-10-31 | [vstest]: stablize fgnhg test (sonic-net#1491) [lguohan] * 9b0696e 2020-10-29 | Create vnet tunnel map only if it doesn't exist (sonic-net#1482) [Prince Sunny] * 0481e99 2020-10-29 | [acl] Update CRM to include LAG bindings for ACL tables (sonic-net#1487) [Danny Allen] sairedis * 5df11f5 2020-11-01 | [syncd] Lower bulk missing api message level from error to info (sonic-net#687) (HEAD, origin/master, origin/HEAD) [Kamil Cudnik] * dc73a1d 2020-10-30 | [saiplayer] Fix log messages (sonic-net#686) [Kamil Cudnik] Signed-off-by: Guohan Lu <lguohan@gmail.com>

swss: * d7643f2 2020-11-01 | [tlm_teamd]: Make the destionation for std::transform() to use std::back_inserter() for allocating new space for the copied objects (#1490) (HEAD, origin/master, origin/HEAD) [pavel-shirshov] * 7fa7cd6 2020-10-31 | [vstest]: stablize fgnhg test (#1491) [lguohan] * 9b0696e 2020-10-29 | Create vnet tunnel map only if it doesn't exist (#1482) [Prince Sunny] * 0481e99 2020-10-29 | [acl] Update CRM to include LAG bindings for ACL tables (#1487) [Danny Allen] sairedis * 5df11f5 2020-11-01 | [syncd] Lower bulk missing api message level from error to info (#687) (HEAD, origin/master, origin/HEAD) [Kamil Cudnik] * dc73a1d 2020-10-30 | [saiplayer] Fix log messages (#686) [Kamil Cudnik] Signed-off-by: Guohan Lu <lguohan@gmail.com>

swss: * d7643f2 2020-11-01 | [tlm_teamd]: Make the destionation for std::transform() to use std::back_inserter() for allocating new space for the copied objects (sonic-net#1490) (HEAD, origin/master, origin/HEAD) [pavel-shirshov] * 7fa7cd6 2020-10-31 | [vstest]: stablize fgnhg test (sonic-net#1491) [lguohan] * 9b0696e 2020-10-29 | Create vnet tunnel map only if it doesn't exist (sonic-net#1482) [Prince Sunny] * 0481e99 2020-10-29 | [acl] Update CRM to include LAG bindings for ACL tables (sonic-net#1487) [Danny Allen] sairedis * 5df11f5 2020-11-01 | [syncd] Lower bulk missing api message level from error to info (sonic-net#687) (HEAD, origin/master, origin/HEAD) [Kamil Cudnik] * dc73a1d 2020-10-30 | [saiplayer] Fix log messages (sonic-net#686) [Kamil Cudnik] Signed-off-by: Guohan Lu <lguohan@gmail.com>

* d58ba82 2021-04-05 | Fix fast-reboot fdb filtering for IPv6 addresses (sonic-net#1548) (github/201811) [Vaibhav Hemant Dixit] * cc7ad9e 2021-03-24 | Fast reboot: check if the files are present before accessing (sonic-net#1526) [Vaibhav Hemant Dixit] * 410323a 2021-03-24 | [201811] Add missing variable DEVPATH to fast-reboot script (sonic-net#1525) [Vaibhav Hemant Dixit] * eb5023a 2021-03-21 | [201811] soft-reboot (sonic-net#1487) [Sujin Kang] * 3f10540 2020-12-08 | [fast-reboot] Fix fast-reboot when NDP entries are present (sonic-net#1295) (HEAD -> 201811) [shlomibitton] Signed-off-by: Ying Xie <ying.xie@microsoft.com>

* d58ba82 2021-04-05 | Fix fast-reboot fdb filtering for IPv6 addresses (#1548) (github/201811) [Vaibhav Hemant Dixit] * cc7ad9e 2021-03-24 | Fast reboot: check if the files are present before accessing (#1526) [Vaibhav Hemant Dixit] * 410323a 2021-03-24 | [201811] Add missing variable DEVPATH to fast-reboot script (#1525) [Vaibhav Hemant Dixit] * eb5023a 2021-03-21 | [201811] soft-reboot (#1487) [Sujin Kang] * 3f10540 2020-12-08 | [fast-reboot] Fix fast-reboot when NDP entries are present (#1295) (HEAD -> 201811) [shlomibitton] Signed-off-by: Ying Xie <ying.xie@microsoft.com>

Signed-off-by: Danny Allen <daall@microsoft.com>

…tically (#21287) #### Why I did it src/sonic-sairedis ``` * 1684aecf - (HEAD -> master, origin/master, origin/HEAD) [syncd] Enable bulk api for next hop objects (#1487) (2 days ago) [Nikola Dancejic] ``` #### How I did it #### How to verify it #### Description for the changelog

…tically (sonic-net#21287) #### Why I did it src/sonic-sairedis ``` * 1684aecf - (HEAD -> master, origin/master, origin/HEAD) [syncd] Enable bulk api for next hop objects (sonic-net#1487) (2 days ago) [Nikola Dancejic] ``` #### How I did it #### How to verify it #### Description for the changelog

…AD automatically (sonic-net#1487) #### Why I did it src/sonic-utilities ``` * 19b52d13 - (HEAD -> 202503, origin/202503) Merge pull request sonic-net#221 from mssonicbld/sonicbld/202503-merge (23 hours ago) [mssonicbld] * 4fe1bc8c - Merge branch '202412' of https://github.com/Azure/sonic-utilities.msft into 202503 (23 hours ago) [Sonic Automation] * aa38dd68 - Added json support intfutil (sonic-net#220) (33 hours ago) [mssonicbld] ``` #### How I did it #### How to verify it #### Description for the changelog

Updating sonic motd with official logo

daa351e

This one should satisfy most of the initial concerns: no more than 80-cols wide, and no more than 80-lines long. Let me know if any suggestion.

jleveque suggested changes Mar 14, 2018

View reviewed changes

jleveque reviewed Feb 7, 2019

View reviewed changes

lguohan mentioned this pull request Nov 1, 2020

[submodule]: swss/sairedis module update #5765

Merged

3 tasks

lguohan force-pushed the master branch from 3690c1a to 512eb6b Compare December 25, 2020 18:34

yxieca mentioned this pull request Apr 8, 2021

[201811][utilities] advance submodule head #7266

Merged

theasianpianist pushed a commit to theasianpianist/sonic-buildimage that referenced this pull request Feb 5, 2022

[acl] Update CRM to include LAG bindings for ACL tables (sonic-net#1487)

0481e99

Signed-off-by: Danny Allen <daall@microsoft.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Updating sonic motd with official logo#1487

Updating sonic motd with official logo#1487
rodnymolina wants to merge 1 commit intosonic-net:masterfrom
rodnymolina:motd_update

rodnymolina commented Mar 13, 2018

Uh oh!

jleveque Mar 14, 2018 •

edited

Loading

Uh oh!

stcheng Mar 15, 2018

Uh oh!

rodnymolina Mar 15, 2018

Uh oh!

lguohan Mar 23, 2018

Uh oh!

lguohan Mar 23, 2018

Uh oh!

rodnymolina Mar 27, 2018

Uh oh!

jleveque left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Conversation

rodnymolina commented Mar 13, 2018

Uh oh!

jleveque Mar 14, 2018 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

stcheng Mar 15, 2018

Choose a reason for hiding this comment

Uh oh!

rodnymolina Mar 15, 2018

Choose a reason for hiding this comment

Uh oh!

lguohan Mar 23, 2018

Choose a reason for hiding this comment

Uh oh!

lguohan Mar 23, 2018

Choose a reason for hiding this comment

Uh oh!

rodnymolina Mar 27, 2018

Choose a reason for hiding this comment

Uh oh!

jleveque left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

jleveque Mar 14, 2018 •

edited

Loading