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

Fix #882, Returned processor ID to default to unbreak toolchain #883

Merged

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Sep 11, 2020

Describe the contribution
Fix #882

CI port selection depends on processor ID, #776 changed the default which broke toolchain.

Testing performed
built, ran, confirmed CI is back to listening on default port.

Expected behavior changes
Defaults work again in the toolchain

System(s) tested on

  • Hardware: cFS Dev Server
  • OS: Ubuntu 18.04
  • Versions: Bundle main + this change.

Additional context
None

Third party code
None

Contributor Info - All information REQUIRED for consideration of pull request
Jacob Hageman - NASA/GSFC

@skliper skliper force-pushed the fix882-processorid-default branch from 792ef77 to 56618e8 Compare September 11, 2020 14:00
@skliper skliper added bug CCB:FastTrack CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Sep 11, 2020
@skliper skliper added this to the 7.0.0 milestone Sep 11, 2020
@yammajamma yammajamma added IC-20200916 and removed CCB:Ready Ready for discussion at the Configuration Control Board (CCB) labels Sep 16, 2020
@yammajamma
Copy link
Contributor

yammajamma commented Sep 17, 2020

CCB 2020-09-16 APPROVED

@yammajamma yammajamma changed the base branch from main to integration-candidate September 17, 2020 13:43
@yammajamma yammajamma merged commit c3ab053 into nasa:integration-candidate Sep 17, 2020
@skliper skliper deleted the fix882-processorid-default branch February 1, 2021 22:08
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.

Default linux port got changed again...
2 participants