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

Repeated attempt to save member despite S913 #2406

Closed
Tracked by #3167
ibmrcruicks opened this issue Aug 1, 2023 · 9 comments · Fixed by #3359
Closed
Tracked by #3167

Repeated attempt to save member despite S913 #2406

ibmrcruicks opened this issue Aug 1, 2023 · 9 comments · Fixed by #3359
Assignees
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.
Milestone

Comments

@ibmrcruicks
Copy link

Description of problem

**Zowe Extension user edits a read-only dataset, and hits save -- system log shows continual attempts to save the update, despite incurring S913 abends

Results in pages and pages of log entries like:

 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 605      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00328,0D33,VPWRKD, 606          
 ZXP.PUBLIC.JCL(REPRO)                                                      
 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 607      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00330,0D33,VPWRKD, 608          
 ZXP.PUBLIC.JCL(REPRO)                                                      
 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 609      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00332,0D33,VPWRKD, 610          
 ZXP.PUBLIC.JCL(REPRO)                                                      
 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 611      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00334,0D33,VPWRKD, 612          
 ZXP.PUBLIC.JCL(REPRO)                                                      
 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 613      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00336,0D33,VPWRKD, 614          
 ZXP.PUBLIC.JCL(REPRO)                                                      
 ICH408I USER(Zxxxxx  ) GROUP(STUDENT2) NAME(####################) 615      
   ZXP.PUBLIC.JCL CL(DATASET ) VOL(VPWRKD)                                  
   INSUFFICIENT ACCESS AUTHORITY                                            
   FROM ZXP.PUBLIC.** (G)                                                   
   ACCESS INTENT(UPDATE )  ACCESS ALLOWED(READ   )                          
 IEC150I 913-38,IFG0194E,Zxxxxx,IZUFPROC,SYS00338,0D33,VPWRKD, 616          
 ZXP.PUBLIC.JCL(REPRO)                                                      
******************************************************************************

with each entry appears 1-3 seconds apart.

Reproduce Error

In VSCode/ZOWE Extension, Open SYS1.MACLIB (be sure you don't have WRITE access!) open ABEND, make arbitrary update, and save (cmd-/ctrl-S, or file->save)

Expected behavior

** Single attempt to write to dataset - note S913 and stop any further attempt to save **

Screenshots

Desktop (please complete the following information):

  • OS: Windows and MacOS
  • Zowe Explorer Version: 2.9.*
  • (Optional) Zowe CLI Version:
  • (Optional) Are you using Secure Credential Store?

Additional context

@ibmrcruicks ibmrcruicks added the bug Something isn't working label Aug 1, 2023
@github-actions
Copy link

github-actions bot commented Aug 1, 2023

Thank you for creating a bug report.
We will investigate the bug and evaluate its impact on the product.
If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

@traeok
Copy link
Member

traeok commented Aug 16, 2023

Hi @ibmrcruicks ,

Do you have the setting "Files: Auto Save" enabled in VSCode? The auto-save feature might be issuing repeated save requests, even if the previous requests are failing.

Regardless, it sounds like we should try to address this in Zowe Explorer, but this feedback will help clarify the source of the problem.

Thanks!

@ibmrcruicks
Copy link
Author

not available for the historic events, but I will try and get the user VSCode Autosave setting the next time this shows up.

@ibmrcruicks
Copy link
Author

finally have a screenshot of a user's VSCode Autosave settings when the S913 loop occured.
vsc-zowe-autosave

@ibmrcruicks
Copy link
Author

ibmrcruicks commented Dec 15, 2023

to try and address the endless S913 abends, and associated SYSLOG flooding, I revoked the TSO account; this then produces repeating ICH408I errors for the revoked account (slightly less logging overhead, but not much)


 ICH408I USER(Zxxxxx  ) GROUP(STUDENTU) NAME(####################) 395 
   LOGON/JOB INITIATION - REVOKED USER ACCESS ATTEMPT                  
 ICH408I USER(Zxxxxx  ) GROUP(STUDENTU) NAME(####################) 396 
   LOGON/JOB INITIATION - REVOKED USER ACCESS ATTEMPT                  

@traeok
Copy link
Member

traeok commented Dec 20, 2023

Hi @ibmrcruicks, thanks for the update.

I believe this is a side effect of using the "Auto Save" functionality with a short delay. Unfortunately, Zowe Explorer does not support the "Auto Save" feature. If this feature is enabled, it can cause undefined behavior (this is most noticeable with the afterDelay option selected).

As a temporary workaround, I recommend disabling the "Auto Save" functionality or increasing the delay for the afterDelay option when working within Zowe Explorer. I will discuss the severity/priority of this issue with the squad to determine if we can fix this behavior in v2. In the long-term, this could be resolved in v3 with the implementation of the FileSystemProvider.

@ibmrcruicks
Copy link
Author

ibmrcruicks commented Dec 20, 2023

thanks @traeok -- disappointing; when you say the "latest version" does that imply some previous version did/does support AutoSave, or just that it is not supported by Zowe Explorer yet?

Turning off AutoSave affects all the VSCode workspaces -- what would you recommend as a better afterDelay value?

@adam-wolfe adam-wolfe added priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex. labels Dec 21, 2023
@adam-wolfe
Copy link
Contributor

Related to: #2627

@traeok
Copy link
Member

traeok commented Dec 21, 2023

thanks @traeok -- disappointing; when you say the "latest version" does that imply some previous version did/does support AutoSave, or just that it is not supported by Zowe Explorer yet?

Turning off AutoSave affects all the VSCode workspaces -- what would you recommend as a better afterDelay value?

@ibmrcruicks I realized my previous reply was a bit unclear - from my understanding, Zowe Explorer has never supported the auto-save functionality.

When it comes to the afterDelay value, it's difficult to recommend a "one-size-fits-all" value since the current behavior can vary depending on network latency, etc. I'd set the delay to at least 2.5 seconds (2500ms) to account for that latency.
As an alternative, the onFocusChange option would be less error-prone, reducing the chance of repeated save attempts.

We plan to investigate this issue for v2 and I will keep this issue updated as the resolution moves forward.

@t1m0thyj t1m0thyj moved this from New Issues to Medium Priority in Zowe Explorer for VS Code Dec 26, 2023
This was referenced Apr 8, 2024
@JillieBeanSim JillieBeanSim added this to the v2.18.0 milestone Jul 11, 2024
@JillieBeanSim JillieBeanSim modified the milestones: v2.18.0, v2.18.1 Sep 11, 2024
@traeok traeok moved this from Medium Priority to In Progress in Zowe Explorer for VS Code Sep 19, 2024
@traeok traeok linked a pull request Dec 12, 2024 that will close this issue
15 tasks
@traeok traeok moved this from In Progress to Review/QA in Zowe Explorer for VS Code Dec 12, 2024
@JTonda JTonda closed this as completed Dec 17, 2024
@github-project-automation github-project-automation bot moved this from Review/QA to Closed in Zowe Explorer for VS Code Dec 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.
Projects
Status: Closed
Development

Successfully merging a pull request may close this issue.

5 participants