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

update docker IOCs #841

Merged
merged 6 commits into from
Apr 7, 2023
Merged

update docker IOCs #841

merged 6 commits into from
Apr 7, 2023

Conversation

prjemian
Copy link
Contributor

@prjemian prjemian commented Apr 7, 2023

@prjemian prjemian added this to the 1.6.16 milestone Apr 7, 2023
@prjemian prjemian self-assigned this Apr 7, 2023
@prjemian
Copy link
Contributor Author

prjemian commented Apr 7, 2023

failed to connect with gp:gp:float1, 30s timeout expired

@prjemian
Copy link
Contributor Author

prjemian commented Apr 7, 2023

Arrrgh! There is a mismatch in Python versions in the CI. Again! In the Py3.10 test matrix, a Python 3.11 library is called:

==================================== ERRORS ====================================
___________ ERROR collecting apstools/plans/tests/test_alignment.py ____________
apstools/plans/tests/test_alignment.py:38: in <module>
    obj.wait_for_connection()
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/ophyd/signal.py:1786: in wait_for_connection
    self._ensure_connected(self._read_pv, self._write_pv, timeout=timeout)
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/ophyd/signal.py:1230: in _ensure_connected
    raise TimeoutError(
E   TimeoutError: gp:gp:float1 could not connect within 30.0-second timeout.
=============================== warnings summary ===============================
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/setuptools_scm/git.py:135
  /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/setuptools_scm/git.py:135: UserWarning: "/home/runner/work/apstools/apstools" is shallow and may cause errors
    warnings.warn(f'"{wd.path}" is shallow and may cause errors')

../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:121
  /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:121: DeprecationWarning: pkg_resources is deprecated as an API
    warnings.warn("pkg_resources is deprecated as an API", DeprecationWarning)

../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870
  /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('mpl_toolkits')`.
  Implementing implicit namespace packages (as specified in PEP 4[20](https://github.com/BCDA-APS/apstools/actions/runs/4640798978/jobs/8213126305#step:15:21)) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:[28](https://github.com/BCDA-APS/apstools/actions/runs/4640798978/jobs/8213126305#step:15:29)70
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870
../../../micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870
  /home/runner/micromamba-root/envs/anaconda-test-env-py-3.10/lib/python3.11/site-packages/pkg_resources/__init__.py:2870: DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace('sphinxcontrib')`.
  Implementing implicit namespace packages (as specified in PEP 420) is preferred to `pkg_resources.declare_namespace`. See https://setuptools.pypa.io/en/latest/references/keywords.html#keyword-namespace-packages
    declare_namespace(pkg)

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
=========================== short test summary info ============================
ERROR apstools/plans/tests/test_alignment.py - TimeoutError: gp:gp:float1 could not connect within [30](https://github.com/BCDA-APS/apstools/actions/runs/4640798978/jobs/8213126305#step:15:31).0-second timeout.
!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!
======================== 7 warnings, 1 error in 36.[32](https://github.com/BCDA-APS/apstools/actions/runs/4640798978/jobs/8213126305#step:15:33)s =========================
Error: Process completed with exit code 1.

@prjemian
Copy link
Contributor Author

prjemian commented Apr 7, 2023

@prjemian
Copy link
Contributor Author

prjemian commented Apr 7, 2023

Also note this advice in CI:

Code style
Node.js 12 actions are deprecated. Please update the following actions to use Node.js 16: actions/checkout@v2. For more information see: https://github.blog/changelog/2022-09-22-github-actions-all-actions-will-begin-running-on-node16-instead-of-node12/.

Current version of actions/checkout is v3.50.

@prjemian
Copy link
Contributor Author

prjemian commented Apr 7, 2023

Pulled the image on separate, local workstation and confirmed the PV is not available:

root@iockgp:/home# gp.sh start                                                  
Starting kgp:                                                                   
/opt/synApps/support /opt/synApps/iocs/iocgp/iocBoot/iocgp/softioc              
/opt/synApps/iocs/iocgp/iocBoot/iocgp/softioc                                   
/opt/synApps/iocs/iocgp/iocBoot/iocgp /opt/synApps/iocs/iocgp/iocBoot/iocgp/soft
ioc                                                                             
/opt/synApps/iocs/iocgp/iocBoot/iocgp/softioc                                   
kgp: is running (pid=21) in a screen session (pid=17)                           
root@iockgp:/home# cd iocgp                                                     
root@iockgp:/home/iocgp# grep float1 dbl-all.txt                                
root@iockgp:/home/iocgp#

@prjemian prjemian merged commit 4be24d1 into main Apr 7, 2023
@prjemian prjemian deleted the 840-new-docker-image branch April 7, 2023 22:14
canismarko pushed a commit to spc-group/apstools that referenced this pull request Apr 17, 2023
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.

CI can use new docker image
1 participant