Skip to content

Remove Nuage Network Provider#9953

Merged
Fryguy merged 3 commits into
ManageIQ:masterfrom
agrare:remove_nuage_network_provider
Apr 9, 2026
Merged

Remove Nuage Network Provider#9953
Fryguy merged 3 commits into
ManageIQ:masterfrom
agrare:remove_nuage_network_provider

Conversation

@agrare

@agrare agrare commented Apr 2, 2026

Copy link
Copy Markdown
Member

Drop all references to Nuage, most of these should have been implemented differently anyway.

I see a number of references to e.g.:

          Array [
            "Nuage Zone", 
            11,
          ],

In e.g. app/javascript/spec/ansible-edit-catalog-form/__snapshots__/ansible-edit-catalog-form.spec.js.snap

I'm not sure if these have to be deleted also?

ManageIQ/manageiq#23759

@agrare agrare requested a review from a team as a code owner April 2, 2026 20:13
Comment thread app/helpers/application_helper/button/ems_network.rb
@GilbertCherrie

GilbertCherrie commented Apr 2, 2026

Copy link
Copy Markdown
Member

@agrare can you run yarn jest -u to update the snaps, you might need to run that to get rid of the snapshot file refrences

Comment on lines -319 to -320
google_net = FactoryBot.create(:ems_google_network)
get :edit, :params => { :id => google_net.id}

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This was a good example of using a specific provider just because it happens to support/not-support a feature versus using stub_supports/stub_supports_not

@agrare

agrare commented Apr 2, 2026

Copy link
Copy Markdown
Member Author

@GilbertCherrie I see a few backtraces but they don't look related to nuage:

 RUNS  app/javascript/spec/ansible-repository-form/helpers.spec.js
 RUNS  app/javascript/spec/http_api.spec.js
 RUNS  app/javascript/spec/ansible-playbook-workflow/ansible-playbook-workflow.spec.js
 RUNS  app/javascript/spec/textual_summary/tests/simple_table.test.js
 RUNS  app/javascript/spec/textual_summary/tests/textual_summary.test.js
 RUNS  app/javascript/spec/flavor-form/flavor-form.spec.js
 RUNS  app/javascript/spec/pxe-iso-image-form/pxe-iso-image-form.spec.js
 RUNS  app/javascript/spec/service-form/edit-service-form.spec.js
  console.error
    Warning: React does not recognize the `loadingMessage` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `loadingmessage` instead. If you accidentally passed it from a parent component, remove it from the DOM element.
        in select (created by Select)
        in div (created by Select)
        in div (created by Select)
        in div (created by Select)
        in Select (created by ClearedSelect)
        in ClearedSelect (created by Select)

In the end it says everything passes and no files are modified

A worker process has failed to exit gracefully and has been force exited. This is likely caused by tests leaking due to improper teardown. Try running with --detectOpenHandles to find leaks.
Test Suites: 159 passed, 159 total
Tests:       704 passed, 704 total
Snapshots:   306 passed, 306 total
Time:        10.996 s
Ran all test suites.

@GilbertCherrie

Copy link
Copy Markdown
Member

@agrare don't worry about the warnings, it's strange that none of the files were changed. We might need to delete the nuage provider references somewhere else

@GilbertCherrie

GilbertCherrie commented Apr 2, 2026

Copy link
Copy Markdown
Member

@agrare oh it looks like it is just in app/javascript/spec/ansible-edit-catalog-form/mockdata.js, which is our spec file. This can just be deleted and that should update the snapshot files

@agrare agrare force-pushed the remove_nuage_network_provider branch from 9644f71 to 024865a Compare April 2, 2026 20:35
@agrare

agrare commented Apr 2, 2026

Copy link
Copy Markdown
Member Author

Okay I did that @GilbertCherrie there are still a number of these, if there is something else I need to do let me know.

@GilbertCherrie

Copy link
Copy Markdown
Member

@agrare I checked out your branch and it looks fine now. If you are referring to the references in the language json files don't worry about those. Those don't affect anything and should be updated when we update the translation files.

@Fryguy Fryguy assigned GilbertCherrie and unassigned Fryguy Apr 3, 2026
@Fryguy

Fryguy commented Apr 3, 2026

Copy link
Copy Markdown
Member

@GilbertCherrie assigning this to you - when you're ready feel free to merge - I'll monitor and then merge the core PR when this is in.

...options,
};

const baseUrl = '/api/providers?expand=resources&attributes=id,name,supports_cloud_network_create&filter[]=supports_cloud_network_create=true';

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@agrare agrare force-pushed the remove_nuage_network_provider branch from c225538 to a1aa591 Compare April 3, 2026 14:15
@agrare

agrare commented Apr 3, 2026

Copy link
Copy Markdown
Member Author

I was able to drop that app/javascript/helpers/network-providers.js as it was only used in app/javascript/spec/network-router-form/network-router-form.spec.js and could be easily removed from there

@agrare

agrare commented Apr 3, 2026

Copy link
Copy Markdown
Member Author

I don't think the whitesource failure is related to this PR

@agrare agrare force-pushed the remove_nuage_network_provider branch from 66c1170 to fa2671b Compare April 7, 2026 18:44
@miq-bot

miq-bot commented Apr 7, 2026

Copy link
Copy Markdown
Member

Checked commits agrare/manageiq-ui-classic@1068246~...fa2671b with ruby 3.3.10, rubocop 1.86.0, haml-lint 0.72.0, and yamllint 1.37.1
4 files checked, 3 offenses detected

app/helpers/application_helper/toolbar/ems_network_center.rb

app/helpers/application_helper/toolbar/ems_networks_center.rb

@Fryguy Fryguy merged commit e1134e2 into ManageIQ:master Apr 9, 2026
11 checks passed
@agrare agrare deleted the remove_nuage_network_provider branch April 9, 2026 18:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants