Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
[geclipse-dev] Quality Issues

Hi All,

 

Just to not forget about them here is Ariel’s list of quality issues and a few comments from me. Please add your comments if you already fixed something. I know it is a loooong list but in order to make it shorter you should REALLY have a look at it and comment or even fix things!!!

 

 

* P1: Grid project view: VO subtree resources get fetched every single time the

  a folder is closed/opened again. This looks a bit strange for any user,

  and gets annoying for users with slow network.

            - WP2: Get some kind of (light) caching of the information

 

Comments:

 

 

* P2: Glue & Grid project views: info displaying could be faster

            - WP1/2: Avoid fetching the whole LDAP tree initially, often it is enough

              to have the hostnames only to display the tree, the rest can come

              later (new API? fetchComputingNames(), fetchComputingDetails(CE),...)

 

Comments:

 

 

* I1: Simple tests: could profit from a "Hostname" test, ie, direct/reverse

  DNS lookup

            - WP3

 

Comments:

 

 

* I2: Project view: resources in VO tree could have "Properties" in the

  context menu which opens/focuses the properties view

            - WP1/2?

 

Comments:

- Mathias: Definitely a new feature and therefore not applicable during the quality phase. We may think about creating a feature request in bugzilla.

 

 

* B1: LFC mounts do not work (Exception at LFCServer.connect:131)

  At some point also got the not very clarifying ProblemDialog:

                        Child Fetcher @ LFC <host>

                        Reason: <host>

                        Details: Fetch Error

            - WP2

 

Comments:

 

 

* B2: If a mount fails (host unreachable) the mount folder still gets shown but with

  name: "null(Error: Failed to establish a network connection)"

            - WP1

 

Comments:

- Mathias: This should be fixed in HEAD. Nevertheless I am curious how this can happen since the mount action should not need to open a network connection. It just creates a new linked resource but does not do any online actions.

 

 

* B3: Local connection allows to rename a folder owned by another user,

  creating a new one with the new name and trying to delete the old one -> error

            - WP1: Rename should not be "create new + delete old"

 

Comments:

- Mathias: The rename is the standard resource rename from the platform. Unfortunately this is implemented as “create new + delete old”. So at the moment there is no easy solution for that.

 

 

* B4: Child fetching doesn't "recover" when the underlying connection throws an

  error: "100% parsing data" progress bar stays, "Folder is empty" shown

  together with a folder for instance.

            - WP1

 

Comments:

- Mathias: Fixed in HEAD.

 

* B5: Exception while renaming file in GSIFtp -> whole grid project &

  connection view empty!!! And this still happens after restart! (project is

  still in workspace though)

            - WP2, WP1?

 

Comments:

- Mathias: Not able to reproduce this one. I assume this was fixed with the second fix that caused the workspace to be deleted if a connection element was deleted.

 

 

* B6: Refresh doesn't work for connections: reject creating a proxy when a

  connection requests one -> connection  shows "folder is empty". Now try

  to refresh with F5,colapse/expand/whatever (even with proxy now) -> folder

  still empty

            - WP1

 

Comments:

- Mathias: Refresh/F5 do definitely not work. Nevertheless collapsing and expanding again is fixed. Refreshing is not yet implemented for connections.

 

 

* F1: GridFTP shows "Unspecified IO problem" in the editor area when

  opening a 0-sized file

            - WP2: Make sure 0-sized files are handled the same as in local connections

 

Comments:

- Mathias: As far as I know this is fixed in HEAD. But I’m sure Mateusz can give a more detailed answer.

 

 

* F2: (gLite, SRM) Storage elements do not offer SRM as an option for mount?

  Is SRM v2.2 ready/working?

            - WP2

 

Comments:

 

 

* F3: If a project is associated to a unique VO, and a VO determines the resources,

  then a project is MW-specific. Why do we offer the choice to submit a

  job-description to GRIA or gLite at the same time? GRIA resources shouldn't

  be even accessible within a gLite project and vice-versa

            - WP2

 

Comments:

- Mathias: I had a discussion about this in Linz with Pawel. I personally share the position of Ariel. Nevertheless Pawel had also some arguments for the other possibility. We should discuss this after M4.

 

 

* F4: in the job submission wizard we should allow the user to give his job a name

  (if JSDL is named "myJob.jsdl" the default job name should be "myJob[N]" and not

  ".myJob.job")

            - WP2

 

Comments:

 

 

* G1: New connection wizard: the gecl and null protocols are useless as options

  (can be selected/filled in and things get enabled but a NPE is thrown)

            - WP1: Filter out those entries

 

Comments:

- Mathias: Fixed in HEAD. Furthermore implemented sorting for the remaining schemes.

 

 

