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

Submodule update #1351

Merged
merged 2 commits into from
Jan 30, 2018
Merged

Submodule update #1351

merged 2 commits into from
Jan 30, 2018

Conversation

prsunny
Copy link
Contributor

@prsunny prsunny commented Jan 30, 2018

- What I did

Submodule update for

  1. sonic-py-swsssdk
  2. sonic-snmpagent
  3. sonic-utilities

@lguohan
Copy link
Collaborator

lguohan commented Jan 30, 2018

conflict, can you check?

@yxieca
Copy link
Contributor

yxieca commented Jan 30, 2018

The conflict is kind of expected with Qi's PR #1350. And it should be easy to resolve.

Looks like we cannot solve it online.

Prince,

I think what you need to do is:

from your working branch, do a 'git fetch origin', then "git rebase -i origin/master", then take your change on snmp repo. then force push to your fork.

We can do it together tomorrow if you'd like to.

Regards,
Ying

@prsunny
Copy link
Contributor Author

prsunny commented Jan 30, 2018

@yxieca, I did a merge from upstream and seems to have resolved the conflict. Can you check.

@stcheng
Copy link
Contributor

stcheng commented Jan 30, 2018

@prsunny what's this pull request's update?

mssonicbld added a commit that referenced this pull request Feb 8, 2024
…tically (#18068)

#### Why I did it
src/sonic-sairedis
```
* a504933 - (HEAD -> master, origin/master, origin/HEAD) Change dash API pipeline name (#1351) (11 hours ago) [Kamil Cudnik]
```
#### How I did it
#### How to verify it
#### Description for the changelog
sonic-otn pushed a commit to Weitang-Zheng/sonic-buildimage that referenced this pull request Mar 11, 2024
…tically (sonic-net#18068)

#### Why I did it
src/sonic-sairedis
```
* a504933 - (HEAD -> master, origin/master, origin/HEAD) Change dash API pipeline name (sonic-net#1351) (11 hours ago) [Kamil Cudnik]
```
#### How I did it
#### How to verify it
#### Description for the changelog
saksarav-nokia pushed a commit to saksarav-nokia/sonic-buildimage that referenced this pull request Mar 12, 2024
…tically (sonic-net#18068)

#### Why I did it
src/sonic-sairedis
```
* a504933 - (HEAD -> master, origin/master, origin/HEAD) Change dash API pipeline name (sonic-net#1351) (11 hours ago) [Kamil Cudnik]
```
#### 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