Skip to content

Commit 6203b2b

Browse files
committed
Check for proxy validity, not creation
1 parent 7a4205a commit 6203b2b

File tree

7 files changed

+11
-10
lines changed

7 files changed

+11
-10
lines changed

osgtest/tests/test_400_proxy.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ def test_03_grid_proxy_init(self):
3333
core.state['proxy.created'] = True
3434

3535
def test_04_grid_proxy_info(self):
36+
core.state['proxy.valid'] = False
3637
core.skip_ok_unless_installed('globus-proxy-utils')
37-
self.skip_bad_unless(core.state['proxy.created'], 'Proxy creation failed')
3838
command = ('grid-proxy-info', '-debug')
3939
core.check_system(command, 'Normal grid-proxy-info', user=True)
40+
core.state['proxy.valid'] = True

osgtest/tests/test_410_jobs.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ def test_02_condor_ce_run_condor(self):
5858
self.skip_bad_unless(service.is_running('condor-ce'), 'ce not running')
5959
self.skip_bad_unless(service.is_running('condor'), 'condor not running')
6060
self.skip_bad_unless(core.state['jobs.env-set'], 'job environment not set')
61-
self.skip_bad_unless(core.state['proxy.created'], 'requires a proxy cert')
61+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
6262

6363
command = ('condor_ce_run', '-r', '%s:9619' % core.get_hostname(), '/bin/env')
6464
self.run_job_in_tmp_dir(command, 'condor_ce_run a Condor job')

osgtest/tests/test_465_xrootd_tpc.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def setUp(self):
1717

1818
def test_01_create_macaroons(self):
1919
core.skip_ok_unless_installed('x509-scitokens-issuer-client', by_dependency=True)
20-
self.skip_bad_unless(core.state['proxy.created'], 'Proxy creation failed')
20+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
2121
core.config['xrootd.tpc.macaroon-1'] = None
2222
core.config['xrootd.tpc.macaroon-2'] = None
2323

osgtest/tests/test_550_condorce.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ def test_02_queue(self):
8383

8484
def test_03_ping(self):
8585
self.general_requirements()
86-
self.skip_bad_unless(core.state['proxy.created'], 'requires a proxy cert')
86+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
8787

8888
command = ('condor_ce_ping', 'WRITE', '-verbose')
8989
stdout, _, _ = core.check_system(command, 'ping using GSI and gridmap', user=True)
@@ -92,7 +92,7 @@ def test_03_ping(self):
9292
def test_04_trace(self):
9393
self.general_requirements()
9494
self.skip_bad_unless(core.state['condor-ce.schedd-ready'], 'CE schedd not ready to accept jobs')
95-
self.skip_bad_unless(core.state['proxy.created'], 'requires a proxy cert')
95+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
9696

9797
cwd = os.getcwd()
9898
os.chdir('/tmp')
@@ -108,7 +108,7 @@ def test_05_pbs_trace(self):
108108
core.skip_ok_unless_installed('torque-mom', 'torque-server', 'torque-scheduler', 'torque-client', 'munge',
109109
by_dependency=True)
110110
self.skip_ok_unless(service.is_running('pbs_server'), 'pbs service not running')
111-
self.skip_bad_unless(core.state['proxy.created'], 'requires a proxy cert')
111+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
112112
self.run_blahp_trace('pbs')
113113

114114
def test_06_slurm_trace(self):
@@ -117,7 +117,7 @@ def test_06_slurm_trace(self):
117117
self.skip_bad_unless(service.is_running('munge'), 'slurm requires munge')
118118
self.skip_bad_unless(core.state['condor-ce.schedd-ready'], 'CE schedd not ready to accept jobs')
119119
self.skip_ok_unless(service.is_running(core.config['slurm.service-name']), 'slurm service not running')
120-
self.skip_bad_unless(core.state['proxy.created'], 'requires a proxy cert')
120+
self.skip_bad_unless(core.state['proxy.valid'], 'requires a proxy cert')
121121
self.run_blahp_trace('slurm')
122122

123123
def test_07_ceview(self):

osgtest/tests/test_580_gfal2util.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ class TestGFAL2Util(osgunittest.OSGTestCase):
1212

1313

1414
def setUp(self):
15-
self.skip_ok_unless(core.state['proxy.created'] or core.state['voms.got-proxy'])
15+
self.skip_ok_unless(core.state['proxy.valid'] or core.state['voms.got-proxy'])
1616
core.skip_ok_unless_installed('gfal2-util', 'gfal2-plugin-file')
1717

1818
def get_gftp_url_base(self):

osgtest/tests/test_590_gsiopenssh.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ class TestGSIOpenSSH(osgunittest.OSGTestCase):
2121
def setUp(self):
2222
core.skip_ok_unless_installed('gsi-openssh-server', 'gsi-openssh-clients')
2323
self.skip_ok_unless(core.state['gsisshd.can-run'], "Couldn't run gsisshd (see above)")
24-
self.skip_ok_unless(core.state['proxy.created'] or core.state['voms.got-proxy'], 'no proxy')
24+
self.skip_ok_unless(core.state['proxy.valid'] or core.state['voms.got-proxy'], 'no proxy')
2525
self.skip_bad_unless(core.state['gsisshd.started-service'], 'gsisshd service not running')
2626
self.port = core.config['gsisshd.port']
2727

osgtest/tests/test_960_proxy.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,6 @@ class TestGridProxyDestroy(osgunittest.OSGTestCase):
55

66
def test_01_check_proxy(self):
77
core.skip_ok_unless_installed('globus-proxy-utils')
8-
self.skip_ok_unless(core.state['proxy.created'], 'missing proxy')
8+
self.skip_ok_unless(core.state['proxy.created'], "didn't create proxy")
99
command = ('grid-proxy-destroy', '-debug')
1010
core.check_system(command, 'Run grid-proxy-destroy', user=True)

0 commit comments

Comments
 (0)