* G2: Context menus: they do make little sense sometimes

            - WP2: "Mount SRM" / "Mount LFC" appear for ALL service types,

              EXCEPT for the SEs!

            - WP3: Move the Simple tests context entry together with the

              "Test service..." entry (and not above Refresh)!!

            - ALL: Order / collect the entries logically, ie,

                        New

                        ---

                        Cut,copy,paste,...,refresh   (ie, all global and eclipse actions)

                        ---

                        Functionality entries (Mount...,Monitor CE,Open SSH,etc)

                        ---

                        Test entries (Simple tests,STest <service>...)

                        ---

                        (Properties)

 

Comments:

- Mathias: I think the context menus need a general rework. Not applicable for M4 but we should definitely keep this in mind!

 

 

* G3: All Services (even BDII, WMS...) display the "Mount SRM/LFC" actions!

  And some of the "Other Services" are marked as unsupported, but the "Mount SRM"

  action seems to work...

            - WP2: check/update list of supported services and the actions offered

 

Comments:

 

 

* G4: Service "lcg-file-catalog" marked as unsupported

            - WP2: Is supported, should behave the same as "lcg-local-file-catalog"

 

Comments:

 

 

* G5: Simple tests: do we really need a dialog w/parameters for a ping test?

            - WP3: Suggestion: make a 3 ping test without any further intermediate

              action from the user and show the output briefly, eventually even a

              plain "OK"/"Failure" with a "More details" whit the full ping output

 

Comments:

- Mathias: Well, in principle I like the dialog and I like to have options even for a ping test. Nevertheless this particular dialog is somehow ugly (sorry Harald) and needs some cosmetics.

 

 

* G6: Service tests: the wizard should not automatically jump to the

  VO selection page even if there is only one test to select (cert. lifetime test)

            - WP3: Use normal wizard behaviour

 

Comments:

 

 

* G7: Certificate lifetime test: wizard title is missing, descriptions should be

  improved, difficult to understand.

            - WP3

 

Comments:

 

 

* G7: Certificate lifetime test: very 'strange' "workflow" ;-)

  1- If i select "Test storage element..." form the context menu of a given SE, i don't

  want to be asked again which resources to test!

  2- Moreover, after choosing that action, i land in the "Project selection page" (uh??),

  and i can go _back_ one page to see that there is only a "Certificate lifetime test"

  available (see G6).

            - WP3: reorganize the wizard, removing the Project and resource selection steps

 

Comments:

 

 

* G8: CE Monitoring: the process status viewer shows only the PIDs, what is

  the usecase there?

            - WP4: Make the view provide useful data or remove the Monitor Computing

              Resource context action?

 

Comments:

 

 

* G9: why does the CA import wizard show EU-PMA and PEM certificate loaders for local imports?

  What is the difference, shouldn't we have a descriptive title? (user doesn't care what

  EU-PMA is, aren't we importing from disk here?)

            - WP1

 

Comments:

- Mathias: Yepp, some good points here. Nevertheless nothing for M4. We have to discuss this when you are back Ariel.

 

 

* G10: in the job view, the ID column takes a lot of space in a prominent place and shifts

  the status column to the right.

            - WP2: either allow the user to reoder the view's columns or move "ID" to the right

 

Comments:

 

 

* D1: Welcome/Tutorials screen: the "Grid" tutorials are at the far bottom

  of the page (not visible w/o scrolling)

            - WP1: Move them to the top of the second column or some more

              prominent place

 

Comments:

- Mathias: Mmmh, not sure if this is even possible. I would rather think it isn’t. But maybe Markus can clarify here.

 

 

* D2: Help system: the "Getting Started" page content is not in sync with TOC

            - WP1

 

Comments:

- Mathias: You’re right but I object that this is WP1 responsibility. It is out of sync (and also other things are) because things are added by somebody but the corresponding links or referring pages are not updated. So if you add something to the help please verify that everything is still synced afterwards.

 

 

* D3: Help system: some features are not mentioned (Import VO...)

            - ALL: Update help

 

Comments:

- Mathias: The import VO was added a few weeks ago (as part of “VO settings”).

 

 

* D4: Cheatsheets: some tasks involve opening a modal dialog -> user cannot

  read/see/click on the cheatsheets anymore unless he "clicks to perform"

            - ALL: Dont break tasks at those places or find a better way around...

 

Comments:

 

 

* D5: Cheatsheets: strange error meassage appearing/blocking if user doesn't

  exactly follow some precise sequence of clicks

            - WP1: Solve #217424

 

Comments:

- Mathias: Yepp, this is a tricky one. Will see what I can do.


Back to the top