Skip to content

Conversation

tylertitsworth
Copy link
Contributor

@tylertitsworth tylertitsworth commented Jun 25, 2024

Description

Change the idp installation command from pip install to conda install to avoid an issue.

Related Issue

#177

Changes Made

  • Remove shell declaration from cannibalizing .bashrc activation
  • Switch from pip install to conda install
  • The code follows the project's coding standards.
  • No Intel Internal IP is present within the changes.
  • The documentation has been updated to reflect any changes in functionality.

Validation

Manual, since infra is still down.

  • I have tested any changes in container groups locally with test_runner.py with all existing tests passing, and I have added new tests where applicable.

@tylertitsworth tylertitsworth self-assigned this Jun 25, 2024
@tylertitsworth tylertitsworth linked an issue Jun 25, 2024 that may be closed by this pull request
7 tasks
Copy link

github-actions bot commented Jun 25, 2024

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

PackageVersionScoreDetails

Scanned Manifest Files

@tylertitsworth
Copy link
Contributor Author

2024-06-25 09:11:54,798 - root - INFO - Setup Completed - Running Tests
2024-06-25 09:11:54,799 - root - INFO - Running Test: import-core
2024-06-25 09:11:55,487 - root - INFO - Running Test: perf-full
2024-06-25 09:11:56,299 - root - INFO - Running Test: perf-core
2024-06-25 09:11:57,165 - root - INFO - Running Test: perf-stock
2024-06-25 09:11:59,672 - root - INFO - 4 Images Removed
2024-06-25 09:11:59,673 - root - INFO - 
|   # | Test        | Status   |
|-----+-------------+----------|
|   1 | import-core | PASS     |
|   2 | perf-full   | PASS     |
|   3 | perf-core   | PASS     |
|   4 | perf-stock  | PASS     |

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@tylertitsworth tylertitsworth merged commit 2a45e07 into main Jun 26, 2024
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
Co-authored-by: Jitendra Patil <jitendra.patil@intel.com>
Signed-off-by: ma-pineda <miguel.pineda.juarez@intel.com>
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
Co-authored-by: Jitendra Patil <jitendra.patil@intel.com>
Signed-off-by: ma-pineda <miguel.pineda.juarez@intel.com>
ma-pineda pushed a commit that referenced this pull request Jun 28, 2024
Co-authored-by: Jitendra Patil <jitendra.patil@intel.com>
Signed-off-by: ma-pineda <miguel.pineda.juarez@intel.com>
@tylertitsworth tylertitsworth deleted the tylertitsworth/python-full-patch branch July 1, 2024 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Intel Python full container fails to build
2 participants