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

Integration Candidate 20190906 #26

Merged
merged 10 commits into from
Sep 16, 2019
Merged

Integration Candidate 20190906 #26

merged 10 commits into from
Sep 16, 2019

Conversation

skliper
Copy link
Contributor

@skliper skliper commented Sep 6, 2019

Describe the contribution
Merge of CCB 20190904 approved commits.
Fixes #24, fixes #4, fixes #12, fixes #6, fixes #11, fixes #5, fixes #7, fixes #14, fixes #1

Testing performed
Steps taken to test the contribution:

  1. Checked out https://github.com/nasa/cFS master and followed setup instructions
  2. Checked out ic-20190906
  3. Followed standard build instructions (no cmdUtil build warnings observed)
  4. Started core-cpu1 in build/exe/cpu1 directory
  5. Ran make in tools/cFS-GroundSystem/Subsystem/cmdUtil to make cmdUtil executable
  6. Started ground too in tools/cFS-GroundSystem
python GroundSystem.py
  1. Sent "Enable Tlm" command with input 127.0.0.1
  2. Observed response from core-cpu1
EVS Port1 42/1/TO_LAB_APP 3: TO telemetry output enabled for IP 127.0.0.1
  1. Observed Packet Count increments in telemetry system GUI

Expected behavior changes

  • No impact to behavior from any of these changes

System(s) tested on:

  • Server
  • OS: Ubuntu/Linux 4.4.0-146-generic
  • Versions: bundle 6.6 master with this branch

Additional context
None

Contributor Info
Jacob Hageman/NASA

Community contributors
CLA on file

@skliper skliper requested a review from a team September 6, 2019 17:58
@skliper
Copy link
Contributor Author

skliper commented Sep 10, 2019

Just tested with 6.7 bundle successfully (same test execution process as for 6.6)

Copy link
Contributor

@jphickey jphickey left a comment

Choose a reason for hiding this comment

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

Confirmed that this change set also builds and runs OK for me. No issues.

@skliper skliper added this to the 2.1.0 milestone Sep 16, 2019
@skliper skliper merged commit ae911ab into master Sep 16, 2019
@skliper skliper deleted the ic-20190906 branch September 16, 2019 20:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants