Skip to content

Update sonic-platform-modules-arista for 201811#3232

Merged
yxieca merged 1 commit intosonic-net:201811from
zzhiyuan:201811aristaupdate
Jul 31, 2019
Merged

Update sonic-platform-modules-arista for 201811#3232
yxieca merged 1 commit intosonic-net:201811from
zzhiyuan:201811aristaupdate

Conversation

@zzhiyuan
Copy link
Copy Markdown
Contributor

Enable platform API functionalities in 201811, including reboot cause.

- What I did

- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

@Staphylo
Copy link
Copy Markdown
Collaborator

I just realized that the PR I opened 2 weeks ago didn't merge #3170 .
This one overlaps as it point to more recent commit from our driver repository.
However without the fancontrol fixes from mine both the 7170-64C and the 7260CX3-64 will start reporting errors. Driver update and fancontrol fixes have to merge at the same time.

@lguohan
Copy link
Copy Markdown
Collaborator

lguohan commented Jul 30, 2019

#3170 merged, can you resolve the conflict?

@zzhiyuan zzhiyuan force-pushed the 201811aristaupdate branch from f72b37c to d38a4c0 Compare July 30, 2019 16:58
@zzhiyuan
Copy link
Copy Markdown
Contributor Author

#3170 merged, can you resolve the conflict?

Conflict resolved

@yxieca yxieca merged commit 1ddfa13 into sonic-net:201811 Jul 31, 2019
mssonicbld added a commit that referenced this pull request Jul 23, 2024
…lly (#19654)

#### Why I did it
src/sonic-swss
```
* 2367bca - (HEAD -> master, origin/master, origin/HEAD) Add a check if OID exists before setting Host Tx Ready Signal Enable (#3232) (15 hours ago) [noaOrMlnx]
```
#### How I did it
#### How to verify it
#### Description for the changelog
mssonicbld added a commit that referenced this pull request Jul 26, 2024
…lly (#19707)

#### Why I did it
src/sonic-swss
```
* 51a5b51f - (HEAD -> 202405, origin/202405) Add a check if OID exists before setting Host Tx Ready Signal Enable (#3232) (7 hours ago) [noaOrMlnx]
```
#### 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants